Inspection of "Fix SQLite bug with indexes on older versions of l..." - cycle/annotated - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#5)

unknown

created 2020-02-01 18:48 UTC

Switch Indentation -8 removed lines patch added patch discarded remove patch

@@ -1,9 +1,1 @@
 block discarded – undo
 <?php
-/**
- * File comment.
- *
- * @package    Package
- * @subpackage Subpackage
- * @author     Squiz Pty Ltd <[email protected]>
- * @copyright  2010-2014 Squiz Pty Ltd (ABN 77 084 670 600)
- *

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/Commenting/BlockCommentUnitTest.inc 1 patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -144,26 +144,26 @@
 block discarded – undo
 }
 
 switch ($var) {
-    case 'foo':
-        /*
+        case 'foo':
+            /*
             Foo comment.
             This is a multiple
             line comment for Foo.
         */
 
-        echo 'Foo';
-    break;
+            echo 'Foo';
+        break;
 
-    default:
+        default:
 
-        /*
+            /*
             Foo comment.
             This is a multiple
             line comment for Foo.
         */
 
-        echo 'Default';
-    break;
+            echo 'Default';
+        break;
 }//end switch
 
 /**

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/Commenting/PostStatementCommentUnitTest.php 1 patch

Switch Indentation +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -28,23 +28,23 @@
 block discarded – undo
     public function getErrorList($testFile='PostStatementCommentUnitTest.inc')
     {
         switch ($testFile) {
-        case 'PostStatementCommentUnitTest.inc':
-            return [
-                6  => 1,
-                10 => 1,
-                18 => 1,
-                35 => 1,
-            ];
+            case 'PostStatementCommentUnitTest.inc':
+                return [
+                    6  => 1,
+                    10 => 1,
+                    18 => 1,
+                    35 => 1,
+                ];
 
-        case 'PostStatementCommentUnitTest.1.js':
-            return [
-                1 => 1,
-                4 => 1,
-                9 => 1,
-            ];
+            case 'PostStatementCommentUnitTest.1.js':
+                return [
+                    1 => 1,
+                    4 => 1,
+                    9 => 1,
+                ];
 
-        default:
-            return [];
+            default:
+                return [];
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/Commenting/FileCommentUnitTest.php 1 patch

Switch Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

@@ -28,29 +28,29 @@
 block discarded – undo
     public function getErrorList($testFile='FileCommentUnitTest.inc')
     {
         switch ($testFile) {
-        case 'FileCommentUnitTest.1.inc':
-        case 'FileCommentUnitTest.1.js':
-            return [
-                1  => 1,
-                22 => 2,
-                23 => 1,
-                24 => 2,
-                25 => 2,
-                26 => 1,
-                27 => 2,
-                28 => 2,
-                32 => 2,
-            ];
+            case 'FileCommentUnitTest.1.inc':
+            case 'FileCommentUnitTest.1.js':
+                return [
+                    1  => 1,
+                    22 => 2,
+                    23 => 1,
+                    24 => 2,
+                    25 => 2,
+                    26 => 1,
+                    27 => 2,
+                    28 => 2,
+                    32 => 2,
+                ];
 
-        case 'FileCommentUnitTest.4.inc':
-        case 'FileCommentUnitTest.6.inc':
-            return [1 => 1];
+            case 'FileCommentUnitTest.4.inc':
+            case 'FileCommentUnitTest.6.inc':
+                return [1 => 1];
 
-        case 'FileCommentUnitTest.5.inc':
-            return [2 => 1];
+            case 'FileCommentUnitTest.5.inc':
+                return [2 => 1];
 
-        default:
-            return [];
+            default:
+                return [];
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

Standards/Squiz/Tests/Commenting/LongConditionClosingCommentUnitTest.inc 1 patch

Switch Indentation +158 added lines, -158 removed lines patch added patch discarded remove patch

@@ -493,41 +493,41 @@  discard block
 block discarded – undo
 }
 
 switch ($something) {
-    case '1':
-    // Line 1
-    // Line 2
-    // Line 3
-    // Line 4
-    // Line 5
-    break;
-    case '2':
-    // Line 1
-    // Line 2
-    // Line 3
-    // Line 4
-    // Line 5
-    break;
-    case '3':
-    // Line 1
-    // Line 2
-    // Line 3
-    // Line 4
-    // Line 5
-    break;
-    case '4':
-    // Line 1
-    // Line 2
-    // Line 3
-    // Line 4
-    // Line 5
-    break;
-    case '5':
-    // Line 1
-    // Line 2
-    // Line 3
-    // Line 4
-    // Line 5
-    break;
+        case '1':
+        // Line 1
+        // Line 2
+        // Line 3
+        // Line 4
+        // Line 5
+        break;
+        case '2':
+        // Line 1
+        // Line 2
+        // Line 3
+        // Line 4
+        // Line 5
+        break;
+        case '3':
+        // Line 1
+        // Line 2
+        // Line 3
+        // Line 4
+        // Line 5
+        break;
+        case '4':
+        // Line 1
+        // Line 2
+        // Line 3
+        // Line 4
+        // Line 5
+        break;
+        case '5':
+        // Line 1
+        // Line 2
+        // Line 3
+        // Line 4
+        // Line 5
+        break;
 }
 
 // Wrong comment
@@ -629,69 +629,69 @@  discard block
 block discarded – undo
 }
 
 switch ($foo) {
-    case 'one' : {
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
+        case 'one' : {
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            break;
+        }
+        case 'one' :
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
+            // some code here.
         break;
-    }
-    case 'one' :
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-        // some code here.
-    break;
 }//end switch
 
 // Yes, code like this does exist.
@@ -701,67 +701,67 @@  discard block
 block discarded – undo
     return $bar;
 
 switch ($foo) {
-    case 1:
-    case 2:
-    case 3:
-    case 4:
-    case 5:
-    case 6:
-    case 7:
-    case 8:
-    case 9:
-    case 10:
-    case 11:
-    case 12:
-    case 13:
-    case 14:
-    case 15:
-    case 16:
-    case 17:
-    case 18:
-    case 19:
-    case 20:
-    case 21:
-    case 22:
-    case 23:
-    case 24:
-    case 25:
-    case 26:
-    case 27:
-    case 28:
-    case 29:
-    case 30:
-    case 31:
-    case 32:
-    case 33:
-    case 34:
-    case 35:
-    case 36:
-    case 37:
-    case 38:
-    case 39:
-    case 40:
-    case 41:
-    case 42:
-    case 43:
-    case 44:
-    case 45:
-    case 46:
-    case 47:
-    case 48:
-    case 49:
-    case 50:
-    case 51:
-    case 52:
-    case 53:
-    case 54:
-    case 55:
-    case 56:
-    case 57:
-    case 58:
-    case 59:
-        echo $foo;
-        break;
+        case 1:
+        case 2:
+        case 3:
+        case 4:
+        case 5:
+        case 6:
+        case 7:
+        case 8:
+        case 9:
+        case 10:
+        case 11:
+        case 12:
+        case 13:
+        case 14:
+        case 15:
+        case 16:
+        case 17:
+        case 18:
+        case 19:
+        case 20:
+        case 21:
+        case 22:
+        case 23:
+        case 24:
+        case 25:
+        case 26:
+        case 27:
+        case 28:
+        case 29:
+        case 30:
+        case 31:
+        case 32:
+        case 33:
+        case 34:
+        case 35:
+        case 36:
+        case 37:
+        case 38:
+        case 39:
+        case 40:
+        case 41:
+        case 42:
+        case 43:
+        case 44:
+        case 45:
+        case 46:
+        case 47:
+        case 48:
+        case 49:
+        case 50:
+        case 51:
+        case 52:
+        case 53:
+        case 54:
+        case 55:
+        case 56:
+        case 57:
+        case 58:
+        case 59:
+            echo $foo;
+            break;
 }
 
 // phpcs:set Squiz.Commenting.LongConditionClosingComment lineLimit 5

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/Commenting/InlineCommentUnitTest.php 1 patch

Switch Indentation +39 added lines, -39 removed lines patch added patch discarded remove patch

		@@ -28,46 +28,46 @@
		block discarded – undo
28	28	public function getErrorList($testFile='InlineCommentUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'InlineCommentUnitTest.inc':
32		- $errors = [
33		- 17 => 1,
34		- 27 => 1,
35		- 28 => 1,
36		- 32 => 2,
37		- 36 => 1,
38		- 44 => 2,
39		- 58 => 1,
40		- 61 => 1,
41		- 64 => 1,
42		- 67 => 1,
43		- 95 => 1,
44		- 96 => 1,
45		- 97 => 3,
46		- 118 => 1,
47		- 126 => 2,
48		- 130 => 2,
49		- 149 => 1,
50		- ];
	31	+ case 'InlineCommentUnitTest.inc':
	32	+ $errors = [
	33	+ 17 => 1,
	34	+ 27 => 1,
	35	+ 28 => 1,
	36	+ 32 => 2,
	37	+ 36 => 1,
	38	+ 44 => 2,
	39	+ 58 => 1,
	40	+ 61 => 1,
	41	+ 64 => 1,
	42	+ 67 => 1,
	43	+ 95 => 1,
	44	+ 96 => 1,
	45	+ 97 => 3,
	46	+ 118 => 1,
	47	+ 126 => 2,
	48	+ 130 => 2,
	49	+ 149 => 1,
	50	+ ];
51	51
52		- return $errors;
53		- case 'InlineCommentUnitTest.js':
54		- return [
55		- 31 => 1,
56		- 36 => 2,
57		- 48 => 1,
58		- 51 => 1,
59		- 54 => 1,
60		- 57 => 1,
61		- 102 => 1,
62		- 103 => 1,
63		- 104 => 3,
64		- 118 => 1,
65		- 121 => 1,
66		- 125 => 2,
67		- 129 => 2,
68		- ];
69		- default:
70		- return [];
	52	+ return $errors;
	53	+ case 'InlineCommentUnitTest.js':
	54	+ return [
	55	+ 31 => 1,
	56	+ 36 => 2,
	57	+ 48 => 1,
	58	+ 51 => 1,
	59	+ 54 => 1,
	60	+ 57 => 1,
	61	+ 102 => 1,
	62	+ 103 => 1,
	63	+ 104 => 3,
	64	+ 118 => 1,
	65	+ 121 => 1,
	66	+ 125 => 2,
	67	+ 129 => 2,
	68	+ ];
	69	+ default:
	70	+ return [];
71	71	}//end switch
72	72
73	73	}//end getErrorList()

Please login to merge, or discard this patch.

Standards/Squiz/Tests/Commenting/LongConditionClosingCommentUnitTest.php 1 patch

Switch Indentation +49 added lines, -49 removed lines patch added patch discarded remove patch

		@@ -28,55 +28,55 @@
		block discarded – undo
28	28	public function getErrorList($testFile='LongConditionClosingCommentUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'LongConditionClosingCommentUnitTest.inc':
32		- return [
33		- 49 => 1,
34		- 99 => 1,
35		- 146 => 1,
36		- 192 => 1,
37		- 215 => 1,
38		- 238 => 1,
39		- 261 => 1,
40		- 286 => 1,
41		- 309 => 1,
42		- 332 => 1,
43		- 355 => 1,
44		- 378 => 1,
45		- 493 => 1,
46		- 531 => 1,
47		- 536 => 1,
48		- 540 => 1,
49		- 562 => 1,
50		- 601 => 1,
51		- 629 => 1,
52		- 663 => 1,
53		- 765 => 1,
54		- 798 => 1,
55		- 811 => 1,
56		- 897 => 1,
57		- 931 => 1,
58		- 962 => 1,
59		- ];
60		- break;
61		- case 'LongConditionClosingCommentUnitTest.js':
62		- return [
63		- 47 => 1,
64		- 97 => 1,
65		- 144 => 1,
66		- 190 => 1,
67		- 213 => 1,
68		- 238 => 1,
69		- 261 => 1,
70		- 284 => 1,
71		- 307 => 1,
72		- 401 => 1,
73		- 439 => 1,
74		- 444 => 1,
75		- ];
76		- break;
77		- default:
78		- return [];
79		- break;
	31	+ case 'LongConditionClosingCommentUnitTest.inc':
	32	+ return [
	33	+ 49 => 1,
	34	+ 99 => 1,
	35	+ 146 => 1,
	36	+ 192 => 1,
	37	+ 215 => 1,
	38	+ 238 => 1,
	39	+ 261 => 1,
	40	+ 286 => 1,
	41	+ 309 => 1,
	42	+ 332 => 1,
	43	+ 355 => 1,
	44	+ 378 => 1,
	45	+ 493 => 1,
	46	+ 531 => 1,
	47	+ 536 => 1,
	48	+ 540 => 1,
	49	+ 562 => 1,
	50	+ 601 => 1,
	51	+ 629 => 1,
	52	+ 663 => 1,
	53	+ 765 => 1,
	54	+ 798 => 1,
	55	+ 811 => 1,
	56	+ 897 => 1,
	57	+ 931 => 1,
	58	+ 962 => 1,
	59	+ ];
	60	+ break;
	61	+ case 'LongConditionClosingCommentUnitTest.js':
	62	+ return [
	63	+ 47 => 1,
	64	+ 97 => 1,
	65	+ 144 => 1,
	66	+ 190 => 1,
	67	+ 213 => 1,
	68	+ 238 => 1,
	69	+ 261 => 1,
	70	+ 284 => 1,
	71	+ 307 => 1,
	72	+ 401 => 1,
	73	+ 439 => 1,
	74	+ 444 => 1,
	75	+ ];
	76	+ break;
	77	+ default:
	78	+ return [];
	79	+ break;
80	80	}//end switch
81	81
82	82	}//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/Arrays/ArrayDeclarationUnitTest.php 1 patch

Switch Indentation +163 added lines, -163 removed lines patch added patch discarded remove patch

		@@ -28,169 +28,169 @@
		block discarded – undo
28	28	public function getErrorList($testFile='')
29	29	{
30	30	switch ($testFile) {
31		- case 'ArrayDeclarationUnitTest.1.inc':
32		- return [
33		- 2 => 1,
34		- 8 => 2,
35		- 10 => 2,
36		- 22 => 1,
37		- 23 => 2,
38		- 24 => 2,
39		- 25 => 1,
40		- 31 => 2,
41		- 35 => 1,
42		- 36 => 2,
43		- 41 => 1,
44		- 46 => 1,
45		- 47 => 1,
46		- 50 => 1,
47		- 51 => 1,
48		- 53 => 1,
49		- 56 => 1,
50		- 58 => 1,
51		- 61 => 1,
52		- 62 => 1,
53		- 63 => 1,
54		- 64 => 1,
55		- 65 => 1,
56		- 66 => 3,
57		- 70 => 1,
58		- 76 => 2,
59		- 77 => 1,
60		- 78 => 7,
61		- 79 => 2,
62		- 81 => 2,
63		- 82 => 4,
64		- 87 => 1,
65		- 88 => 1,
66		- 92 => 1,
67		- 97 => 1,
68		- 100 => 1,
69		- 101 => 1,
70		- 102 => 1,
71		- 105 => 1,
72		- 106 => 1,
73		- 107 => 1,
74		- 125 => 1,
75		- 126 => 1,
76		- 141 => 1,
77		- 144 => 1,
78		- 146 => 1,
79		- 148 => 1,
80		- 151 => 1,
81		- 157 => 1,
82		- 174 => 3,
83		- 179 => 1,
84		- 182 => 1,
85		- 188 => 1,
86		- 207 => 1,
87		- 212 => 2,
88		- 214 => 1,
89		- 218 => 2,
90		- 219 => 2,
91		- 223 => 1,
92		- 255 => 1,
93		- 294 => 1,
94		- 295 => 1,
95		- 296 => 1,
96		- 311 => 1,
97		- 317 => 1,
98		- 339 => 2,
99		- 348 => 2,
100		- 352 => 2,
101		- 355 => 3,
102		- 358 => 3,
103		- 359 => 2,
104		- 360 => 1,
105		- 362 => 1,
106		- 363 => 2,
107		- 364 => 1,
108		- 365 => 2,
109		- 366 => 2,
110		- 367 => 2,
111		- 368 => 2,
112		- 369 => 1,
113		- 370 => 1,
114		- 383 => 1,
115		- 394 => 1,
116		- ];
117		- case 'ArrayDeclarationUnitTest.2.inc':
118		- return [
119		- 2 => 1,
120		- 10 => 1,
121		- 23 => 2,
122		- 24 => 2,
123		- 25 => 1,
124		- 31 => 2,
125		- 36 => 2,
126		- 41 => 1,
127		- 46 => 1,
128		- 47 => 1,
129		- 51 => 1,
130		- 53 => 1,
131		- 56 => 1,
132		- 61 => 1,
133		- 63 => 1,
134		- 64 => 1,
135		- 65 => 1,
136		- 66 => 2,
137		- 70 => 1,
138		- 76 => 1,
139		- 77 => 1,
140		- 78 => 7,
141		- 79 => 2,
142		- 81 => 2,
143		- 82 => 4,
144		- 87 => 1,
145		- 88 => 1,
146		- 92 => 1,
147		- 97 => 1,
148		- 100 => 1,
149		- 101 => 1,
150		- 102 => 1,
151		- 105 => 1,
152		- 106 => 1,
153		- 107 => 1,
154		- 125 => 1,
155		- 126 => 1,
156		- 141 => 1,
157		- 144 => 1,
158		- 146 => 1,
159		- 148 => 1,
160		- 151 => 1,
161		- 157 => 1,
162		- 174 => 3,
163		- 179 => 1,
164		- 190 => 1,
165		- 191 => 1,
166		- 192 => 1,
167		- 207 => 1,
168		- 210 => 1,
169		- 211 => 1,
170		- 215 => 1,
171		- 247 => 1,
172		- 286 => 1,
173		- 287 => 1,
174		- 288 => 1,
175		- 303 => 1,
176		- 309 => 1,
177		- 331 => 2,
178		- 345 => 3,
179		- 348 => 3,
180		- 349 => 2,
181		- 350 => 1,
182		- 352 => 2,
183		- 353 => 2,
184		- 354 => 2,
185		- 355 => 2,
186		- 356 => 2,
187		- 357 => 1,
188		- 358 => 1,
189		- 372 => 1,
190		- 383 => 1,
191		- ];
192		- default:
193		- return [];
	31	+ case 'ArrayDeclarationUnitTest.1.inc':
	32	+ return [
	33	+ 2 => 1,
	34	+ 8 => 2,
	35	+ 10 => 2,
	36	+ 22 => 1,
	37	+ 23 => 2,
	38	+ 24 => 2,
	39	+ 25 => 1,
	40	+ 31 => 2,
	41	+ 35 => 1,
	42	+ 36 => 2,
	43	+ 41 => 1,
	44	+ 46 => 1,
	45	+ 47 => 1,
	46	+ 50 => 1,
	47	+ 51 => 1,
	48	+ 53 => 1,
	49	+ 56 => 1,
	50	+ 58 => 1,
	51	+ 61 => 1,
	52	+ 62 => 1,
	53	+ 63 => 1,
	54	+ 64 => 1,
	55	+ 65 => 1,
	56	+ 66 => 3,
	57	+ 70 => 1,
	58	+ 76 => 2,
	59	+ 77 => 1,
	60	+ 78 => 7,
	61	+ 79 => 2,
	62	+ 81 => 2,
	63	+ 82 => 4,
	64	+ 87 => 1,
	65	+ 88 => 1,
	66	+ 92 => 1,
	67	+ 97 => 1,
	68	+ 100 => 1,
	69	+ 101 => 1,
	70	+ 102 => 1,
	71	+ 105 => 1,
	72	+ 106 => 1,
	73	+ 107 => 1,
	74	+ 125 => 1,
	75	+ 126 => 1,
	76	+ 141 => 1,
	77	+ 144 => 1,
	78	+ 146 => 1,
	79	+ 148 => 1,
	80	+ 151 => 1,
	81	+ 157 => 1,
	82	+ 174 => 3,
	83	+ 179 => 1,
	84	+ 182 => 1,
	85	+ 188 => 1,
	86	+ 207 => 1,
	87	+ 212 => 2,
	88	+ 214 => 1,
	89	+ 218 => 2,
	90	+ 219 => 2,
	91	+ 223 => 1,
	92	+ 255 => 1,
	93	+ 294 => 1,
	94	+ 295 => 1,
	95	+ 296 => 1,
	96	+ 311 => 1,
	97	+ 317 => 1,
	98	+ 339 => 2,
	99	+ 348 => 2,
	100	+ 352 => 2,
	101	+ 355 => 3,
	102	+ 358 => 3,
	103	+ 359 => 2,
	104	+ 360 => 1,
	105	+ 362 => 1,
	106	+ 363 => 2,
	107	+ 364 => 1,
	108	+ 365 => 2,
	109	+ 366 => 2,
	110	+ 367 => 2,
	111	+ 368 => 2,
	112	+ 369 => 1,
	113	+ 370 => 1,
	114	+ 383 => 1,
	115	+ 394 => 1,
	116	+ ];
	117	+ case 'ArrayDeclarationUnitTest.2.inc':
	118	+ return [
	119	+ 2 => 1,
	120	+ 10 => 1,
	121	+ 23 => 2,
	122	+ 24 => 2,
	123	+ 25 => 1,
	124	+ 31 => 2,
	125	+ 36 => 2,
	126	+ 41 => 1,
	127	+ 46 => 1,
	128	+ 47 => 1,
	129	+ 51 => 1,
	130	+ 53 => 1,
	131	+ 56 => 1,
	132	+ 61 => 1,
	133	+ 63 => 1,
	134	+ 64 => 1,
	135	+ 65 => 1,
	136	+ 66 => 2,
	137	+ 70 => 1,
	138	+ 76 => 1,
	139	+ 77 => 1,
	140	+ 78 => 7,
	141	+ 79 => 2,
	142	+ 81 => 2,
	143	+ 82 => 4,
	144	+ 87 => 1,
	145	+ 88 => 1,
	146	+ 92 => 1,
	147	+ 97 => 1,
	148	+ 100 => 1,
	149	+ 101 => 1,
	150	+ 102 => 1,
	151	+ 105 => 1,
	152	+ 106 => 1,
	153	+ 107 => 1,
	154	+ 125 => 1,
	155	+ 126 => 1,
	156	+ 141 => 1,
	157	+ 144 => 1,
	158	+ 146 => 1,
	159	+ 148 => 1,
	160	+ 151 => 1,
	161	+ 157 => 1,
	162	+ 174 => 3,
	163	+ 179 => 1,
	164	+ 190 => 1,
	165	+ 191 => 1,
	166	+ 192 => 1,
	167	+ 207 => 1,
	168	+ 210 => 1,
	169	+ 211 => 1,
	170	+ 215 => 1,
	171	+ 247 => 1,
	172	+ 286 => 1,
	173	+ 287 => 1,
	174	+ 288 => 1,
	175	+ 303 => 1,
	176	+ 309 => 1,
	177	+ 331 => 2,
	178	+ 345 => 3,
	179	+ 348 => 3,
	180	+ 349 => 2,
	181	+ 350 => 1,
	182	+ 352 => 2,
	183	+ 353 => 2,
	184	+ 354 => 2,
	185	+ 355 => 2,
	186	+ 356 => 2,
	187	+ 357 => 1,
	188	+ 358 => 1,
	189	+ 372 => 1,
	190	+ 383 => 1,
	191	+ ];
	192	+ default:
	193	+ return [];
194	194	}//end switch
195	195
196	196	}//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.php 1 patch

Switch Indentation +60 added lines, -60 removed lines patch added patch discarded remove patch

		@@ -28,66 +28,66 @@
		block discarded – undo
28	28	public function getErrorList($testFile='OperatorBracketUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'OperatorBracketUnitTest.inc':
32		- return [
33		- 3 => 1,
34		- 6 => 1,
35		- 9 => 1,
36		- 12 => 1,
37		- 15 => 1,
38		- 18 => 2,
39		- 20 => 1,
40		- 25 => 1,
41		- 28 => 1,
42		- 31 => 1,
43		- 34 => 1,
44		- 37 => 1,
45		- 40 => 1,
46		- 43 => 2,
47		- 45 => 1,
48		- 47 => 5,
49		- 48 => 1,
50		- 50 => 2,
51		- 55 => 2,
52		- 56 => 1,
53		- 63 => 2,
54		- 64 => 1,
55		- 67 => 1,
56		- 86 => 1,
57		- 90 => 1,
58		- 109 => 1,
59		- 130 => 1,
60		- 134 => 1,
61		- 135 => 2,
62		- 137 => 1,
63		- 139 => 1,
64		- 150 => 1,
65		- 161 => 1,
66		- 163 => 2,
67		- 165 => 2,
68		- 169 => 1,
69		- ];
70		- break;
71		- case 'OperatorBracketUnitTest.js':
72		- return [
73		- 5 => 1,
74		- 8 => 1,
75		- 11 => 1,
76		- 14 => 1,
77		- 24 => 1,
78		- 30 => 1,
79		- 33 => 1,
80		- 36 => 1,
81		- 39 => 1,
82		- 46 => 1,
83		- 47 => 1,
84		- 63 => 1,
85		- 108 => 1,
86		- ];
87		- break;
88		- default:
89		- return [];
90		- break;
	31	+ case 'OperatorBracketUnitTest.inc':
	32	+ return [
	33	+ 3 => 1,
	34	+ 6 => 1,
	35	+ 9 => 1,
	36	+ 12 => 1,
	37	+ 15 => 1,
	38	+ 18 => 2,
	39	+ 20 => 1,
	40	+ 25 => 1,
	41	+ 28 => 1,
	42	+ 31 => 1,
	43	+ 34 => 1,
	44	+ 37 => 1,
	45	+ 40 => 1,
	46	+ 43 => 2,
	47	+ 45 => 1,
	48	+ 47 => 5,
	49	+ 48 => 1,
	50	+ 50 => 2,
	51	+ 55 => 2,
	52	+ 56 => 1,
	53	+ 63 => 2,
	54	+ 64 => 1,
	55	+ 67 => 1,
	56	+ 86 => 1,
	57	+ 90 => 1,
	58	+ 109 => 1,
	59	+ 130 => 1,
	60	+ 134 => 1,
	61	+ 135 => 2,
	62	+ 137 => 1,
	63	+ 139 => 1,
	64	+ 150 => 1,
	65	+ 161 => 1,
	66	+ 163 => 2,
	67	+ 165 => 2,
	68	+ 169 => 1,
	69	+ ];
	70	+ break;
	71	+ case 'OperatorBracketUnitTest.js':
	72	+ return [
	73	+ 5 => 1,
	74	+ 8 => 1,
	75	+ 11 => 1,
	76	+ 14 => 1,
	77	+ 24 => 1,
	78	+ 30 => 1,
	79	+ 33 => 1,
	80	+ 36 => 1,
	81	+ 39 => 1,
	82	+ 46 => 1,
	83	+ 47 => 1,
	84	+ 63 => 1,
	85	+ 108 => 1,
	86	+ ];
	87	+ break;
	88	+ default:
	89	+ return [];
	90	+ break;
91	91	}//end switch
92	92
93	93	}//end getErrorList()

Please login to merge, or discard this patch.

		@@ -1,9 +1,1 @@
		block discarded – undo
1	1	<?php
2		-/**
3		- * File comment.
4		- *
5		- * @package Package
6		- * @subpackage Subpackage
7		- * @author Squiz Pty Ltd <[email protected]>
8		- * @copyright 2010-2014 Squiz Pty Ltd (ABN 77 084 670 600)
9		- *

		@@ -144,26 +144,26 @@
		block discarded – undo
144	144	}
145	145
146	146	switch ($var) {
147		- case 'foo':
148		- /*
	147	+ case 'foo':
	148	+ /*
149	149	Foo comment.
150	150	This is a multiple
151	151	line comment for Foo.
152	152	*/
153	153
154		- echo 'Foo';
155		- break;
	154	+ echo 'Foo';
	155	+ break;
