timw4mail / HummingBirdAnimeClient

build/CodeIgniter/Sniffs/WhiteSpace/ElseOnNewLineSniff.php

Indentation +36 added lines, -36 removed lines

@@ -33,48 +33,48 @@
 class ElseOnNewLineSniff implements Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_ELSE,
-            T_ELSEIF,
-        );
-    }//end register()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_ELSE,
+			T_ELSEIF,
+		);
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        $else_token = $tokens[$stackPtr];
-        $previous_non_blank_token_ptr = $phpcsFile->findPrevious(array(T_WHITESPACE), $stackPtr - 1, null, true);
+		$else_token = $tokens[$stackPtr];
+		$previous_non_blank_token_ptr = $phpcsFile->findPrevious(array(T_WHITESPACE), $stackPtr - 1, null, true);
 
-        if (false === $previous_non_blank_token_ptr) {
-        	// else is no preceded with any symbol, but it is not the responsibility of this sniff.
-        	return;
-        }
+		if (false === $previous_non_blank_token_ptr) {
+			// else is no preceded with any symbol, but it is not the responsibility of this sniff.
+			return;
+		}
 
-        $previous_non_blank_token = $tokens[$previous_non_blank_token_ptr];
-        if ($previous_non_blank_token['line'] === $else_token['line']) {
-            $error = '"' . $else_token['content'] . '" should be on a new line.';
-            $phpcsFile->addError($error, $stackPtr);
-        }
+		$previous_non_blank_token = $tokens[$previous_non_blank_token_ptr];
+		if ($previous_non_blank_token['line'] === $else_token['line']) {
+			$error = '"' . $else_token['content'] . '" should be on a new line.';
+			$phpcsFile->addError($error, $stackPtr);
+		}
 
-    }//end process()
+	}//end process()
 
 
 }//end class

build/CodeIgniter/Sniffs/WhiteSpace/DisallowWitheSpaceAroundPhpTagsSniff.php

Indentation +49 added lines, -49 removed lines

@@ -34,60 +34,60 @@
 class DisallowWitheSpaceAroundPhpTagsSniff implements Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-                T_OPEN_TAG,
-                T_CLOSE_TAG
-               );
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+				T_OPEN_TAG,
+				T_CLOSE_TAG
+			   );
 
-    }//end register()
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        $php_tag_token = $tokens[$stackPtr];
-        $php_tag_code = $php_tag_token['code'];
+		$php_tag_token = $tokens[$stackPtr];
+		$php_tag_code = $php_tag_token['code'];
 
-        if (T_OPEN_TAG === $php_tag_code) {
-            // opening php tag should be the first token.
-            // any whitespace beofre an opening php tag is tokenized
-            // as T_INLINE_HTML, so no need to check the content of the token.
-            $isFirst = 0 === $stackPtr;
-            if ( ! $isFirst) {
-                $error = 'Any char before the opening PHP tag is prohibited. Please remove newline or indentation before the opening PHP tag.';
-                $phpcsFile->addError($error, $stackPtr);
-            }
-        } else {
-            // if (T_CLOSE_TAG === $php_tag_code)
-            // closing php tag should be the last token
-            // and it must not contain any whitespace.
-            $php_tag_string = $php_tag_token['content'];
-            $isLast = count($tokens) - 1 === $stackPtr;
-            // both of the two closing php tags contains 2 chars exactly.
-            $containsEndTagOnly = strlen($php_tag_string) > 2;
-            if ( ! $isLast || ! $containsEndTagOnly ) {
-                $error = 'Any char after the closing PHP tag is prohibited. Please removes newline or spaces after the closing PHP tag.';
-                $phpcsFile->addError($error, $stackPtr);
-            }
-        }
-    }//end process()
+		if (T_OPEN_TAG === $php_tag_code) {
+			// opening php tag should be the first token.
+			// any whitespace beofre an opening php tag is tokenized
+			// as T_INLINE_HTML, so no need to check the content of the token.
+			$isFirst = 0 === $stackPtr;
+			if ( ! $isFirst) {
+				$error = 'Any char before the opening PHP tag is prohibited. Please remove newline or indentation before the opening PHP tag.';
+				$phpcsFile->addError($error, $stackPtr);
+			}
+		} else {
+			// if (T_CLOSE_TAG === $php_tag_code)
+			// closing php tag should be the last token
+			// and it must not contain any whitespace.
+			$php_tag_string = $php_tag_token['content'];
+			$isLast = count($tokens) - 1 === $stackPtr;
+			// both of the two closing php tags contains 2 chars exactly.
+			$containsEndTagOnly = strlen($php_tag_string) > 2;
+			if ( ! $isLast || ! $containsEndTagOnly ) {
+				$error = 'Any char after the closing PHP tag is prohibited. Please removes newline or spaces after the closing PHP tag.';
+				$phpcsFile->addError($error, $stackPtr);
+			}
+		}
+	}//end process()
 
 
 }//end class

