bmitch / Codor

src/Codor/Sniffs/Syntax/NullCoalescingSniff.php

Spacing +5 added lines, -5 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\Syntax;
 
@@ -14,7 +14,7 @@ 
      */
     public function register(): array
     {
-        return [T_ISSET];
+        return [ T_ISSET ];
     }
 
     /**
@@ -55,15 +55,15 @@ 
         $semiColinFound = false;
 
         while (true) {
-            if ($tokens[$index]['type'] == 'T_SEMICOLON') {
+            if ($tokens[ $index ][ 'type' ] == 'T_SEMICOLON') {
                 break;
             }
 
-            if ($tokens[$index]['type'] == 'T_INLINE_THEN') {
+            if ($tokens[ $index ][ 'type' ] == 'T_INLINE_THEN') {
                 $questionMarkFound = true;
             }
 
-            if ($tokens[$index]['type'] == 'T_INLINE_ELSE') {
+            if ($tokens[ $index ][ 'type' ] == 'T_INLINE_ELSE') {
                 $semiColinFound = true;
             }
             $index++;

src/Codor/Sniffs/ControlStructures/NestedIfSniff.php

Spacing +11 added lines, -11 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\ControlStructures;
 
@@ -14,7 +14,7 @@ 
      */
     public function register(): array
     {
-        return [T_IF];
+        return [ T_IF ];
     }
 
     /**
@@ -41,13 +41,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);
         }
     }
 
@@ -59,7 +59,7 @@ 
      */
     protected function checkForNestedIf(array $token, int $stackPtr)
     {
-        if (! $this->isIfStatement($token)) {
+        if (!$this->isIfStatement($token)) {
             return;
         }
 
@@ -68,7 +68,7 @@ 
         }
 
         $this->phpcsFile->addError('Nested if statement found.', $stackPtr, __CLASS__);
-        $this->errorStack[] = $stackPtr;
+        $this->errorStack[ ] = $stackPtr;
     }
 
     /**
@@ -93,7 +93,7 @@ 
      */
     protected function isIfStatement(array $token): bool
     {
-        if ($token['type'] === 'T_IF') {
+        if ($token[ 'type' ] === 'T_IF') {
             return true;
         }
 

src/Codor/Sniffs/ControlStructures/NoElseSniff.php

Spacing +2 added lines, -2 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\ControlStructures;
 
@@ -17,7 +17,7 @@ 
      */
     public function register(): array
     {
-        return [T_ELSE, T_ELSEIF];
+        return [ T_ELSE, T_ELSEIF ];
     }
 
     /**

src/Codor/Sniffs/Syntax/LinesAfterMethodSniff.php

Spacing +8 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\Syntax;
 
@@ -14,7 +14,7 @@ 
      */
     public function register(): array
     {
-        return [T_FUNCTION];
+        return [ T_FUNCTION ];
     }
 
     /**
@@ -27,21 +27,21 @@ 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
-        if (! isset($tokens[$stackPtr]['scope_closer'])) {
+        if (!isset($tokens[ $stackPtr ][ 'scope_closer' ])) {
             return;
         }
 
-        $endOfMethodIndex = $tokens[$stackPtr]['scope_closer'];
+        $endOfMethodIndex = $tokens[ $stackPtr ][ 'scope_closer' ];
         $nextCodeLine = 0;
 
-        for ($index=$endOfMethodIndex + 1; $index <= count($tokens); $index++) {
-            if (isset($tokens[$index]) && $tokens[$index]['type'] !== 'T_WHITESPACE') {
-                $nextCodeLine = $tokens[$index]['line'];
+        for ($index = $endOfMethodIndex + 1; $index <= count($tokens); $index++) {
+            if (isset($tokens[ $index ]) && $tokens[ $index ][ 'type' ] !== 'T_WHITESPACE') {
+                $nextCodeLine = $tokens[ $index ][ 'line' ];
                 break;
             }
         }
 
-        $linesBetween = $nextCodeLine - $tokens[$endOfMethodIndex]['line'] - 1;
+        $linesBetween = $nextCodeLine - $tokens[ $endOfMethodIndex ][ 'line' ] - 1;
         if ($linesBetween > 1) {
             $phpcsFile->addError("No more than 1 line after a method/function is allowed.", $stackPtr, __CLASS__);
         }

src/Codor/Sniffs/Files/ReturnNullSniff.php

Spacing +6 added lines, -6 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\Files;
 
@@ -14,7 +14,7 @@ 
      */
     public function register(): array
     {
-        return [T_RETURN];
+        return [ T_RETURN ];
     }
 
     /**
@@ -30,14 +30,14 @@ 
         $returnTokenIndex = $stackPtr;
 
         $scope = array_slice($tokens, $returnTokenIndex, null, true);
-        $semicolons = array_filter($scope, function ($token) {
-            return $token['type'] === 'T_SEMICOLON';
+        $semicolons = array_filter($scope, function($token) {
+            return $token[ 'type' ] === 'T_SEMICOLON';
         });
 
         $returnValueIndex = key($semicolons) - 1;
         $comparisonIndex = key($semicolons) - 3;
 
-        if ($scope[$returnValueIndex]['type'] === 'T_NULL' && $this->isComparisonType($scope[$comparisonIndex]['type']) === false) {
+        if ($scope[ $returnValueIndex ][ 'type' ] === 'T_NULL' && $this->isComparisonType($scope[ $comparisonIndex ][ 'type' ]) === false) {
             $error = "Return null value found.";
             $phpcsFile->addError($error, $returnValueIndex, __CLASS__);
         }
@@ -45,6 +45,6 @@ 
 
     private function isComparisonType(string $type) : bool
     {
-        return in_array($type, ['T_IS_NOT_IDENTICAL', 'T_IS_IDENTICAL', 'T_IS_NOT_EQUAL', 'T_IS_EQUAL'], true);
+        return in_array($type, [ 'T_IS_NOT_IDENTICAL', 'T_IS_IDENTICAL', 'T_IS_NOT_EQUAL', 'T_IS_EQUAL' ], true);
     }
 }

src/Codor/Sniffs/Classes/NewInstanceSniff.php

Spacing +13 added lines, -13 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\Classes;
 
@@ -13,7 +13,7 @@ 
      */
     public function register(): array
     {
-        return [T_FUNCTION];
+        return [ T_FUNCTION ];
     }
 
     /**
@@ -27,25 +27,25 @@ 
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
-        $functionName = $tokens[$stackPtr + 2]['content'];
+        $functionName = $tokens[ $stackPtr + 2 ][ 'content' ];
 
         $scopes = $this->getBracketsIndex($tokens, $stackPtr);
-        if (true === empty($scopes['open']) || true === empty($scopes['close'])) {
+        if (true === empty($scopes[ 'open' ]) || true === empty($scopes[ 'close' ])) {
             return;
         }
 
-        for ($index=$scopes['open']; $index <= $scopes['close']; $index++) {
-            if ($tokens[$index]['type'] === 'T_NEW') {
-                $phpcsFile->addWarning($this->getWarningMessage($functionName), $tokens[$index]['column'], __CLASS__);
+        for ($index = $scopes[ 'open' ]; $index <= $scopes[ 'close' ]; $index++) {
+            if ($tokens[ $index ][ 'type' ] === 'T_NEW') {
+                $phpcsFile->addWarning($this->getWarningMessage($functionName), $tokens[ $index ][ 'column' ], __CLASS__);
             }
         }
     }
 
     private function getBracketsIndex(array $tokens, int $stackPtr) : array
     {
-        $token = $tokens[$stackPtr];
-        $open = $token['scope_opener'] ?? null;
-        $close = $token['scope_closer'] ?? null;
+        $token = $tokens[ $stackPtr ];
+        $open = $token[ 'scope_opener' ] ?? null;
+        $close = $token[ 'scope_closer' ] ?? null;
 
         if (true === empty($open)) {
             return $this->searchBrackets($tokens, $stackPtr);
@@ -60,11 +60,11 @@ 
     private function searchBrackets(array $tokens, int $stackPtr) : array
     {
         $open = $close = null;
-        for ($i=$stackPtr; $i < count($tokens); $i++) {
-            if ($tokens[$i]['type'] === 'T_OPEN_CURLY_BRACKET') {
+        for ($i = $stackPtr; $i < count($tokens); $i++) {
+            if ($tokens[ $i ][ 'type' ] === 'T_OPEN_CURLY_BRACKET') {
                 $open = $i;
             }
-            if ($tokens[$i]['type'] === 'T_CLOSE_CURLY_BRACKET') {
+            if ($tokens[ $i ][ 'type' ] === 'T_CLOSE_CURLY_BRACKET') {
                 $close = $i;
             }
         }

src/Codor/Sniffs/Classes/PropertyDeclarationSniff.php

Spacing +23 added lines, -23 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Codor\Sniffs\Classes;
 
@@ -12,19 +12,19 @@ 
      * The decalred member variables in the class.
      * @var array
      */
-    protected $memberVars = [];
+    protected $memberVars = [ ];
 
     /**
      * The referenced member variables in the class.
      * @var array
      */
-    protected $referencedMemberVars = [];
+    protected $referencedMemberVars = [ ];
 
     /**
      * Visibility Tokens.
      * @var array
      */
-    protected $visibilityTokens = ['T_PRIVATE', 'T_PUBLIC', 'T_PROTECTED'];
+    protected $visibilityTokens = [ 'T_PRIVATE', 'T_PUBLIC', 'T_PROTECTED' ];
 
     /**
      * Holds the value if the class is extended or not.
@@ -38,7 +38,7 @@ 
      */
     public function register(): array
     {
-        return [T_CLASS];
+        return [ T_CLASS ];
     }
 
     /**
@@ -52,8 +52,8 @@ 
      */
     public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
     {
-        $this->memberVars = [];
-        $this->referencedMemberVars = [];
+        $this->memberVars = [ ];
+        $this->referencedMemberVars = [ ];
         $tokens = $phpcsFile->getTokens();
         foreach ($tokens as $index => $token) {
             $this->handleToken($tokens, $index);
@@ -74,9 +74,9 @@ 
      */
     protected function handleToken($tokens, $index)
     {
-        $token = $tokens[$index];
+        $token = $tokens[ $index ];
 
-        if ($token['type'] === 'T_EXTENDS') {
+        if ($token[ 'type' ] === 'T_EXTENDS') {
             $this->isExtendedClass = true;
             return;
         }
@@ -87,7 +87,7 @@ 
         }
 
 
-        if ($token['type'] === 'T_OBJECT_OPERATOR') {
+        if ($token[ 'type' ] === 'T_OBJECT_OPERATOR') {
             $this->handleObjectOperatorToken($tokens, $index);
             return;
         }
@@ -100,7 +100,7 @@ 
      */
     protected function isVisibilityToken($token): bool
     {
-        return in_array($token['type'], $this->visibilityTokens);
+        return in_array($token[ 'type' ], $this->visibilityTokens);
     }
 
     /**
@@ -114,20 +114,20 @@ 
         $count = count($tokens);
 
         for ($i = $index + 1; $i < $count; $i++) {
-            $possibleVariable = $tokens[$i];
+            $possibleVariable = $tokens[ $i ];
 
-            if ($possibleVariable['type'] === 'T_CONST'
-                || $possibleVariable['type'] === 'T_FUNCTION'
-                || $possibleVariable['type'] === 'T_SEMICOLON'
+            if ($possibleVariable[ 'type' ] === 'T_CONST'
+                || $possibleVariable[ 'type' ] === 'T_FUNCTION'
+                || $possibleVariable[ 'type' ] === 'T_SEMICOLON'
             ) {
                 return;
             }
 
-            if ($possibleVariable['type'] !== 'T_VARIABLE') {
+            if ($possibleVariable[ 'type' ] !== 'T_VARIABLE') {
                 continue;
             }
 
-            $this->memberVars[] = str_replace('$', '', $possibleVariable['content']);
+            $this->memberVars[ ] = str_replace('$', '', $possibleVariable[ 'content' ]);
             return;
         }
     }
@@ -140,19 +140,19 @@ 
      */
     protected function handleObjectOperatorToken($tokens, $index)
     {
-        $previousToken = $tokens[$index-1];
-        $nextToken = $tokens[$index+1];
-        $tokenAfterNext = $tokens[$index+2];
+        $previousToken = $tokens[ $index - 1 ];
+        $nextToken = $tokens[ $index + 1 ];
+        $tokenAfterNext = $tokens[ $index + 2 ];
 
-        if ($previousToken['content'] !== '$this') {
+        if ($previousToken[ 'content' ] !== '$this') {
             return;
         }
 
-        if (($tokenAfterNext['type'] !== 'T_WHITESPACE') && ($tokenAfterNext['type'] !== 'T_EQUAL')) {
+        if (($tokenAfterNext[ 'type' ] !== 'T_WHITESPACE') && ($tokenAfterNext[ 'type' ] !== 'T_EQUAL')) {
             return;
         }
 
-        $this->referencedMemberVars[] = $nextToken['content'];
+        $this->referencedMemberVars[ ] = $nextToken[ 'content' ];
     }
 
     /**