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

Bug #10983 phpcs outputs notices when checking invalid PHP
Submitted: 2007-05-08 18:38 UTC
From: cipri Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version CVS)
PHP Version: 5.2.1 OS: Fedora Core 6
Roadmaps: (Not assigned)    
Subscription  


 [2007-05-08 18:38 UTC] cipri (Cipriano Groenendal)
Description: ------------ phpcs outputs notices when it tries to check an invalid php file. Actual result: -------------- PHP Notice: Undefined offset: 5771 in /usr/share/pear/PHP/CodeSniffer/File.php on line 884 PHP Notice: Undefined offset: 5771 in /usr/share/pear/PHP/CodeSniffer/File.php on line 840 PHP Notice: Undefined index: content in /usr/share/pear/PHP/CodeSniffer/File.php on line 740 PHP Notice: Undefined index: line in /usr/share/pear/PHP/CodeSniffer/Standards/Generic/Sniffs/Files/LineLengthSniff.php on line 83 PHP Notice: Undefined index: content in /usr/share/pear/PHP/CodeSniffer/Standards/Generic/Sniffs/Files/LineLengthSniff.php on line 93 PHP Notice: Undefined index: line in /usr/share/pear/PHP/CodeSniffer/Standards/Generic/Sniffs/Files/LineLengthSniff.php on line 83 PHP Notice: Undefined index: content in /usr/share/pear/PHP/CodeSniffer/Standards/Generic/Sniffs/Files/LineLengthSniff.php on line 93 PHP Notice: Undefined offset: 5773 in /usr/share/pear/PHP/CodeSniffer/Standards/Generic/Sniffs/Files/LineLengthSniff.php on line 83 PHP Notice: Undefined offset: 5773 in /usr/share/pear/PHP/CodeSniffer/Standards/Generic/Sniffs/Files/LineLengthSniff.php on line 93 PHP Notice: Undefined index: code in /usr/share/pear/PHP/CodeSniffer/File.php on line 426 PHP Notice: Undefined index: code in /usr/share/pear/PHP/CodeSniffer/File.php on line 426 PHP Notice: Undefined index: code in /usr/share/pear/PHP/CodeSniffer/File.php on line 426

Comments

 [2007-05-08 18:40 UTC] cipri (Cipriano Groenendal)
The file I tried to check is available at http://www.cipri.com/pear/Bug-10983.txt
 [2007-05-08 22:17 UTC] squiz (Greg Sherwood)
Thank you for taking the time to report a problem with the package. This problem may have been already fixed by a previous change that is in the CVS of the package. Please log into CVS with: cvs -d :pserver:cvsread@cvs.php.net:/repository login and check out the CVS repository of this package and upgrade cvs -d :pserver:cvsread@cvs.php.net:/repository co pear/PHP_CodeSniffer pear upgrade pear/PHP_CodeSniffer/package2.xml or pear upgrade pear/PHP_CodeSniffer/package.xml If you are able to reproduce the bug with the latest CVS, please change the status back to "Open". Again, thank you for your continued support of PEAR. I fixed a similar issue testing a file with a parse error yesterday. The fix is in CVS at the moment, and I don't see any notices using this version. If you can test with the CVS version, let me know if it works for you.
 [2007-05-08 23:10 UTC] cipri (Cipriano Groenendal)
Just cvsup'd, and it works, thanks!
 [2007-05-09 01:32 UTC] squiz (Greg Sherwood)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.