build/CodeIgniter/Sniffs/WhiteSpace/LogicalNotSpacingSniff.php

Indentation +31 added lines, -31 removed lines

@@ -33,41 +33,41 @@
 class LogicalNotSpacingSniff implements Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_BOOLEAN_NOT,
-        );
-    }//end register()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_BOOLEAN_NOT,
+		);
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        $operator_token = $tokens[$stackPtr];
+		$operator_token = $tokens[$stackPtr];
 
-        $previous_token = $tokens[$stackPtr - 1];
-        $next_token = $tokens[$stackPtr + 1];
-        if (T_WHITESPACE !== $previous_token['code'] || T_WHITESPACE !== $next_token['code']) {
-            $error = 'Logical operator ! should always be preceded and followed with a whitespace.';
-            $phpcsFile->addError($error, $stackPtr);
-        }
-    }//end process()
+		$previous_token = $tokens[$stackPtr - 1];
+		$next_token = $tokens[$stackPtr + 1];
+		if (T_WHITESPACE !== $previous_token['code'] || T_WHITESPACE !== $next_token['code']) {
+			$error = 'Logical operator ! should always be preceded and followed with a whitespace.';
+			$phpcsFile->addError($error, $stackPtr);
+		}
+	}//end process()
 
 
 }//end class

build/CodeIgniter/Sniffs/WhiteSpace/DisallowSpaceIndentSniff.php

Indentation +41 added lines, -41 removed lines

@@ -33,53 +33,53 @@
 class DisallowSpaceIndentSniff implements Sniff
 {
 
-    /**
-     * A list of tokenizers this sniff supports.
-     *
-     * @var array
-     */
-    public $supportedTokenizers = array(
-                                   'PHP',
-                                   'JS',
-                                   'CSS',
-                                  );
+	/**
+	 * A list of tokenizers this sniff supports.
+	 *
+	 * @var array
+	 */
+	public $supportedTokenizers = array(
+								   'PHP',
+								   'JS',
+								   'CSS',
+								  );
 
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(T_WHITESPACE);
-    }//end register()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(T_WHITESPACE);
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile All the tokens found in the document.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile All the tokens found in the document.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        // Make sure this is whitespace used for indentation.
-        $line = $tokens[$stackPtr]['line'];
-        if ($stackPtr > 0 && $tokens[($stackPtr - 1)]['line'] === $line) {
-            return;
-        }
+		// Make sure this is whitespace used for indentation.
+		$line = $tokens[$stackPtr]['line'];
+		if ($stackPtr > 0 && $tokens[($stackPtr - 1)]['line'] === $line) {
+			return;
+		}
 
-        if (strpos($tokens[$stackPtr]['content'], " ") !== false) {
-            $error = 'Tabs must be used to indent lines; spaces are not allowed for code indentation';
-            $phpcsFile->addError($error, $stackPtr, 'SpacesUsedForIndentation');
-        }
-    }//end process()
+		if (strpos($tokens[$stackPtr]['content'], " ") !== false) {
+			$error = 'Tabs must be used to indent lines; spaces are not allowed for code indentation';
+			$phpcsFile->addError($error, $stackPtr, 'SpacesUsedForIndentation');
+		}
+	}//end process()
 
 
 }//end class

build/CodeIgniter/Sniffs/Strings/DoubleQuoteUsageSniff.php

Indentation +123 added lines, -123 removed lines