156	156
157		- default:
	157	+ default:
158	158
159		- /*
	159	+ /*
160	160	Foo comment.
161	161	This is a multiple
162	162	line comment for Foo.
163	163	*/
164	164
165		- echo 'Default';
166		- break;
	165	+ echo 'Default';
	166	+ break;
167	167	}//end switch
168	168
169	169	/**

		@@ -28,23 +28,23 @@
		block discarded – undo
28	28	public function getErrorList($testFile='PostStatementCommentUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'PostStatementCommentUnitTest.inc':
32		- return [
33		- 6 => 1,
34		- 10 => 1,
35		- 18 => 1,
36		- 35 => 1,
37		- ];
	31	+ case 'PostStatementCommentUnitTest.inc':
	32	+ return [
	33	+ 6 => 1,
	34	+ 10 => 1,
	35	+ 18 => 1,
	36	+ 35 => 1,
	37	+ ];
38	38
39		- case 'PostStatementCommentUnitTest.1.js':
40		- return [
41		- 1 => 1,
42		- 4 => 1,
43		- 9 => 1,
44		- ];
	39	+ case 'PostStatementCommentUnitTest.1.js':
	40	+ return [
	41	+ 1 => 1,
	42	+ 4 => 1,
	43	+ 9 => 1,
	44	+ ];
45	45
46		- default:
47		- return [];
	46	+ default:
	47	+ return [];
48	48	}//end switch
49	49
50	50	}//end getErrorList()

		@@ -28,29 +28,29 @@
		block discarded – undo
28	28	public function getErrorList($testFile='FileCommentUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'FileCommentUnitTest.1.inc':
32		- case 'FileCommentUnitTest.1.js':
33		- return [
34		- 1 => 1,
35		- 22 => 2,
36		- 23 => 1,
37		- 24 => 2,
38		- 25 => 2,
39		- 26 => 1,
40		- 27 => 2,
41		- 28 => 2,
42		- 32 => 2,
43		- ];
	31	+ case 'FileCommentUnitTest.1.inc':
	32	+ case 'FileCommentUnitTest.1.js':
	33	+ return [
	34	+ 1 => 1,
	35	+ 22 => 2,
	36	+ 23 => 1,
	37	+ 24 => 2,
	38	+ 25 => 2,
	39	+ 26 => 1,
	40	+ 27 => 2,
	41	+ 28 => 2,
	42	+ 32 => 2,
	43	+ ];
44	44
45		- case 'FileCommentUnitTest.4.inc':
46		- case 'FileCommentUnitTest.6.inc':
47		- return [1 => 1];
	45	+ case 'FileCommentUnitTest.4.inc':
	46	+ case 'FileCommentUnitTest.6.inc':
	47	+ return [1 => 1];
48	48
49		- case 'FileCommentUnitTest.5.inc':
50		- return [2 => 1];
	49	+ case 'FileCommentUnitTest.5.inc':
	50	+ return [2 => 1];
51	51
52		- default:
53		- return [];
	52	+ default:
	53	+ return [];
54	54	}//end switch
55	55
56	56	}//end getErrorList()

		@@ -493,41 +493,41 @@ discard block
		block discarded – undo
493	493	}
494	494
495	495	switch ($something) {
496		- case '1':
497		- // Line 1
498		- // Line 2
499		- // Line 3
500		- // Line 4
501		- // Line 5
502		- break;
503		- case '2':
504		- // Line 1
505		- // Line 2
506		- // Line 3
507		- // Line 4
508		- // Line 5
509		- break;
510		- case '3':
511		- // Line 1
512		- // Line 2
513		- // Line 3
514		- // Line 4
515		- // Line 5
516		- break;
517		- case '4':
518		- // Line 1
519		- // Line 2
520		- // Line 3
521		- // Line 4
522		- // Line 5
523		- break;
524		- case '5':
525		- // Line 1
526		- // Line 2
527		- // Line 3
528		- // Line 4
529		- // Line 5
530		- break;
	496	+ case '1':
	497	+ // Line 1
	498	+ // Line 2
	499	+ // Line 3
	500	+ // Line 4
	501	+ // Line 5
	502	+ break;
	503	+ case '2':
	504	+ // Line 1
	505	+ // Line 2
	506	+ // Line 3
	507	+ // Line 4
	508	+ // Line 5
	509	+ break;
	510	+ case '3':
	511	+ // Line 1
	512	+ // Line 2
	513	+ // Line 3
	514	+ // Line 4
	515	+ // Line 5
	516	+ break;
	517	+ case '4':
	518	+ // Line 1
	519	+ // Line 2
	520	+ // Line 3
	521	+ // Line 4
	522	+ // Line 5
	523	+ break;
	524	+ case '5':
	525	+ // Line 1
	526	+ // Line 2
	527	+ // Line 3
	528	+ // Line 4
	529	+ // Line 5
	530	+ break;
531	531	}
532	532
533	533	// Wrong comment
		@@ -629,69 +629,69 @@ discard block
		block discarded – undo
629	629	}
630	630
631	631	switch ($foo) {
632		- case 'one' : {
633		- // some code here.
634		- // some code here.
635		- // some code here.
636		- // some code here.
637		- // some code here.
638		- // some code here.
639		- // some code here.
640		- // some code here.
641		- // some code here.
642		- // some code here.
643		- // some code here.
644		- // some code here.
645		- // some code here.
646		- // some code here.
647		- // some code here.
648		- // some code here.
649		- // some code here.
650		- // some code here.
651		- // some code here.
652		- // some code here.
653		- // some code here.
654		- // some code here.
655		- // some code here.
656		- // some code here.
657		- // some code here.
658		- // some code here.
659		- // some code here.
660		- // some code here.
661		- // some code here.
	632	+ case 'one' : {
	633	+ // some code here.
	634	+ // some code here.
	635	+ // some code here.
	636	+ // some code here.
	637	+ // some code here.
	638	+ // some code here.
	639	+ // some code here.
	640	+ // some code here.
	641	+ // some code here.
	642	+ // some code here.
	643	+ // some code here.
	644	+ // some code here.
	645	+ // some code here.
	646	+ // some code here.
	647	+ // some code here.
	648	+ // some code here.
	649	+ // some code here.
	650	+ // some code here.
	651	+ // some code here.
	652	+ // some code here.
	653	+ // some code here.
	654	+ // some code here.
	655	+ // some code here.
	656	+ // some code here.
	657	+ // some code here.
	658	+ // some code here.
	659	+ // some code here.
	660	+ // some code here.
	661	+ // some code here.
	662	+ break;
	663	+ }
	664	+ case 'one' :
	665	+ // some code here.
	666	+ // some code here.
	667	+ // some code here.
	668	+ // some code here.
	669	+ // some code here.
	670	+ // some code here.
	671	+ // some code here.
	672	+ // some code here.
	673	+ // some code here.
	674	+ // some code here.
	675	+ // some code here.
	676	+ // some code here.
	677	+ // some code here.
	678	+ // some code here.
	679	+ // some code here.
	680	+ // some code here.
	681	+ // some code here.
	682	+ // some code here.
	683	+ // some code here.
	684	+ // some code here.
	685	+ // some code here.
	686	+ // some code here.
	687	+ // some code here.
	688	+ // some code here.
	689	+ // some code here.
	690	+ // some code here.
	691	+ // some code here.
	692	+ // some code here.
	693	+ // some code here.
662	694	break;
663		- }
664		- case 'one' :
665		- // some code here.
666		- // some code here.
667		- // some code here.
668		- // some code here.
669		- // some code here.
670		- // some code here.
671		- // some code here.
672		- // some code here.
673		- // some code here.
674		- // some code here.
675		- // some code here.
676		- // some code here.
677		- // some code here.
678		- // some code here.
679		- // some code here.
680		- // some code here.
681		- // some code here.
682		- // some code here.
683		- // some code here.
684		- // some code here.
685		- // some code here.
686		- // some code here.
687		- // some code here.
688		- // some code here.
689		- // some code here.
690		- // some code here.
691		- // some code here.
692		- // some code here.
693		- // some code here.
694		- break;
695	695	}//end switch
696	696
697	697	// Yes, code like this does exist.
		@@ -701,67 +701,67 @@ discard block
		block discarded – undo
701	701	return $bar;
702	702
703	703	switch ($foo) {
704		- case 1:
705		- case 2:
706		- case 3:
707		- case 4:
708		- case 5:
709		- case 6:
710		- case 7:
711		- case 8:
712		- case 9:
713		- case 10:
714		- case 11:
715		- case 12:
716		- case 13:
717		- case 14:
718		- case 15:
719		- case 16:
720		- case 17:
721		- case 18:
722		- case 19:
723		- case 20:
724		- case 21:
725		- case 22:
726		- case 23:
727		- case 24:
728		- case 25:
729		- case 26:
730		- case 27:
731		- case 28:
732		- case 29:
733		- case 30:
734		- case 31:
735		- case 32:
736		- case 33:
737		- case 34:
738		- case 35:
739		- case 36:
740		- case 37:
741		- case 38:
742		- case 39:
743		- case 40:
744		- case 41:
745		- case 42:
746		- case 43:
747		- case 44:
748		- case 45:
749		- case 46:
750		- case 47:
751		- case 48:
752		- case 49:
753		- case 50:
754		- case 51:
755		- case 52:
756		- case 53:
757		- case 54:
758		- case 55:
759		- case 56:
760		- case 57:
761		- case 58:
762		- case 59:
763		- echo $foo;
764		- break;
	704	+ case 1:
	705	+ case 2:
	706	+ case 3:
	707	+ case 4:
	708	+ case 5:
	709	+ case 6:
	710	+ case 7:
	711	+ case 8:
	712	+ case 9:
	713	+ case 10:
	714	+ case 11:
	715	+ case 12:
	716	+ case 13:
	717	+ case 14:
	718	+ case 15:
	719	+ case 16:
	720	+ case 17:
	721	+ case 18:
	722	+ case 19:
	723	+ case 20:
	724	+ case 21:
	725	+ case 22:
	726	+ case 23:
	727	+ case 24:
	728	+ case 25:
	729	+ case 26:
	730	+ case 27:
	731	+ case 28:
	732	+ case 29:
	733	+ case 30:
	734	+ case 31:
	735	+ case 32:
	736	+ case 33:
	737	+ case 34:
	738	+ case 35:
	739	+ case 36:
	740	+ case 37:
	741	+ case 38:
	742	+ case 39:
	743	+ case 40:
	744	+ case 41:
	745	+ case 42:
	746	+ case 43:
	747	+ case 44:
	748	+ case 45:
	749	+ case 46:
	750	+ case 47:
	751	+ case 48:
	752	+ case 49:
	753	+ case 50:
	754	+ case 51:
	755	+ case 52:
	756	+ case 53:
	757	+ case 54:
	758	+ case 55:
	759	+ case 56:
	760	+ case 57:
	761	+ case 58:
	762	+ case 59:
	763	+ echo $foo;
	764	+ break;
765	765	}
766	766
767	767	// phpcs:set Squiz.Commenting.LongConditionClosingComment lineLimit 5

cycle / annotated

Pull Request — master (#5)

Category

Switch Indentation -8 removed lines patch added patch discarded remove patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Switch Indentation +15 added lines, -15 removed lines patch added patch discarded remove patch

Switch Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

Switch Indentation +158 added lines, -158 removed lines patch added patch discarded remove patch

Switch Indentation +39 added lines, -39 removed lines patch added patch discarded remove patch

Switch Indentation +49 added lines, -49 removed lines patch added patch discarded remove patch

Switch Indentation +163 added lines, -163 removed lines patch added patch discarded remove patch

Switch Indentation +60 added lines, -60 removed lines patch added patch discarded remove patch