Description: ------------ Using XML_Beautifier with ERROR_REPORTING set ton E_ALL produces a list of notices (listed below). The script used is example1.php, supplied with the latest version of this package (1.0) No change made to my php.ini, it's the default setup. Reproduce code: --------------- /* it's all in the example1.php file */ Expected result: ---------------- No notice. This setup (error_reporting set to "all) is supposed to be definitely used for secure coding. Actual result: -------------- Notice: Undefined index: children in c:\easyphp\www\pear\PEAR\XML\Beautifier\Tokenizer.php on line 352 Notice: Undefined index: children in c:\easyphp\www\pear\PEAR\XML\Beautifier\Tokenizer.php on line 352 Notice: Undefined index: children in c:\easyphp\www\pear\PEAR\XML\Beautifier\Tokenizer.php on line 352 Notice: Undefined variable: xml in c:\easyphp\www\pear\PEAR\XML\Beautifier\Renderer.php on line 146 Notice: Undefined variable: xml in c:\easyphp\www\pear\PEAR\XML\Beautifier\Renderer.php on line 146

Add-on : this is not exactly the php.ini-dist. The package I'm using for testing purposes (Easyphp) sets ERROR_REPORTING to E_ALL by default. But I still think that this notices shall be removed, to use the up-to-date security recommendations.

This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.

Thank you for your bug report. This issue has already been fixed in the latest released version of the package, which you can download at http://pear.php.net/packages.php