@@ -327,137 +327,137 @@
  */
 class DoubleQuoteUsageSniff extends VariableUsageSniff
 {
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_DOUBLE_QUOTED_STRING,
-            T_CONSTANT_ENCAPSED_STRING,
-        );
-    }//end register()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_DOUBLE_QUOTED_STRING,
+			T_CONSTANT_ENCAPSED_STRING,
+		);
+	}//end register()
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        // no variable are in the string from here
-        $tokens = $phpcsFile->getTokens();
-        $qtString = $tokens[$stackPtr]['content'];
-        // makes sure that it is about a double quote string,
-        // since variables are not parsed out of double quoted string
-        $open_qt_str = substr($qtString, 0, 1);
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		// no variable are in the string from here
+		$tokens = $phpcsFile->getTokens();
+		$qtString = $tokens[$stackPtr]['content'];
+		// makes sure that it is about a double quote string,
+		// since variables are not parsed out of double quoted string
+		$open_qt_str = substr($qtString, 0, 1);
 
-        // clean the enclosing quotes
-        $qtString = substr($qtString, 1, strlen($qtString) - 1 - 1);
+		// clean the enclosing quotes
+		$qtString = substr($qtString, 1, strlen($qtString) - 1 - 1);
 
-        if (0 === strcmp($open_qt_str, '"')) {
-            $this->processDoubleQuotedString($phpcsFile, $stackPtr, $qtString);
-        } else if (0 === strcmp($open_qt_str, "'")) {
-            $this->processSingleQuotedString($phpcsFile, $stackPtr, $qtString);
-        }
-    }//end process()
+		if (0 === strcmp($open_qt_str, '"')) {
+			$this->processDoubleQuotedString($phpcsFile, $stackPtr, $qtString);
+		} else if (0 === strcmp($open_qt_str, "'")) {
+			$this->processSingleQuotedString($phpcsFile, $stackPtr, $qtString);
+		}
+	}//end process()
 
 
-    /**
-     * Processes this test, when the token encountered is a double-quoted string.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     * @param string               $qtString  The double-quoted string content,
-     *                                        i.e. without quotes.
-     *
-     * @return void
-     */
-    protected function processDoubleQuotedString (File $phpcsFile, $stackPtr, $qtString)
-    {
-        // so there should be at least a single quote or a special char
-        // if there are the 2 kinds of quote and no special char, then add a warning
-        $has_variable = parent::processDoubleQuotedString($phpcsFile, $stackPtr, '"'.$qtString.'"');
-        $has_specific_sequence = $this->_hasSpecificSequence($qtString);
-        $dbl_qt_at = strpos($qtString, '"');
-        $smpl_qt_at = strpos($qtString, "'");
-        if (false === $has_variable && false === $has_specific_sequence
-            && false === $smpl_qt_at
-        ) {
-            $error = 'Single-quoted strings should be used unless it contains variables, special chars like \n or single quotes.';
-            $phpcsFile->addError($error, $stackPtr);
-        } else if (false !== $smpl_qt_at && false !== $dbl_qt_at
-            && false === $has_variable && false === $has_specific_sequence
-        ) {
-            $warning = 'It is encouraged to use a single-quoted string, since it doesn\'t contain any variable nor special char though it mixes single and double quotes.';
-            $phpcsFile->addWarning($warning, $stackPtr);
-        }
-    }//end processDoubleQuotedString()
+	/**
+	 * Processes this test, when the token encountered is a double-quoted string.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 * @param string               $qtString  The double-quoted string content,
+	 *                                        i.e. without quotes.
+	 *
+	 * @return void
+	 */
+	protected function processDoubleQuotedString (File $phpcsFile, $stackPtr, $qtString)
+	{
+		// so there should be at least a single quote or a special char
+		// if there are the 2 kinds of quote and no special char, then add a warning
+		$has_variable = parent::processDoubleQuotedString($phpcsFile, $stackPtr, '"'.$qtString.'"');
+		$has_specific_sequence = $this->_hasSpecificSequence($qtString);
+		$dbl_qt_at = strpos($qtString, '"');
+		$smpl_qt_at = strpos($qtString, "'");
+		if (false === $has_variable && false === $has_specific_sequence
+			&& false === $smpl_qt_at
+		) {
+			$error = 'Single-quoted strings should be used unless it contains variables, special chars like \n or single quotes.';
+			$phpcsFile->addError($error, $stackPtr);
+		} else if (false !== $smpl_qt_at && false !== $dbl_qt_at
+			&& false === $has_variable && false === $has_specific_sequence
+		) {
+			$warning = 'It is encouraged to use a single-quoted string, since it doesn\'t contain any variable nor special char though it mixes single and double quotes.';
+			$phpcsFile->addWarning($warning, $stackPtr);
+		}
+	}//end processDoubleQuotedString()
 
 
-    /**
-     * Processes this test, when the token encountered is a single-quoted string.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     * @param string               $qtString  The single-quoted string content,
-     *                                        i.e. without quotes.
-     *
-     * @return void
-     */
-    protected function processSingleQuotedString (File $phpcsFile, $stackPtr, $qtString)
-    {
-        // if there is single quotes without additional double quotes,
-        // then user is allowed to use double quote to avoid having to
-        // escape single quotes. Don't add the warning, if an error was
-        // already added, because a variable was found in a single-quoted
-        // string.
-        $has_variable = parent::processSingleQuotedString($phpcsFile, $stackPtr, "'".$qtString."'");
-        $dbl_qt_at = strpos($qtString, '"');
-        $smpl_qt_at = strpos($qtString, "'");
-        if (false === $has_variable && false !== $smpl_qt_at && false === $dbl_qt_at) {
-            $warning = 'You may also use double-quoted strings if the string contains single quotes, so you do not have to use escape characters.';
-            $phpcsFile->addWarning($warning, $stackPtr);
-        }
-    }//end processSingleQuotedString()
+	/**
+	 * Processes this test, when the token encountered is a single-quoted string.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 * @param string               $qtString  The single-quoted string content,
+	 *                                        i.e. without quotes.
+	 *
+	 * @return void
+	 */
+	protected function processSingleQuotedString (File $phpcsFile, $stackPtr, $qtString)
+	{
+		// if there is single quotes without additional double quotes,
+		// then user is allowed to use double quote to avoid having to
+		// escape single quotes. Don't add the warning, if an error was
+		// already added, because a variable was found in a single-quoted
+		// string.
+		$has_variable = parent::processSingleQuotedString($phpcsFile, $stackPtr, "'".$qtString."'");
+		$dbl_qt_at = strpos($qtString, '"');
+		$smpl_qt_at = strpos($qtString, "'");
+		if (false === $has_variable && false !== $smpl_qt_at && false === $dbl_qt_at) {
+			$warning = 'You may also use double-quoted strings if the string contains single quotes, so you do not have to use escape characters.';
+			$phpcsFile->addWarning($warning, $stackPtr);
+		}
+	}//end processSingleQuotedString()
 
