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

Bug #12645 error message is wrong.
Submitted: 2007-12-09 10:01 UTC Modified: 2007-12-09 23:01 UTC
From: renoiv Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.0.0RC3)
PHP Version: 5.2.5 OS:
Roadmaps: 1.0.0    
Subscription  


 [2007-12-09 10:01 UTC] renoiv (renoiv)
Description: ------------ The error message of Squiz_Sniffs_Arrays_ArrayDeclarationSniff is wrong. I am not good at English. If a sentence of this bug is a unclear meaning, I am sorry. Test script: --------------- <?php $value = in_array('1', array('1' , '2')); ?> Expected result: ---------------- [test@]$ phpcs --standard=Squiz ArrayDeclarationTest.php FILE: ArrayDeclarationTest.php -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 2 | ERROR | Expected 0 space between "'1'" and comma; 2 found -------------------------------------------------------------------------------- Actual result: -------------- [test@]$ phpcs --standard=Squiz ArrayDeclarationTest.php FILE: ArrayDeclarationTest.php -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 2 | ERROR | Expected 1 space between "'1'" and comma; 2 found --------------------------------------------------------------------------------

Comments

 [2007-12-09 23:01 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.