@@ -49,13 +49,13 @@ discard block |
||
| 49 | 49 | } |
| 50 | 50 | |
| 51 | 51 | switch ($lowerVarType) { |
| 52 | - case 'bool': |
|
| 53 | - return 'boolean'; |
|
| 54 | - echo 'hi'; |
|
| 55 | - case 'double': |
|
| 56 | - case 'real': |
|
| 57 | - return 'float'; |
|
| 58 | - echo 'hi'; |
|
| 52 | + case 'bool': |
|
| 53 | + return 'boolean'; |
|
| 54 | + echo 'hi'; |
|
| 55 | + case 'double': |
|
| 56 | + case 'real': |
|
| 57 | + return 'float'; |
|
| 58 | + echo 'hi'; |
|
| 59 | 59 | } |
| 60 | 60 | |
| 61 | 61 | while ($line=fgets($fp,2*1024*1024)) |
@@ -66,14 +66,14 @@ discard block |
||
| 66 | 66 | } |
| 67 | 67 | |
| 68 | 68 | switch ($var) { |
| 69 | - case 1: |
|
| 70 | - echo '1'; |
|
| 71 | - break; |
|
| 69 | + case 1: |
|
| 70 | + echo '1'; |
|
| 71 | + break; |
|
| 72 | 72 | |
| 73 | - echo 'non-executable'; |
|
| 74 | - default: |
|
| 75 | - echo '2'; |
|
| 76 | - break; |
|
| 73 | + echo 'non-executable'; |
|
| 74 | + default: |
|
| 75 | + echo '2'; |
|
| 76 | + break; |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | switch (0) { |
@@ -124,13 +124,13 @@ discard block |
||
| 124 | 124 | }//end myFunction() |
| 125 | 125 | |
| 126 | 126 | switch ($var) { |
| 127 | - case 1: { |
|
| 128 | - return '1'; |
|
| 129 | - } |
|
| 127 | + case 1: { |
|
| 128 | + return '1'; |
|
| 129 | + } |
|
| 130 | 130 | |
| 131 | - case 2: { |
|
| 132 | - return '2'; |
|
| 133 | - } |
|
| 131 | + case 2: { |
|
| 132 | + return '2'; |
|
| 133 | + } |
|
| 134 | 134 | } |
| 135 | 135 | |
| 136 | 136 | defined('FOO') or die('error'); |
@@ -202,12 +202,12 @@ discard block |
||
| 202 | 202 | } |
| 203 | 203 | |
| 204 | 204 | switch ($foo) { |
| 205 | - case 'foo': |
|
| 206 | - if ($foo) |
|
| 207 | - return $foo; |
|
| 208 | - return $bar; |
|
| 209 | - default: |
|
| 210 | - return $bar; |
|
| 205 | + case 'foo': |
|
| 206 | + if ($foo) |
|
| 207 | + return $foo; |
|
| 208 | + return $bar; |
|
| 209 | + default: |
|
| 210 | + return $bar; |
|
| 211 | 211 | } |
| 212 | 212 | |
| 213 | 213 | function foo() |
@@ -216,13 +216,13 @@ discard block |
||
| 216 | 216 | } |
| 217 | 217 | |
| 218 | 218 | switch (true) { |
| 219 | - case 1: |
|
| 220 | - return foo( |
|
| 221 | - function () { |
|
| 222 | - $foo = $bar; // when this is removed it works ok |
|
| 223 | - return false; // from here on it reports unreachable |
|
| 224 | - } |
|
| 225 | - ); |
|
| 219 | + case 1: |
|
| 220 | + return foo( |
|
| 221 | + function () { |
|
| 222 | + $foo = $bar; // when this is removed it works ok |
|
| 223 | + return false; // from here on it reports unreachable |
|
| 224 | + } |
|
| 225 | + ); |
|
| 226 | 226 | } |
| 227 | 227 | |
| 228 | 228 | for($i=0,$j=50; $i<100; $i++) { |
@@ -43,32 +43,32 @@ |
||
| 43 | 43 | public function getWarningList($testFile='CommentedOutCodeUnitTest.inc') |
| 44 | 44 | { |
| 45 | 45 | switch ($testFile) { |
| 46 | - case 'CommentedOutCodeUnitTest.inc': |
|
| 47 | - return [ |
|
| 48 | - 6 => 1, |
|
| 49 | - 8 => 1, |
|
| 50 | - 15 => 1, |
|
| 51 | - 19 => 1, |
|
| 52 | - 35 => 1, |
|
| 53 | - 87 => 1, |
|
| 54 | - 91 => 1, |
|
| 55 | - 97 => 1, |
|
| 56 | - 109 => 1, |
|
| 57 | - 116 => 1, |
|
| 58 | - 128 => 1, |
|
| 59 | - 147 => 1, |
|
| 60 | - 158 => 1, |
|
| 61 | - ]; |
|
| 62 | - break; |
|
| 63 | - case 'CommentedOutCodeUnitTest.css': |
|
| 64 | - return [ |
|
| 65 | - 7 => 1, |
|
| 66 | - 16 => 1, |
|
| 67 | - ]; |
|
| 68 | - break; |
|
| 69 | - default: |
|
| 70 | - return []; |
|
| 71 | - break; |
|
| 46 | + case 'CommentedOutCodeUnitTest.inc': |
|
| 47 | + return [ |
|
| 48 | + 6 => 1, |
|
| 49 | + 8 => 1, |
|
| 50 | + 15 => 1, |
|
| 51 | + 19 => 1, |
|
| 52 | + 35 => 1, |
|
| 53 | + 87 => 1, |
|
| 54 | + 91 => 1, |
|
| 55 | + 97 => 1, |
|
| 56 | + 109 => 1, |
|
| 57 | + 116 => 1, |
|
| 58 | + 128 => 1, |
|
| 59 | + 147 => 1, |
|
| 60 | + 158 => 1, |
|
| 61 | + ]; |
|
| 62 | + break; |
|
| 63 | + case 'CommentedOutCodeUnitTest.css': |
|
| 64 | + return [ |
|
| 65 | + 7 => 1, |
|
| 66 | + 16 => 1, |
|
| 67 | + ]; |
|
| 68 | + break; |
|
| 69 | + default: |
|
| 70 | + return []; |
|
| 71 | + break; |
|
| 72 | 72 | }//end switch |
| 73 | 73 | |
| 74 | 74 | }//end getWarningList() |
@@ -28,56 +28,56 @@ |
||
| 28 | 28 | public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc') |
| 29 | 29 | { |
| 30 | 30 | switch ($testFile) { |
| 31 | - case 'ComparisonOperatorUsageUnitTest.inc': |
|
| 32 | - return [ |
|
| 33 | - 6 => 1, |
|
| 34 | - 7 => 1, |
|
| 35 | - 10 => 1, |
|
| 36 | - 11 => 1, |
|
| 37 | - 18 => 1, |
|
| 38 | - 19 => 1, |
|
| 39 | - 22 => 1, |
|
| 40 | - 23 => 1, |
|
| 41 | - 29 => 2, |
|
| 42 | - 32 => 2, |
|
| 43 | - 38 => 4, |
|
| 44 | - 47 => 2, |
|
| 45 | - 69 => 1, |
|
| 46 | - 72 => 1, |
|
| 47 | - 75 => 1, |
|
| 48 | - 78 => 1, |
|
| 49 | - 80 => 1, |
|
| 50 | - 82 => 1, |
|
| 51 | - 83 => 1, |
|
| 52 | - 89 => 1, |
|
| 53 | - 92 => 1, |
|
| 54 | - 100 => 1, |
|
| 55 | - 106 => 1, |
|
| 56 | - 112 => 1, |
|
| 57 | - 123 => 1, |
|
| 58 | - 127 => 1, |
|
| 59 | - 131 => 1, |
|
| 60 | - 135 => 1, |
|
| 61 | - ]; |
|
| 62 | - break; |
|
| 63 | - case 'ComparisonOperatorUsageUnitTest.js': |
|
| 64 | - return [ |
|
| 65 | - 5 => 1, |
|
| 66 | - 6 => 1, |
|
| 67 | - 17 => 1, |
|
| 68 | - 18 => 1, |
|
| 69 | - 28 => 2, |
|
| 70 | - 40 => 1, |
|
| 71 | - 47 => 1, |
|
| 72 | - 52 => 1, |
|
| 73 | - 63 => 1, |
|
| 74 | - 67 => 1, |
|
| 75 | - 71 => 1, |
|
| 76 | - ]; |
|
| 77 | - break; |
|
| 78 | - default: |
|
| 79 | - return []; |
|
| 80 | - break; |
|
| 31 | + case 'ComparisonOperatorUsageUnitTest.inc': |
|
| 32 | + return [ |
|
| 33 | + 6 => 1, |
|
| 34 | + 7 => 1, |
|
| 35 | + 10 => 1, |
|
| 36 | + 11 => 1, |
|
| 37 | + 18 => 1, |
|
| 38 | + 19 => 1, |
|
| 39 | + 22 => 1, |
|
| 40 | + 23 => 1, |
|
| 41 | + 29 => 2, |
|
| 42 | + 32 => 2, |
|
| 43 | + 38 => 4, |
|
| 44 | + 47 => 2, |
|
| 45 | + 69 => 1, |
|
| 46 | + 72 => 1, |
|
| 47 | + 75 => 1, |
|
| 48 | + 78 => 1, |
|
| 49 | + 80 => 1, |
|
| 50 | + 82 => 1, |
|
| 51 | + 83 => 1, |
|
| 52 | + 89 => 1, |
|
| 53 | + 92 => 1, |
|
| 54 | + 100 => 1, |
|
| 55 | + 106 => 1, |
|
| 56 | + 112 => 1, |
|
| 57 | + 123 => 1, |
|
| 58 | + 127 => 1, |
|
| 59 | + 131 => 1, |
|
| 60 | + 135 => 1, |
|
| 61 | + ]; |
|
| 62 | + break; |
|
| 63 | + case 'ComparisonOperatorUsageUnitTest.js': |
|
| 64 | + return [ |
|
| 65 | + 5 => 1, |
|
| 66 | + 6 => 1, |
|
| 67 | + 17 => 1, |
|
| 68 | + 18 => 1, |
|
| 69 | + 28 => 2, |
|
| 70 | + 40 => 1, |
|
| 71 | + 47 => 1, |
|
| 72 | + 52 => 1, |
|
| 73 | + 63 => 1, |
|
| 74 | + 67 => 1, |
|
| 75 | + 71 => 1, |
|
| 76 | + ]; |
|
| 77 | + break; |
|
| 78 | + default: |
|
| 79 | + return []; |
|
| 80 | + break; |
|
| 81 | 81 | }//end switch |
| 82 | 82 | |
| 83 | 83 | }//end getErrorList() |
@@ -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; |
|
| 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 | + } |
|
| 126 | 127 | } |
| 127 | 128 | } |
| 128 | - } |
|
| 129 | - break; |
|
| 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)) { |
@@ -71,18 +71,18 @@ |
||
| 71 | 71 | $prefix = $stackPtr; |
| 72 | 72 | while (($prefix = $phpcsFile->findPrevious(Tokens::$methodPrefixes, ($prefix - 1), $prev)) !== false) { |
| 73 | 73 | switch ($tokens[$prefix]['code']) { |
| 74 | - case T_STATIC: |
|
| 75 | - $static = $prefix; |
|
| 76 | - break; |
|
| 77 | - case T_ABSTRACT: |
|
| 78 | - $abstract = $prefix; |
|
| 79 | - break; |
|
| 80 | - case T_FINAL: |
|
| 81 | - $final = $prefix; |
|
| 82 | - break; |
|
| 83 | - default: |
|
| 84 | - $visibility = $prefix; |
|
| 85 | - break; |
|
| 74 | + case T_STATIC: |
|
| 75 | + $static = $prefix; |
|
| 76 | + break; |
|
| 77 | + case T_ABSTRACT: |
|
| 78 | + $abstract = $prefix; |
|
| 79 | + break; |
|
| 80 | + case T_FINAL: |
|
| 81 | + $final = $prefix; |
|
| 82 | + break; |
|
| 83 | + default: |
|
| 84 | + $visibility = $prefix; |
|
| 85 | + break; |
|
| 86 | 86 | } |
| 87 | 87 | } |
| 88 | 88 | |
@@ -67,14 +67,14 @@ |
||
| 67 | 67 | $fix = $phpcsFile->addFixableError($error, $stackPtr, 'MultipleDeclarations'); |
| 68 | 68 | if ($fix === true) { |
| 69 | 69 | switch ($tokens[($stackPtr + 2)]['content']) { |
| 70 | - case 'const': |
|
| 71 | - $baseUse = 'use const'; |
|
| 72 | - break; |
|
| 73 | - case 'function': |
|
| 74 | - $baseUse = 'use function'; |
|
| 75 | - break; |
|
| 76 | - default: |
|
| 77 | - $baseUse = 'use'; |
|
| 70 | + case 'const': |
|
| 71 | + $baseUse = 'use const'; |
|
| 72 | + break; |
|
| 73 | + case 'function': |
|
| 74 | + $baseUse = 'use function'; |
|
| 75 | + break; |
|
| 76 | + default: |
|
| 77 | + $baseUse = 'use'; |
|
| 78 | 78 | } |
| 79 | 79 | |
| 80 | 80 | if ($tokens[($next + 1)]['code'] !== T_WHITESPACE) { |
@@ -34,10 +34,10 @@ discard block |
||
| 34 | 34 | } |
| 35 | 35 | |
| 36 | 36 | switch ($foo) { |
| 37 | - case'Foo': { |
|
| 38 | - echo 'foo'; |
|
| 39 | - break; |
|
| 40 | - } |
|
| 37 | + case'Foo': { |
|
| 38 | + echo 'foo'; |
|
| 39 | + break; |
|
| 40 | + } |
|
| 41 | 41 | } |
| 42 | 42 | |
| 43 | 43 | while ($i < 10) { |
@@ -52,10 +52,10 @@ discard block |
||
| 52 | 52 | } |
| 53 | 53 | |
| 54 | 54 | switch (true) { |
| 55 | - case is_resource($value): |
|
| 56 | - throw new Exception('foo'); |
|
| 57 | - case is_object($value): |
|
| 58 | - return 'object'; |
|
| 55 | + case is_resource($value): |
|
| 56 | + throw new Exception('foo'); |
|
| 57 | + case is_object($value): |
|
| 58 | + return 'object'; |
|
| 59 | 59 | } |
| 60 | 60 | |
| 61 | 61 | switch (0) { |
@@ -105,13 +105,13 @@ discard block |
||
| 105 | 105 | } |
| 106 | 106 | |
| 107 | 107 | switch ($foo) { |
| 108 | - case 1: $bar = 1; break; |
|
| 109 | - case 2: |
|
| 108 | + case 1: $bar = 1; break; |
|
| 109 | + case 2: |
|
| 110 | 110 | |
| 111 | - $bar = 2; break; |
|
| 112 | - case 21: |
|
| 113 | - case 3: return 3; |
|
| 114 | - default: $bar = 0; |
|
| 111 | + $bar = 2; break; |
|
| 112 | + case 21: |
|
| 113 | + case 3: return 3; |
|
| 114 | + default: $bar = 0; |
|
| 115 | 115 | } |
| 116 | 116 | |
| 117 | 117 | switch ($foo) { |
@@ -146,13 +146,13 @@ discard block |
||
| 146 | 146 | } |
| 147 | 147 | |
| 148 | 148 | switch ($foo) { |
| 149 | - case Foo::INTERFACE: |
|
| 150 | - return self::INTERFACE; |
|
| 151 | - case Foo::NAMESPACE: |
|
| 152 | - return self::MODULE; |
|
| 153 | - case Foo::TRAIT: |
|
| 154 | - case Foo::ARRAY: |
|
| 155 | - return self::VALUE; |
|
| 149 | + case Foo::INTERFACE: |
|
| 150 | + return self::INTERFACE; |
|
| 151 | + case Foo::NAMESPACE: |
|
| 152 | + return self::MODULE; |
|
| 153 | + case Foo::TRAIT: |
|
| 154 | + case Foo::ARRAY: |
|
| 155 | + return self::VALUE; |
|
| 156 | 156 | } |
| 157 | 157 | |
| 158 | 158 | // OK: Every clause terminates |
@@ -28,19 +28,19 @@ |
||
| 28 | 28 | public function getErrorList($testFile='') |
| 29 | 29 | { |
| 30 | 30 | switch ($testFile) { |
| 31 | - case 'ClosingTagUnitTest.1.inc': |
|
| 32 | - return [11 => 1]; |
|
| 31 | + case 'ClosingTagUnitTest.1.inc': |
|
| 32 | + return [11 => 1]; |
|
| 33 | 33 | |
| 34 | - case 'ClosingTagUnitTest.4.inc': |
|
| 35 | - case 'ClosingTagUnitTest.5.inc': |
|
| 36 | - return [1 => 1]; |
|
| 34 | + case 'ClosingTagUnitTest.4.inc': |
|
| 35 | + case 'ClosingTagUnitTest.5.inc': |
|
| 36 | + return [1 => 1]; |
|
| 37 | 37 | |
| 38 | - case 'ClosingTagUnitTest.6.inc': |
|
| 39 | - case 'ClosingTagUnitTest.7.inc': |
|
| 40 | - return [5 => 1]; |
|
| 38 | + case 'ClosingTagUnitTest.6.inc': |
|
| 39 | + case 'ClosingTagUnitTest.7.inc': |
|
| 40 | + return [5 => 1]; |
|
| 41 | 41 | |
| 42 | - default: |
|
| 43 | - return []; |
|
| 42 | + default: |
|
| 43 | + return []; |
|
| 44 | 44 | } |
| 45 | 45 | |
| 46 | 46 | }//end getErrorList() |