@@ -12,7 +12,7 @@ discard block |
||
| 12 | 12 | * The loop tokens we're lookin for. |
| 13 | 13 | * @var array |
| 14 | 14 | */ |
| 15 | - protected $loops = ['T_FOR', 'T_FOREACH', 'T_WHILE']; |
|
| 15 | + protected $loops = [ 'T_FOR', 'T_FOREACH', 'T_WHILE' ]; |
|
| 16 | 16 | |
| 17 | 17 | /** |
| 18 | 18 | * Returns the token types that this sniff is interested in. |
@@ -20,7 +20,7 @@ discard block |
||
| 20 | 20 | */ |
| 21 | 21 | public function register(): array |
| 22 | 22 | { |
| 23 | - return [T_FUNCTION]; |
|
| 23 | + return [ T_FUNCTION ]; |
|
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | /** |
@@ -34,20 +34,20 @@ discard block |
||
| 34 | 34 | public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr) |
| 35 | 35 | { |
| 36 | 36 | $tokens = $phpcsFile->getTokens(); |
| 37 | - $token = $tokens[$stackPtr]; |
|
| 38 | - $functionNameToken = $tokens[$stackPtr + 2]; |
|
| 39 | - $functionName = $functionNameToken['content']; |
|
| 37 | + $token = $tokens[ $stackPtr ]; |
|
| 38 | + $functionNameToken = $tokens[ $stackPtr + 2 ]; |
|
| 39 | + $functionName = $functionNameToken[ 'content' ]; |
|
| 40 | 40 | |
| 41 | 41 | if ($functionName !== '__construct') { |
| 42 | 42 | return; |
| 43 | 43 | } |
| 44 | 44 | // If this is an interface we don't check it. |
| 45 | - if (! isset($token['scope_opener'])) { |
|
| 45 | + if (!isset($token[ 'scope_opener' ])) { |
|
| 46 | 46 | return; |
| 47 | 47 | } |
| 48 | 48 | |
| 49 | - for ($index=$token['scope_opener']; $index <= $token['scope_closer']; $index++) { |
|
| 50 | - if (in_array($tokens[$index]['type'], $this->loops)) { |
|
| 49 | + for ($index = $token[ 'scope_opener' ]; $index <= $token[ 'scope_closer' ]; $index++) { |
|
| 50 | + if (in_array($tokens[ $index ][ 'type' ], $this->loops)) { |
|
| 51 | 51 | $phpcsFile->addError("Class constructor cannot contain a loop.", $stackPtr); |
| 52 | 52 | continue; |
| 53 | 53 | } |