Re: About PHPCodeSniffer 1.3.0RC2 and IF multi-lines check
| From: | Laurent Laville | Date: | Sat, 05 Feb 2011 08:42:29 +0000 |
| Subject: | Re: About PHPCodeSniffer 1.3.0RC2 and IF multi-lines check | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-54057@lists.php.net to get a copy of this message | ||
Check of IF multi-lines detect also the same error
ERROR | Each line in a multi-line IF statement must begin with a
| boolean operatorfor <code> if (!$reference
instanceof PHP_CompatInfo_Reference_PluginsAbstract) {
// ....} </code> Laurent Le 04/02/2011 20:42, Laurent Laville a écrit :
Hello Greg, I don't know if you've already noticed this issue, and as I didn't find it on the PEAR bug tracker, I ask it here. While I giving a clean-up to my PHP_CompatInfo v2 code, I got this case Look at Lines 691 - 696 of https://github.com/llaville/php-compat-info/blob/master/PHP/CompatInfo.php A preg_match on a multi-line to respect the 85 characters limit. I got these error as it was a IF with boolean conditions ERROR | Each line in a multi-line IF statement must begin with a | boolean operator Is it possible to FIX it ? I used 1.3.0RC2 version of Code Sniffer BTW, I think you can close my request [1] since it's now part of 1.3.0RC2 Thanks Laurent [1] http://pear.php.net/bugs/bug.php?id=18094