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

From: 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 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


« previous php.pear.dev (#54057) next »