@@ -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 | } |