@robocoder opened this Issue on August 11th 2010 Contributor

In #956, we added a check for the xml extension, but it's only a warning if missing.

However, support requests on the forum (e.g., data is collected but graphs don't display) show that this is insufficient.

@robocoder commented on December 1st 2010 Contributor

The xml extension is enabled in a default php build (like SPL).

@mattab commented on December 20th 2010 Member

If only the graphs fail to display, then the XML extension is optional (since the report tables will display fine).
However it would be nice to display it nicely (eg. when a user has a partial working Piwik, let him know that the graph don't work because he is missing xml extension)

@robocoder commented on December 21st 2010 Contributor

Temporarily re-opening for investigation. I'm recompiling php on my test box with --disable-xml (and --without-pear) to document what the full impact is.

@robocoder commented on December 21st 2010 Contributor

FYI the FreeBSD ports collection has a dependency check for:

mysql gd pdo pdo_mysql zlib iconv dom json session xml

@robocoder commented on December 21st 2010 Contributor

Base functionality (including graphs) works with --disable-xml.

Going to rebuild without libxml.

@robocoder commented on December 21st 2010 Contributor

Ok... compiling withput libxml also means no xml, xmlreader, xmlwriter, simplexml, or dom.

Graphs still work but:

  • Feedburner stats show:
Fatal error: Class 'SimpleXMLElement' not found in /home/apang/work/piwik/dev/trunk/plugins/ExampleFeedburner/ExampleFeedburner.php on line 112
  • Piwik.org blog shows:
Fatal error: Call to undefined function libxml_use_internal_errors() in /home/apang/work/piwik/dev/trunk/libs/Zend/Feed.php on line 194
  • SEO widget is empty
@robocoder commented on December 21st 2010 Contributor

(In [3504]) fixes #1575 - removed special logic re: json and xml; add generic check for "other extensions" (json, libxml, dom, and SimpleXML)

@robocoder commented on December 21st 2010 Contributor

(In [3505]) refs #1575 - handle missing "session" extension

This Issue was closed on December 21st 2010
Powered by GitHub Issue Mirror