Re: About PHPCodeSniffer 1.3.0RC2 and IF multi-lines check

In the same category

ERROR | Each line in a multi-line IF statement must begin with a | boolean operator

We have : <code> if ($fileinfo->getMTime() <=

      (time() - $this->options['gc_maxlifetime'])

) {

      unlink($fileinfo->getPathname());

} </code> Le 05/02/2011 09:42, Laurent Laville a écrit :

Check of IF multi-lines detect also the same error ERROR | Each line in a multi-line IF statement must begin with a | boolean operator for <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