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

Bug #19413 php_cs thinks I haven't used a parameter when I have
Submitted: 2012-05-10 02:53 UTC
From: petdance Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.3.3)
PHP Version: 5.3.6 OS: CentOS 5 Linux
Roadmaps: (Not assigned)    
Comments Add Comment Add patch

Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
Solve the problem : 41 - 5 = ?

 [2012-05-10 02:53 UTC] petdance (Andy Lester)
Description: ------------ I ran php_cs on a source file that had a function in it with an argument. php_cs incorrectly said that the function never used the argument but it did. I was able to reduce the problem down to a single sample file, in the "Test script" section. The spacing is important, as noted in the source There may be other ways to get this to fail, but this one is the smallest I was able to come up with and make it Test script: --------------- <?php function foo( $parameter ) { $wango = <<<HERE 1 Must be a HEREdoc of at least one line HERE; $x = $parameter; // This line must be immediately after the HERE; with no intervening blank lines. $tango = <<<HERE 1 Must be a HEREdoc 2 3 4 5 6 7 8 9 at least 9 lines long HERE; } Actual result: -------------- FILE: /home/alester/scraps/sniffbugs.php -------------------------------------------------------------------------------- FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 2 | WARNING | The method parameter $parameter is never used --------------------------------------------------------------------------------


 [2012-05-10 10:57 UTC] squiz (Greg Sherwood)
-Assigned To: +Assigned To: squiz
Fixed in git repo: d8e03c4a11b25fa
 [2012-05-11 10:41 UTC] squiz (Greg Sherwood)
-Status: Assigned +Status: Closed