-    /**
-     * Return TRUE, if a sequence of chars that is parsed in a specific way
-     * in double-quoted strings is found, FALSE otherwise.
-     *
-     * @param string $string String in which sequence of special chars will
-     * be researched.
-     *
-     * @return TRUE, if a sequence of chars that is parsed in a specific way
-     * in double-quoted strings is found, FALSE otherwise.
-     *
-     * @link http://www.php.net/manual/en/language.types.string.php#language.types.string.syntax.double
-     */
-    private function _hasSpecificSequence($string)
-    {
-        $hasSpecificSequence = FALSE;
-        $specialMeaningStrs = array('\n', '\r', '\t', '\v', '\f');
-        foreach ($specialMeaningStrs as $splStr) {
-            if (FALSE !== strpos($string, $splStr)) {
-                $hasSpecificSequence = TRUE;
-            }
-        }
-        $specialMeaningPtrns = array('\[0-7]{1,3}', '\x[0-9A-Fa-f]{1,2}');
-        foreach ($specialMeaningPtrns as $splPtrn) {
-            if (1 === preg_match("/{$splPtrn}/", $string)) {
-                $hasSpecificSequence = TRUE;
-            }
-        }
-        return $hasSpecificSequence;
-    }//end _hasSpecificSequence()
+	/**
+	 * Return TRUE, if a sequence of chars that is parsed in a specific way
+	 * in double-quoted strings is found, FALSE otherwise.
+	 *
+	 * @param string $string String in which sequence of special chars will
+	 * be researched.
+	 *
+	 * @return TRUE, if a sequence of chars that is parsed in a specific way
+	 * in double-quoted strings is found, FALSE otherwise.
+	 *
+	 * @link http://www.php.net/manual/en/language.types.string.php#language.types.string.syntax.double
+	 */
+	private function _hasSpecificSequence($string)
+	{
+		$hasSpecificSequence = FALSE;
+		$specialMeaningStrs = array('\n', '\r', '\t', '\v', '\f');
+		foreach ($specialMeaningStrs as $splStr) {
+			if (FALSE !== strpos($string, $splStr)) {
+				$hasSpecificSequence = TRUE;
+			}
+		}
+		$specialMeaningPtrns = array('\[0-7]{1,3}', '\x[0-9A-Fa-f]{1,2}');
+		foreach ($specialMeaningPtrns as $splPtrn) {
+			if (1 === preg_match("/{$splPtrn}/", $string)) {
+				$hasSpecificSequence = TRUE;
+			}
+		}
+		return $hasSpecificSequence;
+	}//end _hasSpecificSequence()
 
 }//end class
 

