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

Bug #20308 Squiz.ValidVariableNameSniff - wrong variable usage
Submitted: 2014-06-22 16:07 UTC
From: aik099 Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.5.3)
PHP Version: 5.4.20 OS:
Roadmaps: (Not assigned)    
Subscription  
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:
MUST BE VALID
Solve the problem : 30 + 44 = ?

 
 [2014-06-22 16:07 UTC] aik099 (Alexander Obuhovich)
Description: ------------ On http://screencast.com/t/ucz8FAcwGi I'm comparing ValidVariableName sniffs from Squiz and Zend and it's obvious, that in Squiz (on the left) the "$bracket = $objOperator = " should be replaced just with "$bracket =". The $objOperator is assigned later on in fact.

Comments

 [2014-06-24 06:50 UTC] squiz (Greg Sherwood)
-Status: Open +Status: Closed -Assigned To: +Assigned To: squiz
Yep, definitely a typo. Thanks. Committed the fix here: https://github.com/squizlabs/PHP_CodeSniffer/commit/fc9ddddc6cb662389acf567aa59 35e312de0348f