bmitch / Codor

src/Codor/Sniffs/ControlStructures/NoElseSniff.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 {
     public function register()
     {
-        return [T_ELSE, T_ELSEIF];
+        return [ T_ELSE, T_ELSEIF ];
     }
 
     /**

src/Codor/Sniffs/Files/FunctionLengthSniff.php

Spacing +7 added lines, -7 removed lines

@@ -12,25 +12,25 @@
 
     public function register()
     {
-        return [T_FUNCTION];
+        return [ T_FUNCTION ];
     }
 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
-        $token = $tokens[$stackPtr];
+        $token = $tokens[ $stackPtr ];
 
         // Skip function without body.
-        if (isset($token['scope_opener']) === false) {
+        if (isset($token[ 'scope_opener' ]) === false) {
             return 0;
         }
 
-        $firstToken = $tokens[$token['scope_opener']];
-        $lastToken = $tokens[$token['scope_closer']];
-        $length = $lastToken['line'] - $firstToken['line'];
+        $firstToken = $tokens[ $token[ 'scope_opener' ] ];
+        $lastToken = $tokens[ $token[ 'scope_closer' ] ];
+        $length = $lastToken[ 'line' ] - $firstToken[ 'line' ];
 
         if ($length > $this->maxLength) {
-            $tokenType = strtolower(substr($token['type'], 2));
+            $tokenType = strtolower(substr($token[ 'type' ], 2));
             $error = "Function is {$length} lines. Must be {$this->maxLength} lines or fewer.";
             $phpcsFile->addError($error, $stackPtr, sprintf('%sTooBig', ucfirst($tokenType)));
         }

src/Codor/Sniffs/Files/ReturnNullSniff.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
      */
     public function register()
     {
-        return [T_RETURN];
+        return [ T_RETURN ];
     }
 
     /**
@@ -32,8 +32,8 @@ 
 
         $returnValueToken = '';
         for ($index = $returnTokenIndex; $index < count($tokens); $index++) { 
-            if ($tokens[$index]['type'] === 'T_SEMICOLON') {
-                $returnValueToken = $tokens[$index - 1];
+            if ($tokens[ $index ][ 'type' ] === 'T_SEMICOLON') {
+                $returnValueToken = $tokens[ $index - 1 ];
                 break;
             }
         }
@@ -42,7 +42,7 @@ 
             return;
         }
 
-        if ($returnValueToken['type'] === 'T_NULL') {
+        if ($returnValueToken[ 'type' ] === 'T_NULL') {
             $error = "Return null value found.";
             $phpcsFile->addError($error, $stackPtr);
         }

src/Codor/Sniffs/Classes/ClassLengthSniff.php

Spacing +5 added lines, -5 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function register()
     {
-        return [T_CLASS];
+        return [ T_CLASS ];
     }
 
     /**
@@ -35,12 +35,12 @@ 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
-        $token = $tokens[$stackPtr];
+        $token = $tokens[ $stackPtr ];
         
-        $openParenthesis = $tokens[$token['scope_opener']];
-        $closedParenthesis = $tokens[$token['scope_closer']];
+        $openParenthesis = $tokens[ $token[ 'scope_opener' ] ];
+        $closedParenthesis = $tokens[ $token[ 'scope_closer' ] ];
 
-        $length = $closedParenthesis['line'] - $openParenthesis['line'];
+        $length = $closedParenthesis[ 'line' ] - $openParenthesis[ 'line' ];
 
         if ($length > $this->maxLength) {
             $phpcsFile->addError("Class is {$length} lines. Must be {$this->maxLength} lines or fewer.", $stackPtr);

src/Codor/Sniffs/Files/FunctionNameContainsAndOrSniff.php

Spacing +5 added lines, -5 removed lines

@@ -12,7 +12,7 @@ 
      * The forbidden strings this sniff looks for.
      * @var array
      */
-    protected $keywords = ['And', '_and', 'Or', '_or'];
+    protected $keywords = [ 'And', '_and', 'Or', '_or' ];
 
     /**
      * Returns the token types that this sniff is interested in.
@@ -20,7 +20,7 @@ 
      */
     public function register()
     {
-        return [T_FUNCTION];
+        return [ T_FUNCTION ];
     }
 
     /**
@@ -34,10 +34,10 @@ 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
-        $functionNameToken = $tokens[$stackPtr + 2];
-        $functionName = $functionNameToken['content'];
+        $functionNameToken = $tokens[ $stackPtr + 2 ];
+        $functionName = $functionNameToken[ 'content' ];
         
-        if (! $this->containsKeywords($functionName)) {
+        if (!$this->containsKeywords($functionName)) {
             return;
         }
 

src/Codor/Sniffs/Files/FunctionParameterSniff.php

Spacing +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
      */
     public function register()
     {
-        return [T_FUNCTION];
+        return [ T_FUNCTION ];
     }
 
     /**
@@ -34,13 +34,13 @@ 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens           = $phpcsFile->getTokens();
-        $token            = $tokens[$stackPtr];
-        $openParenIndex   = $token['parenthesis_opener'];
-        $closedParenIndex = $token['parenthesis_closer'];
+        $token            = $tokens[ $stackPtr ];
+        $openParenIndex   = $token[ 'parenthesis_opener' ];
+        $closedParenIndex = $token[ 'parenthesis_closer' ];
 
         $numberOfParameters = 0;
-        for ($index=$openParenIndex+1; $index <= $closedParenIndex; $index++) {
-            $tokens[$index]['type'] == 'T_VARIABLE' ? $numberOfParameters++ : null;
+        for ($index = $openParenIndex + 1; $index <= $closedParenIndex; $index++) {
+            $tokens[ $index ][ 'type' ] == 'T_VARIABLE' ? $numberOfParameters++ : null;
         }
 
         if ($numberOfParameters > $this->maxParameters) {

src/Codor/Sniffs/ControlStructures/NestedIfSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 
     /**
      * Returns the token types that this sniff is interested in.
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

Spacing +10 added lines, -10 removed lines

@@ -14,7 +14,7 @@ 
      */
     public function register()
     {
-        return [T_IF];
+        return [ T_IF ];
     }
 
     /**
@@ -29,13 +29,13 @@ 
     {
         $this->phpcsFile = $phpcsFile;
         $tokens          = $phpcsFile->getTokens();
-        $token           = $tokens[$stackPtr];
-        $start           = $token['scope_opener'];
-        $end             = $token['scope_closer'];
+        $token           = $tokens[ $stackPtr ];
+        $start           = $token[ 'scope_opener' ];
+        $end             = $token[ 'scope_closer' ];
 
-        $this->errorStack = [];
-        for ($index=$start; $index <= $end; $index++) {
-            $this->checkForNestedIf($tokens[$index], $stackPtr);
+        $this->errorStack = [ ];
+        for ($index = $start; $index <= $end; $index++) {
+            $this->checkForNestedIf($tokens[ $index ], $stackPtr);
         }
     }
 
@@ -47,7 +47,7 @@ 
      */
     protected function checkForNestedIf(array $token, $stackPtr)
     {
-        if (! $this->isIfStatement($token)) {
+        if (!$this->isIfStatement($token)) {
             return;
         }
 
@@ -56,7 +56,7 @@ 
         }
 
         $this->phpcsFile->addError('Nested if statement found.', $stackPtr);
-        $this->errorStack[] = $stackPtr;
+        $this->errorStack[ ] = $stackPtr;
     }
 
     /**
@@ -81,7 +81,7 @@ 
      */
     protected function isIfStatement(array $token)
     {
-        if ($token['type'] === 'T_IF') {
+        if ($token[ 'type' ] === 'T_IF') {
             return true;
         }
 

src/Codor/Sniffs/Files/IndentationLevelSniff.php

Spacing +12 added lines, -12 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function register()
     {
-        return [T_FUNCTION, T_CLOSURE, T_SWITCH];
+        return [ T_FUNCTION, T_CLOSURE, T_SWITCH ];
     }
 
     /**
@@ -39,11 +39,11 @@ 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
-        $token = $tokens[$stackPtr];
+        $token = $tokens[ $stackPtr ];
         $this->maxIndentationFound = 0;
 
         // Ignore functions with no body
-        if (isset($token['scope_opener']) === false) {
+        if (isset($token[ 'scope_opener' ]) === false) {
             return;
         }
 
@@ -64,15 +64,15 @@ 
      */
     protected function inspectScope(array $token, array $tokens)
     {
-        $start = $token['scope_opener'];
-        $length = $token['scope_closer'] - $start + 1;
-        $relativeScopeLevel = $tokens[$start]['level'];
+        $start = $token[ 'scope_opener' ];
+        $length = $token[ 'scope_closer' ] - $start + 1;
+        $relativeScopeLevel = $tokens[ $start ][ 'level' ];
 
         $scope = array_slice($tokens, $start, $length, true);
         $scope = $this->removeTokenScopes($scope, 'T_SWITCH');
 
-        $this->maxIndentationFound = array_reduce($scope, function ($max, $nestedToken) use ($relativeScopeLevel) {
-            $max = max($max, $nestedToken['level'] - $relativeScopeLevel);
+        $this->maxIndentationFound = array_reduce($scope, function($max, $nestedToken) use ($relativeScopeLevel) {
+            $max = max($max, $nestedToken[ 'level' ] - $relativeScopeLevel);
             return $max;
         });
     }
@@ -85,12 +85,12 @@ 
      */
     protected function removeTokenScopes(array $scope, $type)
     {
-        $typeTokens = array_filter($scope, function ($token) use ($type) {
-            return $token['type'] == $type;
+        $typeTokens = array_filter($scope, function($token) use ($type) {
+            return $token[ 'type' ] == $type;
         });
 
         foreach ($typeTokens as $token) {
-            $range = array_flip(range($token['scope_opener'], $token['scope_closer']));
+            $range = array_flip(range($token[ 'scope_opener' ], $token[ 'scope_closer' ]));
             $scope = array_diff_key($scope, $range);
         }
 
@@ -108,7 +108,7 @@ 
         // indentation limit.
         $indentationFound = $this->maxIndentationFound - 1;
         $indentationLimit = $this->indentationLimit - 1;
-        return "{$indentationFound} indentation levels found. " .
+        return "{$indentationFound} indentation levels found. ".
         "Maximum of {$indentationLimit} indentation levels allowed.";
     }
 }