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

Bug #9844 throw new Exception(\n accidently reported as error but it ain't
Submitted: 2007-01-15 15:12 UTC
From: tkli at php dot net Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 0.3.0)
PHP Version: 5.2.0 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 : 11 + 2 = ?

 
 [2007-01-15 15:12 UTC] tkli at php dot net (Tom Klingenberg)
Description: ------------ codesniffer handles the throw new Exception() as a function call, but it isn't. that lead to reported errors with php code that is valid according to the pear standard. these error should not be reported. <phpcs> 179 | ERROR | Space after opening parenthesis of function call prohibited 181 | ERROR | Space before closing parenthesis of function call prohibited </phpcs> <php> /* validate version parameter */ if (self::_ISBNVersionIs($ver) == false) { throw new ISBN_Exception( 'ISBN Version parameter is not an ISBN Version' ); } </php> check the pear docs here: http://pear.php.net/manual/en/standars.errors.php additional info: this error occured in 0.2.0 as well. Test script: --------------- n/a

Comments

 [2007-01-15 22:35 UTC] squiz at php dot net (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. Added check for "new" keyword before processing. If found, the "function call" is ignored.