Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.2.2

Bug #556 Notices on XML_Beautifier execution
Submitted: 2004-01-11 17:50 UTC Modified: 2004-01-12 15:40 UTC
From: brunobord at tele2 dot fr Assigned: schst
Status: Closed Package: XML_Beautifier
PHP Version: 4.3.3 OS: Windows XP
Roadmaps: (Not assigned)    
Subscription  


 [2004-01-11 17:50 UTC] brunobord at tele2 dot fr
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

Comments

 [2004-01-11 18:02 UTC] brunobord at tele2 dot fr
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.
 [2004-01-12 15:06 UTC] schst
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.
 [2004-01-12 15:40 UTC] schst
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