build/CodeIgniter/Sniffs/Commenting/InlineCommentSniff.php

Indentation +149 added lines, -149 removed lines

@@ -32,155 +32,155 @@
 
 class InlineCommentSniff implements Sniff
 {
-    /**
-     * @var int Limit defining long comments.
-     * Long comments count $longCommentLimit or more lines.
-     */
-    public $longCommentLimit = 5;
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_COMMENT
-        );
-    }//end register()
-
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
-
-        // keep testing only if it's about the first comment of the block
-        $previousCommentPtr = $phpcsFile->findPrevious($tokens[$stackPtr]['code'], $stackPtr - 1);
-        if ($tokens[$previousCommentPtr]['line'] !== $tokens[$stackPtr]['line'] - 1) {
-            if (TRUE !== $this->_checkCommentStyle($phpcsFile, $stackPtr)) {
-                return;
-            }
-
-            $commentLines = $this->_getCommentBlock($phpcsFile, $stackPtr);
-
-            if (count($commentLines) >= $this->longCommentLimit) {
-                $this->_checkBlankLinesAroundLongComment($phpcsFile, $commentLines);
-            }
-        }
-    }//end process()
-
-
-    /**
-     * Add error to $phpcsFile, if comment pointed by $stackPtr doesn't start
-     * with '//'.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        that has to be a comment.
-     *
-     * @return bool TRUE if the content of the token pointed by $stackPtr starts
-     *              with //, FALSE if an error was added to $phpcsFile.
-     */
-    private function _checkCommentStyle(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
-        if ($tokens[$stackPtr]['content']{0} === '#') {
-            $error  = 'Perl-style comments are not allowed; use "// Comment" or DocBlock comments instead';
-            $phpcsFile->addError($error, $stackPtr, 'WrongStyle');
-            return FALSE;
-        } else if (substr($tokens[$stackPtr]['content'], 0, 2) === '/*'
-            || $tokens[$stackPtr]['content']{0} === '*'
-        ) {
-            $error  = 'Multi lines comments are not allowed; use "// Comment" DocBlock comments instead';
-            $phpcsFile->addError($error, $stackPtr, 'WrongStyle');
-            return FALSE;
-        } else if (substr($tokens[$stackPtr]['content'], 0, 2) !== '//') {
-            $error  = 'Use single line or DocBlock comments within code';
-            $phpcsFile->addError($error, $stackPtr, 'WrongStyle');
-            return FALSE;
-        }
-        return TRUE;
-    }//_checkCommentStyle()
-
-
-    /**
-     * Gather into an array all comment lines to which $stackPtr belongs.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  Pointer to the first comment line.
-     *
-     * @return type array Pointers to tokens making up the comment block.
-     */
-    private function _getCommentBlock(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
-        $commentLines = array($stackPtr);
-        $nextComment  = $stackPtr;
-        $lastLine     = $tokens[$stackPtr]['line'];
-
-        while (($nextComment = $phpcsFile->findNext($tokens[$stackPtr]['code'], ($nextComment + 1), null, false)) !== false) {
-            if (($tokens[$nextComment]['line'] - 1) !== $lastLine) {
-                // Not part of the block.
-                break;
-            }
-
-            $lastLine       = $tokens[$nextComment]['line'];
-            $commentLines[] = $nextComment;
-        }
-
-        return $commentLines;
-    }//_getCommentBlock()
-
-
-    /**
-     * Add errors to $phpcsFile, if $commentLines isn't enclosed with blank lines.
-     *
-     * @param File $phpcsFile    The current file being scanned.
-     * @param array                $commentLines Lines of the comment block being checked.
-     *
-     * @return bool TRUE if $commentLines is enclosed with at least a blank line
-     * before and after, FALSE otherwise.
-     */
-    private function _checkBlankLinesAroundLongComment(File $phpcsFile, array $commentLines)
-    {
-        $hasBlankLinesAround = TRUE;
-        $tokens = $phpcsFile->getTokens();
-
-        // check blank line before the long comment
-        $firstCommentPtr = reset($commentLines);
-        $firstPreviousSpacePtr = $firstCommentPtr - 1;
-        while (T_WHITESPACE === $tokens[$firstPreviousSpacePtr]['code'] && $firstPreviousSpacePtr > 0) {
-            $firstPreviousSpacePtr--;
-        }
-        if ($tokens[$firstPreviousSpacePtr]['line'] >= $tokens[$firstCommentPtr]['line'] - 1) {
-            $error  = "Please add a blank line before comments counting more than {$this->longCommentLimit} lines.";
-            $phpcsFile->addError($error, $firstCommentPtr, 'LongCommentWithoutSpacing');
-            $hasBlankLinesAround = FALSE;
-        }
-
-        // check blank line after the long comment
-        $lastCommentPtr = end($commentLines);
-        $lastNextSpacePtr = $lastCommentPtr + 1;
-        while (T_WHITESPACE === $tokens[$lastNextSpacePtr]['code'] && $lastNextSpacePtr < count($tokens)) {
-            $lastNextSpacePtr++;
-        }
-        if ($tokens[$lastNextSpacePtr]['line'] <= $tokens[$lastCommentPtr]['line'] + 1) {
-            $error  = "Please add a blank line after comments counting more than {$this->longCommentLimit} lines.";
-            $phpcsFile->addError($error, $lastCommentPtr, 'LongCommentWithoutSpacing');
-            $hasBlankLinesAround = FALSE;
-        }
-
-        return $hasBlankLinesAround;
-    }//end _checkBlanksAroundLongComment()
+	/**
+	 * @var int Limit defining long comments.
+	 * Long comments count $longCommentLimit or more lines.
+	 */
+	public $longCommentLimit = 5;
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_COMMENT
+		);
+	}//end register()
+
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
+
+		// keep testing only if it's about the first comment of the block
+		$previousCommentPtr = $phpcsFile->findPrevious($tokens[$stackPtr]['code'], $stackPtr - 1);
+		if ($tokens[$previousCommentPtr]['line'] !== $tokens[$stackPtr]['line'] - 1) {
+			if (TRUE !== $this->_checkCommentStyle($phpcsFile, $stackPtr)) {
+				return;
+			}
+
+			$commentLines = $this->_getCommentBlock($phpcsFile, $stackPtr);
+
+			if (count($commentLines) >= $this->longCommentLimit) {
+				$this->_checkBlankLinesAroundLongComment($phpcsFile, $commentLines);
+			}
+		}
+	}//end process()
+
+
+	/**
+	 * Add error to $phpcsFile, if comment pointed by $stackPtr doesn't start
+	 * with '//'.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        that has to be a comment.
+	 *
+	 * @return bool TRUE if the content of the token pointed by $stackPtr starts
+	 *              with //, FALSE if an error was added to $phpcsFile.
+	 */
+	private function _checkCommentStyle(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
+		if ($tokens[$stackPtr]['content']{0} === '#') {
+			$error  = 'Perl-style comments are not allowed; use "// Comment" or DocBlock comments instead';
+			$phpcsFile->addError($error, $stackPtr, 'WrongStyle');
+			return FALSE;
+		} else if (substr($tokens[$stackPtr]['content'], 0, 2) === '/*'
+			|| $tokens[$stackPtr]['content']{0} === '*'
+		) {
+			$error  = 'Multi lines comments are not allowed; use "// Comment" DocBlock comments instead';
+			$phpcsFile->addError($error, $stackPtr, 'WrongStyle');
+			return FALSE;
+		} else if (substr($tokens[$stackPtr]['content'], 0, 2) !== '//') {
+			$error  = 'Use single line or DocBlock comments within code';
+			$phpcsFile->addError($error, $stackPtr, 'WrongStyle');
+			return FALSE;
+		}
+		return TRUE;
+	}//_checkCommentStyle()
+
+
+	/**
+	 * Gather into an array all comment lines to which $stackPtr belongs.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  Pointer to the first comment line.
+	 *
+	 * @return type array Pointers to tokens making up the comment block.
+	 */
+	private function _getCommentBlock(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
+		$commentLines = array($stackPtr);
+		$nextComment  = $stackPtr;
+		$lastLine     = $tokens[$stackPtr]['line'];
+
+		while (($nextComment = $phpcsFile->findNext($tokens[$stackPtr]['code'], ($nextComment + 1), null, false)) !== false) {
+			if (($tokens[$nextComment]['line'] - 1) !== $lastLine) {
+				// Not part of the block.
+				break;
+			}
+
+			$lastLine       = $tokens[$nextComment]['line'];
+			$commentLines[] = $nextComment;
+		}
+
+		return $commentLines;
+	}//_getCommentBlock()
+
+
+	/**
+	 * Add errors to $phpcsFile, if $commentLines isn't enclosed with blank lines.
+	 *
+	 * @param File $phpcsFile    The current file being scanned.
+	 * @param array                $commentLines Lines of the comment block being checked.
+	 *
+	 * @return bool TRUE if $commentLines is enclosed with at least a blank line
+	 * before and after, FALSE otherwise.
+	 */
+	private function _checkBlankLinesAroundLongComment(File $phpcsFile, array $commentLines)
+	{
+		$hasBlankLinesAround = TRUE;
+		$tokens = $phpcsFile->getTokens();
+
+		// check blank line before the long comment
+		$firstCommentPtr = reset($commentLines);
+		$firstPreviousSpacePtr = $firstCommentPtr - 1;
+		while (T_WHITESPACE === $tokens[$firstPreviousSpacePtr]['code'] && $firstPreviousSpacePtr > 0) {
+			$firstPreviousSpacePtr--;
+		}
+		if ($tokens[$firstPreviousSpacePtr]['line'] >= $tokens[$firstCommentPtr]['line'] - 1) {
+			$error  = "Please add a blank line before comments counting more than {$this->longCommentLimit} lines.";
+			$phpcsFile->addError($error, $firstCommentPtr, 'LongCommentWithoutSpacing');
+			$hasBlankLinesAround = FALSE;
+		}
+
+		// check blank line after the long comment
+		$lastCommentPtr = end($commentLines);
+		$lastNextSpacePtr = $lastCommentPtr + 1;
+		while (T_WHITESPACE === $tokens[$lastNextSpacePtr]['code'] && $lastNextSpacePtr < count($tokens)) {
+			$lastNextSpacePtr++;
+		}
+		if ($tokens[$lastNextSpacePtr]['line'] <= $tokens[$lastCommentPtr]['line'] + 1) {
+			$error  = "Please add a blank line after comments counting more than {$this->longCommentLimit} lines.";
+			$phpcsFile->addError($error, $lastCommentPtr, 'LongCommentWithoutSpacing');
+			$hasBlankLinesAround = FALSE;
+		}
+
+		return $hasBlankLinesAround;
+	}//end _checkBlanksAroundLongComment()
 
 }//end class
 

