Symplify / PHP7_CodeSniffer

src/Standards/Squiz/Sniffs/CSS/ShorthandSizeSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
     /**
      * Returns the token types that this sniff is interested in.
      *
-     * @return int[]
+     * @return string[]
      */
     public function register()
     {
@@ -53,7 +53,7 @@ 
     /**
      * Processes the tokens that this sniff is interested in.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file where the token was found.
+     * @param File $phpcsFile The file where the token was found.
      * @param int                  $stackPtr  The position in the stack where
      *                                        the token was found.
      *

Indentation +8 added lines, -8 removed lines

@@ -44,11 +44,11 @@ 
      * @var array
      */
     public $excludeStyles = array(
-                             'background-position'      => 'background-position',
-                             'box-shadow'               => 'box-shadow',
-                             'transform-origin'         => 'transform-origin',
-                             '-webkit-transform-origin' => '-webkit-transform-origin',
-                             '-ms-transform-origin'     => '-ms-transform-origin',
+                                'background-position'      => 'background-position',
+                                'box-shadow'               => 'box-shadow',
+                                'transform-origin'         => 'transform-origin',
+                                '-webkit-transform-origin' => '-webkit-transform-origin',
+                                '-ms-transform-origin'     => '-ms-transform-origin',
                             );
 
 
@@ -165,9 +165,9 @@ 
 
         $error = 'Size definitions must use shorthand if available; expected "%s" but found "%s"';
         $data  = array(
-                  $expected,
-                  $content,
-                 );
+                    $expected,
+                    $content,
+                    );
 
         $fix = $phpcsFile->addFixableError($error, $stackPtr, 'NotUsed', $data);
         if ($fix === true) {

src/Standards/Squiz/Sniffs/Debug/JSLintSniff.php

Doc Comments +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
     /**
      * Returns the token types that this sniff is interested in.
      *
-     * @return int[]
+     * @return integer[]
      */
     public function register()
     {
@@ -39,11 +39,11 @@ 
     /**
      * Processes the tokens that this sniff is interested in.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file where the token was found.
+     * @param File $phpcsFile The file where the token was found.
      * @param int                  $stackPtr  The position in the stack where
      *                                        the token was found.
      *
-     * @return void
+     * @return null|integer
      * @throws PHP_CodeSniffer_Exception If jslint.js could not be run
      */
     public function process(File $phpcsFile, $stackPtr)

src/Standards/Squiz/Sniffs/Formatting/OperatorBracketSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
     /**
      * Processes this test, when one of its tokens is encountered.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token in the
      *                                        stack passed in $tokens.
      *
@@ -237,7 +237,7 @@ 
     /**
      * Add and fix the missing brackets error.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token in the
      *                                        stack passed in $tokens.
      *

Indentation +14 added lines, -14 removed lines

@@ -36,9 +36,9 @@ 
      * @var array
      */
     public $supportedTokenizers = array(
-                                   'PHP',
-                                   'JS',
-                                  );
+                                    'PHP',
+                                    'JS',
+                                    );
 
 
     /**
@@ -138,7 +138,7 @@ 
                                 T_OPEN_SQUARE_BRACKET,
                                 T_CLOSE_SQUARE_BRACKET,
                                 T_MODULUS,
-                               );
+                                );
 
                     for ($prev = ($stackPtr - 1); $prev > $bracket; $prev--) {
                         if (in_array($tokens[$prev]['code'], $allowed) === true) {
@@ -202,13 +202,13 @@ 
                 // A list of tokens that indicate that the token is not
                 // part of an arithmetic operation.
                 $invalidTokens = array(
-                                  T_COMMA,
-                                  T_COLON,
-                                  T_OPEN_PARENTHESIS,
-                                  T_OPEN_SQUARE_BRACKET,
-                                  T_OPEN_SHORT_ARRAY,
-                                  T_CASE,
-                                 );
+                                    T_COMMA,
+                                    T_COLON,
+                                    T_OPEN_PARENTHESIS,
+                                    T_OPEN_SQUARE_BRACKET,
+                                    T_OPEN_SHORT_ARRAY,
+                                    T_CASE,
+                                    );
 
                 if (in_array($tokens[$previousToken]['code'], $invalidTokens) === false) {
                     $this->addMissingBracketsError($phpcsFile, $stackPtr);
@@ -225,8 +225,8 @@ 
             // check is that we are not also enclosed in square brackets
             // like this: ($array[$index + 1]), which is invalid.
             $brackets = array(
-                         T_OPEN_SQUARE_BRACKET,
-                         T_CLOSE_SQUARE_BRACKET,
+                            T_OPEN_SQUARE_BRACKET,
+                            T_CLOSE_SQUARE_BRACKET,
                         );
 
             $squareBracket = $phpcsFile->findPrevious($brackets, ($stackPtr - 1), $lastBracket);
@@ -281,7 +281,7 @@ 
                     T_MODULUS         => true,
                     T_ISSET           => true,
                     T_ARRAY           => true,
-                   );
+                    );
 
         // Find the first token in the expression.
         for ($before = ($stackPtr - 1); $before > 0; $before--) {

src/Standards/Squiz/Sniffs/Functions/FunctionDeclarationArgumentSpacingSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     /**
      * Processes this test, when one of its tokens is encountered.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token in the
      *                                        stack passed in $tokens.
      *
@@ -94,7 +94,7 @@ 
     /**
      * Processes the contents of a single set of brackets.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile   The file being scanned.
+     * @param File $phpcsFile   The file being scanned.
      * @param int                  $openBracket The position of the open bracket
      *                                          in the stack passed in $tokens.
      *

Indentation +34 added lines, -34 removed lines

@@ -62,7 +62,7 @@ 
         return array(
                 T_FUNCTION,
                 T_CLOSURE,
-               );
+                );
 
     }//end register()
 
@@ -141,9 +141,9 @@ 
                 if ($spacesBefore !== $this->equalsSpacing) {
                     $error = 'Incorrect spacing between argument "%s" and equals sign; expected '.$this->equalsSpacing.' but found %s';
                     $data  = array(
-                              $tokens[$nextParam]['content'],
-                              $spacesBefore,
-                             );
+                                $tokens[$nextParam]['content'],
+                                $spacesBefore,
+                                );
 
                     $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpaceBeforeEquals', $data);
                     if ($fix === true) {
@@ -164,9 +164,9 @@ 
                 if ($spacesAfter !== $this->equalsSpacing) {
                     $error = 'Incorrect spacing between default value and equals sign for argument "%s"; expected '.$this->equalsSpacing.' but found %s';
                     $data  = array(
-                              $tokens[$nextParam]['content'],
-                              $spacesAfter,
-                             );
+                                $tokens[$nextParam]['content'],
+                                $spacesAfter,
+                                );
 
                     $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpaceAfterDefault', $data);
                     if ($fix === true) {
@@ -187,9 +187,9 @@ 
                 if ($tokens[($nextComma - 1)]['code'] === T_WHITESPACE) {
                     $error = 'Expected 0 spaces between argument "%s" and comma; %s found';
                     $data  = array(
-                              $tokens[$nextParam]['content'],
-                              strlen($tokens[($nextComma - 1)]['content']),
-                             );
+                                $tokens[$nextParam]['content'],
+                                strlen($tokens[($nextComma - 1)]['content']),
+                                );
 
                     $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpaceBeforeComma', $data);
                     if ($fix === true) {
@@ -235,9 +235,9 @@ 
                     if ($gap !== 1) {
                         $error = 'Expected 1 space between type hint and argument "%s"; %s found';
                         $data  = array(
-                                  $arg,
-                                  $gap,
-                                 );
+                                    $arg,
+                                    $gap,
+                                    );
                         $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterHint', $data);
                         if ($fix === true) {
                             if ($gap === 0) {
@@ -261,9 +261,9 @@ 
                             if ($gap !== 1) {
                                 $error = 'Expected 1 space between comma and type hint "%s"; %s found';
                                 $data  = array(
-                                          $hint,
-                                          $gap,
-                                         );
+                                            $hint,
+                                            $gap,
+                                            );
                                 $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingBeforeHint', $data);
                                 if ($fix === true) {
                                     $phpcsFile->fixer->replaceToken(($comma + 1), ' ');
@@ -285,9 +285,9 @@ 
                         if ($tokens[$whitespace]['line'] === $tokens[($whitespace - 1)]['line']) {
                             $error = 'Expected 1 space between comma and argument "%s"; %s found';
                             $data  = array(
-                                      $arg,
-                                      $gap,
-                                     );
+                                        $arg,
+                                        $gap,
+                                        );
 
                             $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpacingBeforeArg', $data);
                             if ($fix === true) {
@@ -319,9 +319,9 @@ 
                     if ($gap !== 1) {
                         $error = 'Expected 1 space between type hint and argument "%s"; %s found';
                         $data  = array(
-                                  $arg,
-                                  $gap,
-                                 );
+                                    $arg,
+                                    $gap,
+                                    );
                         $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterHint', $data);
                         if ($fix === true) {
                             if ($gap === 0) {
@@ -340,10 +340,10 @@ 
                     if ($multiLine === false && $spaceAfterOpen !== $this->requiredSpacesAfterOpen) {
                         $error = 'Expected %s spaces between opening bracket and type hint "%s"; %s found';
                         $data  = array(
-                                  $this->requiredSpacesAfterOpen,
-                                  $hint,
-                                  $spaceAfterOpen,
-                                 );
+                                    $this->requiredSpacesAfterOpen,
+                                    $hint,
+                                    $spaceAfterOpen,
+                                    );
                         $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterOpenHint', $data);
                         if ($fix === true) {
                             $padding = str_repeat(' ', $this->requiredSpacesAfterOpen);
@@ -357,10 +357,10 @@ 
                 } else if ($multiLine === false && $gap !== $this->requiredSpacesAfterOpen) {
                     $error = 'Expected %s spaces between opening bracket and argument "%s"; %s found';
                     $data  = array(
-                              $this->requiredSpacesAfterOpen,
-                              $arg,
-                              $gap,
-                             );
+                                $this->requiredSpacesAfterOpen,
+                                $arg,
+                                $gap,
+                                );
                     $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterOpen', $data);
                     if ($fix === true) {
                         $padding = str_repeat(' ', $this->requiredSpacesAfterOpen);
@@ -395,10 +395,10 @@ 
             $lastParam = array_pop($params);
             $error     = 'Expected %s spaces between argument "%s" and closing bracket; %s found';
             $data      = array(
-                          $this->requiredSpacesBeforeClose,
-                          $tokens[$lastParam]['content'],
-                          $gap,
-                         );
+                            $this->requiredSpacesBeforeClose,
+                            $tokens[$lastParam]['content'],
+                            $gap,
+                            );
             $fix       = $phpcsFile->addFixableError($error, $closeBracket, 'SpacingBeforeClose', $data);
             if ($fix === true) {
                 $padding = str_repeat(' ', $this->requiredSpacesBeforeClose);

Spacing +6 added lines, -6 removed lines

@@ -238,7 +238,7 @@ 
                                   $arg,
                                   $gap,
                                  );
-                        $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterHint', $data);
+                        $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterHint', $data);
                         if ($fix === true) {
                             if ($gap === 0) {
                                 $phpcsFile->fixer->addContent($whitespace, ' ');
@@ -264,7 +264,7 @@ 
                                           $hint,
                                           $gap,
                                          );
-                                $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingBeforeHint', $data);
+                                $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpacingBeforeHint', $data);
                                 if ($fix === true) {
                                     $phpcsFile->fixer->replaceToken(($comma + 1), ' ');
                                 }
@@ -322,7 +322,7 @@ 
                                   $arg,
                                   $gap,
                                  );
-                        $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterHint', $data);
+                        $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterHint', $data);
                         if ($fix === true) {
                             if ($gap === 0) {
                                 $phpcsFile->fixer->addContent($nextToken, ' ');
@@ -344,7 +344,7 @@ 
                                   $hint,
                                   $spaceAfterOpen,
                                  );
-                        $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterOpenHint', $data);
+                        $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterOpenHint', $data);
                         if ($fix === true) {
                             $padding = str_repeat(' ', $this->requiredSpacesAfterOpen);
                             if ($spaceAfterOpen === 0) {
@@ -361,7 +361,7 @@ 
                               $arg,
                               $gap,
                              );
-                    $fix   = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterOpen', $data);
+                    $fix = $phpcsFile->addFixableError($error, $nextToken, 'SpacingAfterOpen', $data);
                     if ($fix === true) {
                         $padding = str_repeat(' ', $this->requiredSpacesAfterOpen);
                         if ($gap === 0) {
@@ -399,7 +399,7 @@ 
                           $tokens[$lastParam]['content'],
                           $gap,
                          );
-            $fix       = $phpcsFile->addFixableError($error, $closeBracket, 'SpacingBeforeClose', $data);
+            $fix = $phpcsFile->addFixableError($error, $closeBracket, 'SpacingBeforeClose', $data);
             if ($fix === true) {
                 $padding = str_repeat(' ', $this->requiredSpacesBeforeClose);
                 if ($gap === 0) {

src/Standards/Squiz/Sniffs/Functions/MultiLineFunctionDeclarationSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * Should this test be skipped for some reason.
      *
-     * @return void
+     * @return boolean
      */
     protected function shouldSkipTest()
     {

Indentation +3 added lines, -3 removed lines

@@ -34,9 +34,9 @@
      * @var array
      */
     public $supportedTokenizers = array(
-                                   'PHP',
-                                   'JS',
-                                  );
+                                    'PHP',
+                                    'JS',
+                                    );
 
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
      *
      * @return void
      */
-    public function processBracket($phpcsFile, $openBracket, $tokens, $type='function')
+    public function processBracket($phpcsFile, $openBracket, $tokens, $type = 'function')
     {
         $errorPrefix = '';
         if ($type === 'use') {

src/Standards/Squiz/Sniffs/Objects/DisallowObjectStringIndexSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return string[]
      */
     public function register()
     {
@@ -38,7 +38,7 @@ 
     /**
      * Processes this test, when one of its tokens is encountered.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token
      *                                        in the stack passed in $tokens.
      *

src/Standards/Squiz/Sniffs/Objects/ObjectMemberCommaSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
     /**
      * Registers the token types that this sniff wishes to listen to.
      *
-     * @return array
+     * @return string[]
      */
     public function register()
     {
@@ -39,7 +39,7 @@ 
     /**
      * Process the tokens that this sniff is listening for.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file where the token was found.
+     * @param File $phpcsFile The file where the token was found.
      * @param int                  $stackPtr  The position in the stack where
      *                                        the token was found.
      *

src/Standards/Squiz/Sniffs/Operators/ComparisonOperatorUsageSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
     /**
      * Process the tokens that this sniff is listening for.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file where the token was found.
+     * @param File $phpcsFile The file where the token was found.
      * @param int                  $stackPtr  The position in the stack where the token
      *                                        was found.
      *

Indentation +22 added lines, -22 removed lines

@@ -55,9 +55,9 @@ 
      * @var array
      */
     public $supportedTokenizers = array(
-                                   'PHP',
-                                   'JS',
-                                  );
+                                    'PHP',
+                                    'JS',
+                                    );
 
     /**
      * A list of valid comparison operators.
@@ -65,13 +65,13 @@ 
      * @var array
      */
     private static $_validOps = array(
-                                 T_IS_IDENTICAL,
-                                 T_IS_NOT_IDENTICAL,
-                                 T_LESS_THAN,
-                                 T_GREATER_THAN,
-                                 T_IS_GREATER_OR_EQUAL,
-                                 T_IS_SMALLER_OR_EQUAL,
-                                 T_INSTANCEOF,
+                                    T_IS_IDENTICAL,
+                                    T_IS_NOT_IDENTICAL,
+                                    T_LESS_THAN,
+                                    T_GREATER_THAN,
+                                    T_IS_GREATER_OR_EQUAL,
+                                    T_IS_SMALLER_OR_EQUAL,
+                                    T_INSTANCEOF,
                                 );
 
     /**
@@ -80,16 +80,16 @@ 
      * @var array(int => string)
      */
     private static $_invalidOps = array(
-                                   'PHP' => array(
-                                             T_IS_EQUAL     => '===',
-                                             T_IS_NOT_EQUAL => '!==',
-                                             T_BOOLEAN_NOT  => '=== FALSE',
+                                    'PHP' => array(
+                                                T_IS_EQUAL     => '===',
+                                                T_IS_NOT_EQUAL => '!==',
+                                                T_BOOLEAN_NOT  => '=== FALSE',
                                             ),
-                                   'JS'  => array(
-                                             T_IS_EQUAL     => '===',
-                                             T_IS_NOT_EQUAL => '!==',
+                                    'JS'  => array(
+                                                T_IS_EQUAL     => '===',
+                                                T_IS_NOT_EQUAL => '!==',
                                             ),
-                                  );
+                                    );
 
 
     /**
@@ -105,7 +105,7 @@ 
                 T_INLINE_THEN,
                 T_WHILE,
                 T_FOR,
-               );
+                );
 
     }//end register()
 
@@ -191,9 +191,9 @@ 
             if (in_array($type, array_keys(self::$_invalidOps[$tokenizer])) === true) {
                 $error = 'Operator %s prohibited; use %s instead';
                 $data  = array(
-                          $tokens[$i]['content'],
-                          self::$_invalidOps[$tokenizer][$type],
-                         );
+                            $tokens[$i]['content'],
+                            self::$_invalidOps[$tokenizer][$type],
+                            );
                 $phpcsFile->addError($error, $i, 'NotAllowed', $data);
                 $foundOps++;
             } else if (in_array($type, self::$_validOps) === true) {

src/Standards/Squiz/Sniffs/Operators/IncrementDecrementUsageSniff.php

Doc Comments +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
     /**
      * Processes this test, when one of its tokens is encountered.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token
      *                                        in the stack passed in $tokens.
      *
@@ -60,7 +60,7 @@ 
     /**
      * Checks to ensure increment and decrement operators are not confusing.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token
      *                                        in the stack passed in $tokens.
      *
@@ -106,7 +106,7 @@ 
     /**
      * Checks to ensure increment and decrement operators are used.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
+     * @param File $phpcsFile The file being scanned.
      * @param int                  $stackPtr  The position of the current token
      *                                        in the stack passed in $tokens.
      *

Indentation +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
                 48 => 1,
                 70 => 1,
                 71 => 1,
-               );
+                );
 
     }//end getErrorList()