@@ -32,8 +32,8 @@ discard block |
||
| 32 | 32 | function hello3() |
| 33 | 33 | { |
| 34 | 34 | switch ($hello) { |
| 35 | - case 'hello': |
|
| 36 | - break; |
|
| 35 | + case 'hello': |
|
| 36 | + break; |
|
| 37 | 37 | } |
| 38 | 38 | } |
| 39 | 39 | |
@@ -79,20 +79,20 @@ discard block |
||
| 79 | 79 | } else if (FALSE) { |
| 80 | 80 | foreach ($tokens as $token) { |
| 81 | 81 | switch ($token) { |
| 82 | - case '1': |
|
| 83 | - case '2': |
|
| 84 | - if (true) { |
|
| 85 | - if (false) { |
|
| 86 | - if (false) { |
|
| 87 | - if (false) { |
|
| 88 | - echo 'hello'; |
|
| 89 | - } |
|
| 90 | - } |
|
| 91 | - } |
|
| 92 | - } |
|
| 93 | - break; |
|
| 94 | - case '5': |
|
| 95 | - break; |
|
| 82 | + case '1': |
|
| 83 | + case '2': |
|
| 84 | + if (true) { |
|
| 85 | + if (false) { |
|
| 86 | + if (false) { |
|
| 87 | + if (false) { |
|
| 88 | + echo 'hello'; |
|
| 89 | + } |
|
| 90 | + } |
|
| 91 | + } |
|
| 92 | + } |
|
| 93 | + break; |
|
| 94 | + case '5': |
|
| 95 | + break; |
|
| 96 | 96 | } |
| 97 | 97 | do { |
| 98 | 98 | while (true) { |
@@ -161,20 +161,20 @@ discard block |
||
| 161 | 161 | $last = count($ts) - 1; |
| 162 | 162 | |
| 163 | 163 | switch ($token) { |
| 164 | - case '(': |
|
| 164 | + case '(': |
|
| 165 | 165 | |
| 166 | - if ($last >= 3 && |
|
| 167 | - $ts[0]['token'] != T_CLASS && |
|
| 168 | - $ts[$last - 2]['token'] == T_OBJECT_OPERATOR && |
|
| 169 | - $ts[$last - 3]['token'] == T_VARIABLE ) { |
|
| 166 | + if ($last >= 3 && |
|
| 167 | + $ts[0]['token'] != T_CLASS && |
|
| 168 | + $ts[$last - 2]['token'] == T_OBJECT_OPERATOR && |
|
| 169 | + $ts[$last - 3]['token'] == T_VARIABLE ) { |
|
| 170 | 170 | |
| 171 | 171 | |
| 172 | - if (true) { |
|
| 173 | - echo 'hello'; |
|
| 174 | - } |
|
| 175 | - } |
|
| 176 | - array_push($braces, $token); |
|
| 177 | - break; |
|
| 172 | + if (true) { |
|
| 173 | + echo 'hello'; |
|
| 174 | + } |
|
| 175 | + } |
|
| 176 | + array_push($braces, $token); |
|
| 177 | + break; |
|
| 178 | 178 | } |
| 179 | 179 | } |
| 180 | 180 | } |
@@ -256,19 +256,19 @@ discard block |
||
| 256 | 256 | switch ($foo) { |
| 257 | 257 | case 1: |
| 258 | 258 | switch ($bar) { |
| 259 | - default: |
|
| 260 | - echo $string{1}; |
|
| 259 | + default: |
|
| 260 | + echo $string{1}; |
|
| 261 | 261 | } |
| 262 | 262 | break; |
| 263 | 263 | } |
| 264 | 264 | |
| 265 | 265 | function temp($foo, $bar) { |
| 266 | 266 | switch ($foo) { |
| 267 | - case 1: |
|
| 268 | - switch ($bar) { |
|
| 269 | - default: |
|
| 270 | - return $foo; |
|
| 271 | - } |
|
| 267 | + case 1: |
|
| 268 | + switch ($bar) { |
|
| 269 | + default: |
|
| 270 | + return $foo; |
|
| 271 | + } |
|
| 272 | 272 | break; |
| 273 | 273 | } |
| 274 | 274 | } |
@@ -62,14 +62,14 @@ |
||
| 62 | 62 | public function getWarningList($testFile='') |
| 63 | 63 | { |
| 64 | 64 | switch ($testFile) { |
| 65 | - case 'SideEffectsUnitTest.3.inc': |
|
| 66 | - case 'SideEffectsUnitTest.4.inc': |
|
| 67 | - case 'SideEffectsUnitTest.5.inc': |
|
| 68 | - case 'SideEffectsUnitTest.10.inc': |
|
| 69 | - case 'SideEffectsUnitTest.12.inc': |
|
| 70 | - return [1 => 1]; |
|
| 71 | - default: |
|
| 72 | - return []; |
|
| 65 | + case 'SideEffectsUnitTest.3.inc': |
|
| 66 | + case 'SideEffectsUnitTest.4.inc': |
|
| 67 | + case 'SideEffectsUnitTest.5.inc': |
|
| 68 | + case 'SideEffectsUnitTest.10.inc': |
|
| 69 | + case 'SideEffectsUnitTest.12.inc': |
|
| 70 | + return [1 => 1]; |
|
| 71 | + default: |
|
| 72 | + return []; |
|
| 73 | 73 | }//end switch |
| 74 | 74 | |
| 75 | 75 | }//end getWarningList() |
@@ -100,33 +100,33 @@ |
||
| 100 | 100 | } |
| 101 | 101 | |
| 102 | 102 | switch ($tokens[$i]['code']) { |
| 103 | - case T_DOUBLE_COLON: |
|
| 104 | - $usedName = strtolower($tokens[($i - 1)]['content']); |
|
| 105 | - if ($usedName === $systemName) { |
|
| 106 | - // The included system was used, so it is fine. |
|
| 107 | - return; |
|
| 108 | - } |
|
| 109 | - break; |
|
| 110 | - case T_EXTENDS: |
|
| 111 | - $classNameToken = $phpcsFile->findNext(T_STRING, ($i + 1)); |
|
| 112 | - $className = strtolower($tokens[$classNameToken]['content']); |
|
| 113 | - if ($className === $systemName) { |
|
| 114 | - // The included system was used, so it is fine. |
|
| 115 | - return; |
|
| 116 | - } |
|
| 117 | - break; |
|
| 118 | - case T_IMPLEMENTS: |
|
| 119 | - $endImplements = $phpcsFile->findNext([T_EXTENDS, T_OPEN_CURLY_BRACKET], ($i + 1)); |
|
| 120 | - for ($x = ($i + 1); $x < $endImplements; $x++) { |
|
| 121 | - if ($tokens[$x]['code'] === T_STRING) { |
|
| 122 | - $className = strtolower($tokens[$x]['content']); |
|
| 123 | - if ($className === $systemName) { |
|
| 124 | - // The included system was used, so it is fine. |
|
| 125 | - return; |
|
| 126 | - } |
|
| 127 | - } |
|
| 128 | - } |
|
| 129 | - break; |
|
| 103 | + case T_DOUBLE_COLON: |
|
| 104 | + $usedName = strtolower($tokens[($i - 1)]['content']); |
|
| 105 | + if ($usedName === $systemName) { |
|
| 106 | + // The included system was used, so it is fine. |
|
| 107 | + return; |
|
| 108 | + } |
|
| 109 | + break; |
|
| 110 | + case T_EXTENDS: |
|
| 111 | + $classNameToken = $phpcsFile->findNext(T_STRING, ($i + 1)); |
|
| 112 | + $className = strtolower($tokens[$classNameToken]['content']); |
|
| 113 | + if ($className === $systemName) { |
|
| 114 | + // The included system was used, so it is fine. |
|
| 115 | + return; |
|
| 116 | + } |
|
| 117 | + break; |
|
| 118 | + case T_IMPLEMENTS: |
|
| 119 | + $endImplements = $phpcsFile->findNext([T_EXTENDS, T_OPEN_CURLY_BRACKET], ($i + 1)); |
|
| 120 | + for ($x = ($i + 1); $x < $endImplements; $x++) { |
|
| 121 | + if ($tokens[$x]['code'] === T_STRING) { |
|
| 122 | + $className = strtolower($tokens[$x]['content']); |
|
| 123 | + if ($className === $systemName) { |
|
| 124 | + // The included system was used, so it is fine. |
|
| 125 | + return; |
|
| 126 | + } |
|
| 127 | + } |
|
| 128 | + } |
|
| 129 | + break; |
|
| 130 | 130 | }//end switch |
| 131 | 131 | }//end for |
| 132 | 132 | |
@@ -52,17 +52,17 @@ |
||
| 52 | 52 | $typeName = $phpcsFile->findNext(T_CONSTANT_ENCAPSED_STRING, ($stackPtr + 2), null, false, true); |
| 53 | 53 | $typeName = trim($tokens[$typeName]['content'], " '"); |
| 54 | 54 | switch (strtolower($tokens[($stackPtr + 1)]['content'])) { |
| 55 | - case 'includesystem' : |
|
| 56 | - $included = strtolower($typeName); |
|
| 57 | - break; |
|
| 58 | - case 'includeasset' : |
|
| 59 | - $included = strtolower($typeName).'assettype'; |
|
| 60 | - break; |
|
| 61 | - case 'includewidget' : |
|
| 62 | - $included = strtolower($typeName).'widgettype'; |
|
| 63 | - break; |
|
| 64 | - default: |
|
| 65 | - return; |
|
| 55 | + case 'includesystem' : |
|
| 56 | + $included = strtolower($typeName); |
|
| 57 | + break; |
|
| 58 | + case 'includeasset' : |
|
| 59 | + $included = strtolower($typeName).'assettype'; |
|
| 60 | + break; |
|
| 61 | + case 'includewidget' : |
|
| 62 | + $included = strtolower($typeName).'widgettype'; |
|
| 63 | + break; |
|
| 64 | + default: |
|
| 65 | + return; |
|
| 66 | 66 | } |
| 67 | 67 | |
| 68 | 68 | if ($included === strtolower($ownClass)) { |
@@ -178,40 +178,40 @@ |
||
| 178 | 178 | foreach ($lines as $line) { |
| 179 | 179 | $parts = explode('>>', $line); |
| 180 | 180 | switch ($parts[0]) { |
| 181 | - case 'AUTHOR': |
|
| 182 | - if (isset($authorCache[$parts[1]]) === false) { |
|
| 183 | - $authorCache[$parts[1]] = $parts[2]; |
|
| 184 | - } else { |
|
| 185 | - $authorCache[$parts[1]] += $parts[2]; |
|
| 186 | - } |
|
| 187 | - break; |
|
| 188 | - case 'PRAISE': |
|
| 189 | - if (isset($praiseCache[$parts[1]]) === false) { |
|
| 190 | - $praiseCache[$parts[1]] = [ |
|
| 191 | - 'good' => $parts[2], |
|
| 192 | - 'bad' => $parts[3], |
|
| 193 | - ]; |
|
| 194 | - } else { |
|
| 195 | - $praiseCache[$parts[1]]['good'] += $parts[2]; |
|
| 196 | - $praiseCache[$parts[1]]['bad'] += $parts[3]; |
|
| 197 | - } |
|
| 198 | - break; |
|
| 199 | - case 'SOURCE': |
|
| 200 | - if (isset($praiseCache[$parts[1]]) === false) { |
|
| 201 | - $praiseCache[$parts[1]] = []; |
|
| 202 | - } |
|
| 203 | - |
|
| 204 | - if (isset($sourceCache[$parts[1]][$parts[2]]) === false) { |
|
| 205 | - $sourceCache[$parts[1]][$parts[2]] = [ |
|
| 206 | - 'count' => $parts[3], |
|
| 207 | - 'fixable' => (bool) $parts[4], |
|
| 208 | - ]; |
|
| 209 | - } else { |
|
| 210 | - $sourceCache[$parts[1]][$parts[2]]['count'] += $parts[3]; |
|
| 211 | - } |
|
| 212 | - break; |
|
| 213 | - default: |
|
| 214 | - break; |
|
| 181 | + case 'AUTHOR': |
|
| 182 | + if (isset($authorCache[$parts[1]]) === false) { |
|
| 183 | + $authorCache[$parts[1]] = $parts[2]; |
|
| 184 | + } else { |
|
| 185 | + $authorCache[$parts[1]] += $parts[2]; |
|
| 186 | + } |
|
| 187 | + break; |
|
| 188 | + case 'PRAISE': |
|
| 189 | + if (isset($praiseCache[$parts[1]]) === false) { |
|
| 190 | + $praiseCache[$parts[1]] = [ |
|
| 191 | + 'good' => $parts[2], |
|
| 192 | + 'bad' => $parts[3], |
|
| 193 | + ]; |
|
| 194 | + } else { |
|
| 195 | + $praiseCache[$parts[1]]['good'] += $parts[2]; |
|
| 196 | + $praiseCache[$parts[1]]['bad'] += $parts[3]; |
|
| 197 | + } |
|
| 198 | + break; |
|
| 199 | + case 'SOURCE': |
|
| 200 | + if (isset($praiseCache[$parts[1]]) === false) { |
|
| 201 | + $praiseCache[$parts[1]] = []; |
|
| 202 | + } |
|
| 203 | + |
|
| 204 | + if (isset($sourceCache[$parts[1]][$parts[2]]) === false) { |
|
| 205 | + $sourceCache[$parts[1]][$parts[2]] = [ |
|
| 206 | + 'count' => $parts[3], |
|
| 207 | + 'fixable' => (bool) $parts[4], |
|
| 208 | + ]; |
|
| 209 | + } else { |
|
| 210 | + $sourceCache[$parts[1]][$parts[2]]['count'] += $parts[3]; |
|
| 211 | + } |
|
| 212 | + break; |
|
| 213 | + default: |
|
| 214 | + break; |
|
| 215 | 215 | }//end switch |
| 216 | 216 | }//end foreach |
| 217 | 217 | |
@@ -269,14 +269,14 @@ |
||
| 269 | 269 | foreach ($diffLines as $line) { |
| 270 | 270 | if (isset($line[0]) === true) { |
| 271 | 271 | switch ($line[0]) { |
| 272 | - case '-': |
|
| 273 | - $diff[] = "\033[31m$line\033[0m"; |
|
| 274 | - break; |
|
| 275 | - case '+': |
|
| 276 | - $diff[] = "\033[32m$line\033[0m"; |
|
| 277 | - break; |
|
| 278 | - default: |
|
| 279 | - $diff[] = $line; |
|
| 272 | + case '-': |
|
| 273 | + $diff[] = "\033[31m$line\033[0m"; |
|
| 274 | + break; |
|
| 275 | + case '+': |
|
| 276 | + $diff[] = "\033[32m$line\033[0m"; |
|
| 277 | + break; |
|
| 278 | + default: |
|
| 279 | + $diff[] = $line; |
|
| 280 | 280 | } |
| 281 | 281 | } |
| 282 | 282 | } |
@@ -404,19 +404,19 @@ |
||
| 404 | 404 | } else { |
| 405 | 405 | $lowerVarType = strtolower($varType); |
| 406 | 406 | switch ($lowerVarType) { |
| 407 | - case 'bool': |
|
| 408 | - case 'boolean': |
|
| 409 | - return 'boolean'; |
|
| 410 | - case 'double': |
|
| 411 | - case 'real': |
|
| 412 | - case 'float': |
|
| 413 | - return 'float'; |
|
| 414 | - case 'int': |
|
| 415 | - case 'integer': |
|
| 416 | - return 'integer'; |
|
| 417 | - case 'array()': |
|
| 418 | - case 'array': |
|
| 419 | - return 'array'; |
|
| 407 | + case 'bool': |
|
| 408 | + case 'boolean': |
|
| 409 | + return 'boolean'; |
|
| 410 | + case 'double': |
|
| 411 | + case 'real': |
|
| 412 | + case 'float': |
|
| 413 | + return 'float'; |
|
| 414 | + case 'int': |
|
| 415 | + case 'integer': |
|
| 416 | + return 'integer'; |
|
| 417 | + case 'array()': |
|
| 418 | + case 'array': |
|
| 419 | + return 'array'; |
|
| 420 | 420 | }//end switch |
| 421 | 421 | |
| 422 | 422 | if (strpos($lowerVarType, 'array(') !== false) { |
@@ -677,19 +677,19 @@ |
||
| 677 | 677 | $input = trim($input); |
| 678 | 678 | |
| 679 | 679 | switch ($input) { |
| 680 | - case 's': |
|
| 681 | - break(2); |
|
| 682 | - case 'q': |
|
| 683 | - throw new DeepExitException('', 0); |
|
| 684 | - default: |
|
| 685 | - // Repopulate the sniffs because some of them save their state |
|
| 686 | - // and only clear it when the file changes, but we are rechecking |
|
| 687 | - // the same file. |
|
| 688 | - $file->ruleset->populateTokenListeners(); |
|
| 689 | - $file->reloadContent(); |
|
| 690 | - $file->process(); |
|
| 691 | - $this->reporter->cacheFileReport($file, $this->config); |
|
| 692 | - break; |
|
| 680 | + case 's': |
|
| 681 | + break(2); |
|
| 682 | + case 'q': |
|
| 683 | + throw new DeepExitException('', 0); |
|
| 684 | + default: |
|
| 685 | + // Repopulate the sniffs because some of them save their state |
|
| 686 | + // and only clear it when the file changes, but we are rechecking |
|
| 687 | + // the same file. |
|
| 688 | + $file->ruleset->populateTokenListeners(); |
|
| 689 | + $file->reloadContent(); |
|
| 690 | + $file->process(); |
|
| 691 | + $this->reporter->cacheFileReport($file, $this->config); |
|
| 692 | + break; |
|
| 693 | 693 | } |
| 694 | 694 | }//end while |
| 695 | 695 | }//end if |
@@ -1,2 +1,1 @@ |
||
| 1 | 1 | <?php |
| 2 | -/** Why me? |
|