build/CodeIgniter/Sniffs/Operators/LogicalOperatorAndSniff.php

Indentation +34 added lines, -34 removed lines

@@ -33,47 +33,47 @@
 class LogicalOperatorAndSniff implements Sniff
 {
 	/**
-     * Returns an array of tokens this test wants to listen for: symbolic and literal operators and.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_LOGICAL_AND,
-        );
+	 * Returns an array of tokens this test wants to listen for: symbolic and literal operators and.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_LOGICAL_AND,
+		);
 
-    }//end register()
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        $operator_token = $tokens[$stackPtr];
-        $operator_string = $operator_token['content'];
-        $operator_code = $operator_token['code'];
+		$operator_token = $tokens[$stackPtr];
+		$operator_string = $operator_token['content'];
+		$operator_code = $operator_token['code'];
 
-        if ($operator_string !== strtoupper($operator_string)) {
-            $error_message = 'Logical operator should be in upper case;'
-                . ' use "' . strtoupper($operator_string)
-                . '" instead of "' . $operator_string . '"';
-            $phpcsFile->addError($error_message, $stackPtr, 'LowercaseLogicalOperator');
-        }
+		if ($operator_string !== strtoupper($operator_string)) {
+			$error_message = 'Logical operator should be in upper case;'
+				. ' use "' . strtoupper($operator_string)
+				. '" instead of "' . $operator_string . '"';
+			$phpcsFile->addError($error_message, $stackPtr, 'LowercaseLogicalOperator');
+		}
 
-        $warning_message = 'The symbolic form "&&" is preferred over the literal form "AND"';
-        $phpcsFile->addWarning($warning_message, $stackPtr, 'UseOfLiteralAndOperator');
+		$warning_message = 'The symbolic form "&&" is preferred over the literal form "AND"';
+		$phpcsFile->addWarning($warning_message, $stackPtr, 'UseOfLiteralAndOperator');
 
-    }//end process()
+	}//end process()
 
 
 }//end class

build/CodeIgniter/Sniffs/Operators/StrictComparisonOperatorSniff.php

Indentation +36 added lines, -36 removed lines

@@ -33,47 +33,47 @@
 
 class StrictComparisonOperatorSniff implements Sniff
 {
-    private static $_replacements = array(
-        T_IS_EQUAL     => '===',
-        T_IS_NOT_EQUAL => '!==',
-    );
+	private static $_replacements = array(
+		T_IS_EQUAL     => '===',
+		T_IS_NOT_EQUAL => '!==',
+	);
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_IS_EQUAL,
-            T_IS_NOT_EQUAL,
-        );
-    }//end register()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_IS_EQUAL,
+			T_IS_NOT_EQUAL,
+		);
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        $operator_token = $tokens[$stackPtr];
-        $operator_string = $operator_token['content'];
-        $operator_code = $operator_token['code'];
+		$operator_token = $tokens[$stackPtr];
+		$operator_string = $operator_token['content'];
+		$operator_code = $operator_token['code'];
 
-        $error_message = '"==" and "!=" are prohibited; use "'
-            . self::$_replacements[$operator_code] . '" instead of "'
-            . $operator_string . '".';
-        $phpcsFile->addError($error_message, $stackPtr, 'NonStrictComparisonUsed');
-    }//end process()
+		$error_message = '"==" and "!=" are prohibited; use "'
+			. self::$_replacements[$operator_code] . '" instead of "'
+			. $operator_string . '".';
+		$phpcsFile->addError($error_message, $stackPtr, 'NonStrictComparisonUsed');
+	}//end process()
 
 
 }//end class

build/CodeIgniter/Sniffs/Operators/UppercaseLogicalOperatorOrSniff.php

Indentation +40 added lines, -40 removed lines

@@ -32,51 +32,51 @@
 
 class UppercaseLogicalOperatorOrSniff implements Sniff
 {
-    /**
-     * Returns an array of tokens this test wants to listen for: literal and symbolic operators or.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_BOOLEAN_OR,
-            T_LOGICAL_OR,
-        );
+	/**
+	 * Returns an array of tokens this test wants to listen for: literal and symbolic operators or.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_BOOLEAN_OR,
+			T_LOGICAL_OR,
+		);
 
-    }//end register()
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        $operator_token = $tokens[$stackPtr];
-        $operator_string = $operator_token['content'];
-        $operator_code = $operator_token['code'];
+		$operator_token = $tokens[$stackPtr];
+		$operator_string = $operator_token['content'];
+		$operator_code = $operator_token['code'];
 
-        if ($operator_code == T_BOOLEAN_OR) {
-            $error_message = 'Logical operator "' . $operator_string
-                . '" is prohibited; use "OR" instead';
-            $phpcsFile->addError($error_message, $stackPtr, 'UseOf||InsteadOfOR');
-        }
-        // it is literal, if it is not symbolic
-        else if ($operator_string !== strtoupper($operator_string)) {
-            $error_message = 'Logical operator should be in upper case;'
-                . ' use "' . strtoupper($operator_string)
-                . '" instead of "' . $operator_string . '"';
-            $phpcsFile->addError($error_message, $stackPtr, 'UseOfLowercaseOr');
-        }
-    }//end process()
+		if ($operator_code == T_BOOLEAN_OR) {
+			$error_message = 'Logical operator "' . $operator_string
+				. '" is prohibited; use "OR" instead';
+			$phpcsFile->addError($error_message, $stackPtr, 'UseOf||InsteadOfOR');
+		}
+		// it is literal, if it is not symbolic
+		else if ($operator_string !== strtoupper($operator_string)) {
+			$error_message = 'Logical operator should be in upper case;'
+				. ' use "' . strtoupper($operator_string)
+				. '" instead of "' . $operator_string . '"';
+			$phpcsFile->addError($error_message, $stackPtr, 'UseOfLowercaseOr');
+		}
+	}//end process()
 
 
 }//end class