Inspection of "Refresh admin styles" - gravityview/GravityView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#1492)

by Zack

created 2020-08-24 21:19 UTC

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

@@ -26,14 +26,14 @@  discard block
 block discarded – undo
         $case = '1';
         return '1';
 
-case '2':
+	case '2':
     case '3':
         $case = '5';
     break;
 
-case '4':
-    $case = '4';
-break;
+	case '4':
+    	$case = '4';
+	break;
 
     default:
         $case = null;
@@ -49,19 +49,19 @@  discard block
 block discarded – undo
 
 // PEAR style.
 switch ($something) {
-case '1':
-    $case = '1';
-    break;
-case '2':
-case '3':
-    $case = '5';
-    break;
-case '4':
-    $case = '4';
-    break;
-default:
-    $case = null;
-    break;
+	case '1':
+    	$case = '1';
+    	break;
+	case '2':
+	case '3':
+    	$case = '5';
+    	break;
+	case '4':
+    	$case = '4';
+    	break;
+	default:
+    	$case = null;
+    	break;
 }
 
 // Valid, but missing BREAKS.
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
     Case '1' :
         $case = '1';
 
-case  '2':
+	case  '2':
     case  '3' :
         $case = '5';
 
@@ -140,21 +140,21 @@  discard block
 block discarded – undo
 }
 
 switch ($var) {
-    case 'one':
-    case 'two':
-    break;
+    	case 'one':
+    	case 'two':
+    	break;
 
-    case 'three':
-        // Nothing to do.
-    break;
+    	case 'three':
+        	// Nothing to do.
+    	break;
 
-    case 'four':
-        echo $hi;
-    break;
+    	case 'four':
+        	echo $hi;
+    	break;
 
-    default:
-        // No default.
-    break;
+    	default:
+        	// No default.
+    	break;
 }
 
 switch ($var) {
@@ -170,8 +170,8 @@  discard block
 block discarded – undo
 }
 
 switch ($name) {
-    case "1":
-        switch ($name2) {
+    	case "1":
+        	switch ($name2) {
             case "1":
                 return true;
             break;
@@ -183,24 +183,24 @@  discard block
 block discarded – undo
             default:
                 // No default.
             break;
-        }
+        	}
     break;
 
-    case "2":
+    	case "2":
 switch ($name2) {
-    case "1":
-        return true;
-    break;
+    		case "1":
+        		return true;
+    		break;
 
-    case "2":
-    return true;
-    break;
+    		case "2":
+    		return true;
+    		break;
 
-    default:
-        // No default.
-    break;
+    		default:
+        		// No default.
+    		break;
 }
-    break;
+    	break;
 }
 
 switch ($name) {
@@ -221,9 +221,9 @@  discard block
 block discarded – undo
 }
 
 switch ($name2) {
-    default:
-        // No default.
-    break;
+    	default:
+        	// No default.
+    	break;
 }
 
 switch ($foo) {
@@ -258,15 +258,15 @@  discard block
 block discarded – undo
 }
 
 switch ($something) {
-    case '1':
-        $case = '1';
-    break;
+    	case '1':
+        	$case = '1';
+    	break;
 
-    case '2':
-        throw new Exception('message');
+    	case '2':
+        	throw new Exception('message');
 
-    default:
-    throw new Exception('message');
+    	default:
+    	throw new Exception('message');
 }
 
 switch ($something) {
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
 }
 
 switch ($foo) {
-    case '1':
-        return; // comment
-    break;
+    	case '1':
+        	return; // comment
+    	break;
 
 }
 
@@ -305,13 +305,13 @@  discard block
 block discarded – undo
 }
 
 switch ($foo) {
-    case 'bar':
-    throw new \Exception(
-        'bar'
-    );
-
-    default:
-    throw new \Exception(
-        'bar'
-    );
+    	case 'bar':
+    	throw new \Exception(
+        	'bar'
+    	);
+
+    	default:
+    	throw new \Exception(
+        	'bar'
+    	);
 }

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.php 1 patch

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

		@@ -28,76 +28,76 @@ discard block
		block discarded – undo
28	28	public function getErrorList($testFile='ForLoopDeclarationUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'ForLoopDeclarationUnitTest.inc':
32		- return [
33		- 8 => 2,
34		- 11 => 2,
35		- 14 => 2,
36		- 17 => 2,
37		- 21 => 6,
38		- 27 => 1,
39		- 30 => 1,
40		- 37 => 2,
41		- 39 => 2,
42		- 43 => 1,
43		- 49 => 1,
44		- 50 => 1,
45		- 53 => 1,
46		- 54 => 1,
47		- 59 => 4,
48		- 62 => 1,
49		- 63 => 1,
50		- 64 => 1,
51		- 66 => 1,
52		- 69 => 1,
53		- 74 => 1,
54		- 77 => 1,
55		- 82 => 2,
56		- 86 => 2,
57		- 91 => 1,
58		- 95 => 1,
59		- 101 => 2,
60		- 105 => 2,
61		- 110 => 1,
62		- 116 => 2,
63		- ];
	31	+ case 'ForLoopDeclarationUnitTest.inc':
	32	+ return [
	33	+ 8 => 2,
	34	+ 11 => 2,
	35	+ 14 => 2,
	36	+ 17 => 2,
	37	+ 21 => 6,
	38	+ 27 => 1,
	39	+ 30 => 1,
	40	+ 37 => 2,
	41	+ 39 => 2,
	42	+ 43 => 1,
	43	+ 49 => 1,
	44	+ 50 => 1,
	45	+ 53 => 1,
	46	+ 54 => 1,
	47	+ 59 => 4,
	48	+ 62 => 1,
	49	+ 63 => 1,
	50	+ 64 => 1,
	51	+ 66 => 1,
	52	+ 69 => 1,
	53	+ 74 => 1,
	54	+ 77 => 1,
	55	+ 82 => 2,
	56	+ 86 => 2,
	57	+ 91 => 1,
	58	+ 95 => 1,
	59	+ 101 => 2,
	60	+ 105 => 2,
	61	+ 110 => 1,
	62	+ 116 => 2,
	63	+ ];
64	64
65		- case 'ForLoopDeclarationUnitTest.js':
66		- return [
67		- 6 => 2,
68		- 9 => 2,
69		- 12 => 2,
70		- 15 => 2,
71		- 19 => 6,
72		- 33 => 1,
73		- 36 => 1,
74		- 43 => 2,
75		- 45 => 2,
76		- 49 => 1,
77		- 55 => 1,
78		- 56 => 1,
79		- 59 => 1,
80		- 60 => 1,
81		- 65 => 4,
82		- 68 => 1,
83		- 69 => 1,
84		- 70 => 1,
85		- 72 => 1,
86		- 75 => 1,
87		- 80 => 1,
88		- 83 => 1,
89		- 88 => 2,
90		- 92 => 2,
91		- 97 => 1,
92		- 101 => 1,
93		- 107 => 2,
94		- 111 => 2,
95		- 116 => 1,
96		- 122 => 2,
97		- ];
	65	+ case 'ForLoopDeclarationUnitTest.js':
	66	+ return [
	67	+ 6 => 2,
	68	+ 9 => 2,
	69	+ 12 => 2,
	70	+ 15 => 2,
	71	+ 19 => 6,
	72	+ 33 => 1,
	73	+ 36 => 1,
	74	+ 43 => 2,
	75	+ 45 => 2,
	76	+ 49 => 1,
	77	+ 55 => 1,
	78	+ 56 => 1,
	79	+ 59 => 1,
	80	+ 60 => 1,
	81	+ 65 => 4,
	82	+ 68 => 1,
	83	+ 69 => 1,
	84	+ 70 => 1,
	85	+ 72 => 1,
	86	+ 75 => 1,
	87	+ 80 => 1,
	88	+ 83 => 1,
	89	+ 88 => 2,
	90	+ 92 => 2,
	91	+ 97 => 1,
	92	+ 101 => 1,
	93	+ 107 => 2,
	94	+ 111 => 2,
	95	+ 116 => 1,
	96	+ 122 => 2,
	97	+ ];
98	98
99		- default:
100		- return [];
	99	+ default:
	100	+ return [];
101	101	}//end switch
102	102
103	103	}//end getErrorList()
		@@ -116,14 +116,14 @@ discard block
		block discarded – undo
116	116	public function getWarningList($testFile='ForLoopDeclarationUnitTest.inc')
117	117	{
118	118	switch ($testFile) {
119		- case 'ForLoopDeclarationUnitTest.inc':
120		- return [119 => 1];
	119	+ case 'ForLoopDeclarationUnitTest.inc':
	120	+ return [119 => 1];
121	121
122		- case 'ForLoopDeclarationUnitTest.js':
123		- return [125 => 1];
	122	+ case 'ForLoopDeclarationUnitTest.js':
	123	+ return [125 => 1];
124	124
125		- default:
126		- return [];
	125	+ default:
	126	+ return [];
127	127	}//end switch
128	128
129	129	}//end getWarningList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/ControlStructures/ControlSignatureUnitTest.inc 1 patch

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

@@ -157,8 +157,8 @@
 block discarded – undo
 
 switch($foo) {
 
-    case 'bar':
-    break;
+    	case 'bar':
+    	break;
 
 }
 

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/WhiteSpace/FunctionSpacingUnitTest.php 1 patch

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

		@@ -28,69 +28,69 @@
		block discarded – undo
28	28	public function getErrorList($testFile='')
29	29	{
30	30	switch ($testFile) {
31		- case 'FunctionSpacingUnitTest.1.inc':
32		- return [
33		- 26 => 1,
34		- 35 => 1,
35		- 44 => 1,
36		- 51 => 1,
37		- 55 => 1,
38		- 61 => 1,
39		- 64 => 1,
40		- 66 => 1,
41		- 81 => 1,
42		- 100 => 1,
43		- 111 => 1,
44		- 113 => 1,
45		- 119 => 2,
46		- 141 => 1,
47		- 160 => 1,
48		- 173 => 2,
49		- 190 => 1,
50		- 224 => 2,
51		- 281 => 1,
52		- 282 => 1,
53		- 295 => 1,
54		- 297 => 1,
55		- 303 => 1,
56		- 327 => 1,
57		- 329 => 1,
58		- 338 => 1,
59		- 344 => 1,
60		- 345 => 1,
61		- 354 => 2,
62		- 355 => 1,
63		- 356 => 1,
64		- 360 => 2,
65		- 361 => 1,
66		- 362 => 1,
67		- 385 => 1,
68		- 399 => 1,
69		- 411 => 2,
70		- 418 => 2,
71		- 426 => 2,
72		- 432 => 1,
73		- 437 => 1,
74		- 438 => 1,
75		- 442 => 2,
76		- 444 => 1,
77		- 449 => 1,
78		- ];
79		-
80		- case 'FunctionSpacingUnitTest.2.inc':
81		- return [2 => 1];
82		-
83		- case 'FunctionSpacingUnitTest.3.inc':
84		- return [7 => 1];
85		-
86		- case 'FunctionSpacingUnitTest.5.inc':
87		- return [5 => 1];
88		-
89		- case 'FunctionSpacingUnitTest.6.inc':
90		- return [10 => 1];
91		-
92		- default:
93		- return [];
	31	+ case 'FunctionSpacingUnitTest.1.inc':
	32	+ return [
	33	+ 26 => 1,
	34	+ 35 => 1,
	35	+ 44 => 1,
	36	+ 51 => 1,
	37	+ 55 => 1,
	38	+ 61 => 1,
	39	+ 64 => 1,
	40	+ 66 => 1,
	41	+ 81 => 1,
	42	+ 100 => 1,
	43	+ 111 => 1,
	44	+ 113 => 1,
	45	+ 119 => 2,
	46	+ 141 => 1,
	47	+ 160 => 1,
	48	+ 173 => 2,
	49	+ 190 => 1,
	50	+ 224 => 2,
	51	+ 281 => 1,
	52	+ 282 => 1,
	53	+ 295 => 1,
	54	+ 297 => 1,
	55	+ 303 => 1,
	56	+ 327 => 1,
	57	+ 329 => 1,
	58	+ 338 => 1,
	59	+ 344 => 1,
	60	+ 345 => 1,
	61	+ 354 => 2,
	62	+ 355 => 1,
	63	+ 356 => 1,
	64	+ 360 => 2,
	65	+ 361 => 1,
	66	+ 362 => 1,
	67	+ 385 => 1,
	68	+ 399 => 1,
	69	+ 411 => 2,
	70	+ 418 => 2,
	71	+ 426 => 2,
	72	+ 432 => 1,
	73	+ 437 => 1,
	74	+ 438 => 1,
	75	+ 442 => 2,
	76	+ 444 => 1,
	77	+ 449 => 1,
	78	+ ];
	79	+
	80	+ case 'FunctionSpacingUnitTest.2.inc':
	81	+ return [2 => 1];
	82	+
	83	+ case 'FunctionSpacingUnitTest.3.inc':
	84	+ return [7 => 1];
	85	+
	86	+ case 'FunctionSpacingUnitTest.5.inc':
	87	+ return [5 => 1];
	88	+
	89	+ case 'FunctionSpacingUnitTest.6.inc':
	90	+ return [10 => 1];
	91	+
	92	+ default:
	93	+ return [];
94	94	}//end switch
95	95
96	96	}//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/WhiteSpace/FunctionOpeningBraceSpaceUnitTest.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='FunctionOpeningBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
-        case 'FunctionOpeningBraceSpaceUnitTest.inc':
-            return [
-                10 => 1,
-                25 => 1,
-                49 => 1,
-            ];
+        	case 'FunctionOpeningBraceSpaceUnitTest.inc':
+            	return [
+                	10 => 1,
+                	25 => 1,
+                	49 => 1,
+            	];
 
-        case 'FunctionOpeningBraceSpaceUnitTest.js':
-            return [
-                11 => 1,
-                31 => 1,
-                38 => 1,
-                88 => 1,
-            ];
+        	case 'FunctionOpeningBraceSpaceUnitTest.js':
+            	return [
+                	11 => 1,
+                	31 => 1,
+                	38 => 1,
+                	88 => 1,
+            	];
 
-        default:
-            return [];
+        	default:
+            	return [];
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/WhiteSpace/SuperfluousWhitespaceUnitTest.php 1 patch

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

@@ -28,68 +28,68 @@
 block discarded – undo
     public function getErrorList($testFile='SuperfluousWhitespaceUnitTest.inc')
     {
         switch ($testFile) {
-        case 'SuperfluousWhitespaceUnitTest.1.inc':
-            return [
-                2  => 1,
-                4  => 1,
-                5  => 1,
-                6  => 1,
-                7  => 1,
-                16 => 1,
-                23 => 1,
-                28 => 1,
-                33 => 1,
-                49 => 1,
-                62 => 1,
-                65 => 1,
-                73 => 1,
-            ];
-            break;
-        case 'SuperfluousWhitespaceUnitTest.2.inc':
-            return [
-                2 => 1,
-                8 => 1,
-            ];
-            break;
-        case 'SuperfluousWhitespaceUnitTest.3.inc':
-            return [
-                6  => 1,
-                10 => 1,
-            ];
-            break;
-        case 'SuperfluousWhitespaceUnitTest.4.inc':
-        case 'SuperfluousWhitespaceUnitTest.5.inc':
-            return [
-                1 => 1,
-                4 => 1,
-            ];
-            break;
-        case 'SuperfluousWhitespaceUnitTest.1.js':
-            return [
-                1  => 1,
-                3  => 1,
-                4  => 1,
-                5  => 1,
-                6  => 1,
-                15 => 1,
-                22 => 1,
-                29 => 1,
-                38 => 1,
-                56 => 1,
-            ];
-            break;
-        case 'SuperfluousWhitespaceUnitTest.1.css':
-            return [
-                1  => 1,
-                8  => 1,
-                9  => 1,
-                11 => 1,
-                25 => 1,
-            ];
-            break;
-        default:
-            return [];
-            break;
+        	case 'SuperfluousWhitespaceUnitTest.1.inc':
+            	return [
+                	2  => 1,
+                	4  => 1,
+                	5  => 1,
+                	6  => 1,
+                	7  => 1,
+                	16 => 1,
+                	23 => 1,
+                	28 => 1,
+                	33 => 1,
+                	49 => 1,
+                	62 => 1,
+                	65 => 1,
+                	73 => 1,
+            	];
+            	break;
+        	case 'SuperfluousWhitespaceUnitTest.2.inc':
+            	return [
+                	2 => 1,
+                	8 => 1,
+            	];
+            	break;
+        	case 'SuperfluousWhitespaceUnitTest.3.inc':
+            	return [
+                	6  => 1,
+                	10 => 1,
+            	];
+            	break;
+        	case 'SuperfluousWhitespaceUnitTest.4.inc':
+        	case 'SuperfluousWhitespaceUnitTest.5.inc':
+            	return [
+                	1 => 1,
+                	4 => 1,
+            	];
+            	break;
+        	case 'SuperfluousWhitespaceUnitTest.1.js':
+            	return [
+                	1  => 1,
+                	3  => 1,
+                	4  => 1,
+                	5  => 1,
+                	6  => 1,
+                	15 => 1,
+                	22 => 1,
+                	29 => 1,
+                	38 => 1,
+                	56 => 1,
+            	];
+            	break;
+        	case 'SuperfluousWhitespaceUnitTest.1.css':
+            	return [
+                	1  => 1,
+                	8  => 1,
+                	9  => 1,
+                	11 => 1,
+                	25 => 1,
+            	];
+            	break;
+        	default:
+            	return [];
+            	break;
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/WhiteSpace/ControlStructureSpacingUnitTest.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='ControlStructureSpacingUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'ControlStructureSpacingUnitTest.inc':
32		- return [
33		- 3 => 1,
34		- 5 => 1,
35		- 8 => 1,
36		- 15 => 1,
37		- 23 => 1,
38		- 74 => 1,
39		- 79 => 1,
40		- 82 => 1,
41		- 83 => 1,
42		- 87 => 1,
43		- 103 => 1,
44		- 113 => 2,
45		- 114 => 2,
46		- 118 => 1,
47		- 150 => 1,
48		- 153 => 1,
49		- 154 => 1,
50		- 157 => 1,
51		- 170 => 1,
52		- 176 => 2,
53		- 179 => 1,
54		- 189 => 1,
55		- 225 => 1,
56		- 237 => 1,
57		- 242 => 1,
58		- 246 => 1,
59		- 248 => 1,
60		- ];
61		- break;
62		- case 'ControlStructureSpacingUnitTest.js':
63		- return [
64		- 3 => 1,
65		- 9 => 1,
66		- 15 => 1,
67		- 21 => 1,
68		- 56 => 1,
69		- 61 => 1,
70		- 64 => 1,
71		- 65 => 1,
72		- 68 => 1,
73		- 74 => 2,
74		- 75 => 2,
75		- ];
76		- break;
77		- default:
78		- return [];
79		- break;
	31	+ case 'ControlStructureSpacingUnitTest.inc':
	32	+ return [
	33	+ 3 => 1,
	34	+ 5 => 1,
	35	+ 8 => 1,
	36	+ 15 => 1,
	37	+ 23 => 1,
	38	+ 74 => 1,
	39	+ 79 => 1,
	40	+ 82 => 1,
	41	+ 83 => 1,
	42	+ 87 => 1,
	43	+ 103 => 1,
	44	+ 113 => 2,
	45	+ 114 => 2,
	46	+ 118 => 1,
	47	+ 150 => 1,
	48	+ 153 => 1,
	49	+ 154 => 1,
	50	+ 157 => 1,
	51	+ 170 => 1,
	52	+ 176 => 2,
	53	+ 179 => 1,
	54	+ 189 => 1,
	55	+ 225 => 1,
	56	+ 237 => 1,
	57	+ 242 => 1,
	58	+ 246 => 1,
	59	+ 248 => 1,
	60	+ ];
	61	+ break;
	62	+ case 'ControlStructureSpacingUnitTest.js':
	63	+ return [
	64	+ 3 => 1,
	65	+ 9 => 1,
	66	+ 15 => 1,
	67	+ 21 => 1,
	68	+ 56 => 1,
	69	+ 61 => 1,
	70	+ 64 => 1,
	71	+ 65 => 1,
	72	+ 68 => 1,
	73	+ 74 => 2,
	74	+ 75 => 2,
	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/WhiteSpace/OperatorSpacingUnitTest.php 1 patch

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

		@@ -28,121 +28,121 @@
		block discarded – undo
28	28	public function getErrorList($testFile='OperatorSpacingUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'OperatorSpacingUnitTest.inc':
32		- return [
33		- 4 => 1,
34		- 5 => 2,
35		- 6 => 1,
36		- 7 => 1,
37		- 8 => 2,
38		- 11 => 1,
39		- 12 => 2,
40		- 13 => 1,
41		- 14 => 1,
42		- 15 => 2,
43		- 18 => 1,
44		- 19 => 2,
45		- 20 => 1,
46		- 21 => 1,
47		- 22 => 2,
48		- 25 => 1,
49		- 26 => 2,
50		- 27 => 1,
51		- 28 => 1,
52		- 29 => 2,
53		- 32 => 1,
54		- 33 => 2,
55		- 34 => 1,
56		- 35 => 1,
57		- 36 => 2,
58		- 40 => 2,
59		- 42 => 2,
60		- 44 => 2,
61		- 45 => 1,
62		- 46 => 2,
63		- 53 => 4,
64		- 54 => 3,
65		- 59 => 10,
66		- 64 => 1,
67		- 77 => 4,
68		- 78 => 1,
69		- 79 => 1,
70		- 80 => 2,
71		- 81 => 1,
72		- 84 => 6,
73		- 85 => 6,
74		- 87 => 4,
75		- 88 => 5,
76		- 90 => 4,
77		- 91 => 5,
78		- 128 => 4,
79		- 132 => 1,
80		- 133 => 1,
81		- 135 => 1,
82		- 136 => 1,
83		- 140 => 1,
84		- 141 => 1,
85		- 174 => 1,
86		- 177 => 1,
87		- 178 => 1,
88		- 179 => 1,
89		- 185 => 2,
90		- 191 => 4,
91		- 194 => 1,
92		- 195 => 1,
93		- 196 => 2,
94		- 199 => 1,
95		- 200 => 1,
96		- 201 => 2,
97		- 239 => 1,
98		- 246 => 1,
99		- ];
100		- break;
101		- case 'OperatorSpacingUnitTest.js':
102		- return [
103		- 4 => 1,
104		- 5 => 2,
105		- 6 => 1,
106		- 7 => 1,
107		- 8 => 2,
108		- 11 => 1,
109		- 12 => 2,
110		- 13 => 1,
111		- 14 => 1,
112		- 15 => 2,
113		- 18 => 1,
114		- 19 => 2,
115		- 20 => 1,
116		- 21 => 1,
117		- 22 => 2,
118		- 25 => 1,
119		- 26 => 2,
120		- 27 => 1,
121		- 28 => 1,
122		- 29 => 2,
123		- 32 => 1,
124		- 33 => 2,
125		- 34 => 1,
126		- 35 => 1,
127		- 36 => 2,
128		- 40 => 2,
129		- 42 => 2,
130		- 44 => 2,
131		- 45 => 1,
132		- 46 => 2,
133		- 55 => 4,
134		- 65 => 1,
135		- 66 => 1,
136		- 68 => 1,
137		- 69 => 1,
138		- 73 => 1,
139		- 74 => 1,
140		- 100 => 1,
141		- ];
142		- break;
143		- default:
144		- return [];
145		- break;
	31	+ case 'OperatorSpacingUnitTest.inc':
	32	+ return [
	33	+ 4 => 1,
	34	+ 5 => 2,
	35	+ 6 => 1,
	36	+ 7 => 1,
	37	+ 8 => 2,
	38	+ 11 => 1,
	39	+ 12 => 2,
	40	+ 13 => 1,
	41	+ 14 => 1,
	42	+ 15 => 2,
	43	+ 18 => 1,
	44	+ 19 => 2,
	45	+ 20 => 1,
	46	+ 21 => 1,
	47	+ 22 => 2,
	48	+ 25 => 1,
	49	+ 26 => 2,
	50	+ 27 => 1,
	51	+ 28 => 1,
	52	+ 29 => 2,
	53	+ 32 => 1,
	54	+ 33 => 2,
	55	+ 34 => 1,
	56	+ 35 => 1,
	57	+ 36 => 2,
	58	+ 40 => 2,
	59	+ 42 => 2,
	60	+ 44 => 2,
	61	+ 45 => 1,
	62	+ 46 => 2,
	63	+ 53 => 4,
	64	+ 54 => 3,
	65	+ 59 => 10,
	66	+ 64 => 1,
	67	+ 77 => 4,
	68	+ 78 => 1,
	69	+ 79 => 1,
	70	+ 80 => 2,
	71	+ 81 => 1,
	72	+ 84 => 6,
	73	+ 85 => 6,
	74	+ 87 => 4,
	75	+ 88 => 5,
	76	+ 90 => 4,
	77	+ 91 => 5,
	78	+ 128 => 4,
	79	+ 132 => 1,
	80	+ 133 => 1,
	81	+ 135 => 1,
	82	+ 136 => 1,
	83	+ 140 => 1,
	84	+ 141 => 1,
	85	+ 174 => 1,
	86	+ 177 => 1,
	87	+ 178 => 1,
	88	+ 179 => 1,
	89	+ 185 => 2,
	90	+ 191 => 4,
	91	+ 194 => 1,
	92	+ 195 => 1,
	93	+ 196 => 2,
	94	+ 199 => 1,
	95	+ 200 => 1,
	96	+ 201 => 2,
	97	+ 239 => 1,
	98	+ 246 => 1,
	99	+ ];
	100	+ break;
	101	+ case 'OperatorSpacingUnitTest.js':
	102	+ return [
	103	+ 4 => 1,
	104	+ 5 => 2,
	105	+ 6 => 1,
	106	+ 7 => 1,
	107	+ 8 => 2,
	108	+ 11 => 1,
	109	+ 12 => 2,
	110	+ 13 => 1,
	111	+ 14 => 1,
	112	+ 15 => 2,
	113	+ 18 => 1,
	114	+ 19 => 2,
	115	+ 20 => 1,
	116	+ 21 => 1,
	117	+ 22 => 2,
	118	+ 25 => 1,
	119	+ 26 => 2,
	120	+ 27 => 1,
	121	+ 28 => 1,
	122	+ 29 => 2,
	123	+ 32 => 1,
	124	+ 33 => 2,
	125	+ 34 => 1,
	126	+ 35 => 1,
	127	+ 36 => 2,
	128	+ 40 => 2,
	129	+ 42 => 2,
	130	+ 44 => 2,
	131	+ 45 => 1,
	132	+ 46 => 2,
	133	+ 55 => 4,
	134	+ 65 => 1,
	135	+ 66 => 1,
	136	+ 68 => 1,
	137	+ 69 => 1,
	138	+ 73 => 1,
	139	+ 74 => 1,
	140	+ 100 => 1,
	141	+ ];
	142	+ break;
	143	+ default:
	144	+ return [];
	145	+ break;
146	146	}//end switch
147	147
148	148	}//end getErrorList()

Please login to merge, or discard this patch.

src/Standards/Squiz/Tests/WhiteSpace/FunctionClosingBraceSpaceUnitTest.php 1 patch

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

@@ -28,31 +28,31 @@
 block discarded – undo
     public function getErrorList($testFile='FunctionClosingBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
-        case 'FunctionClosingBraceSpaceUnitTest.inc':
-            return [
-                10 => 1,
-                21 => 1,
-                28 => 1,
-                29 => 1,
-                31 => 1,
-                39 => 1,
-            ];
-            break;
-        case 'FunctionClosingBraceSpaceUnitTest.js':
-            return [
-                13  => 1,
-                25  => 1,
-                32  => 1,
-                53  => 1,
-                59  => 1,
-                67  => 1,
-                84  => 1,
-                128 => 1,
-            ];
-            break;
-        default:
-            return [];
-            break;
+        	case 'FunctionClosingBraceSpaceUnitTest.inc':
+            	return [
+                	10 => 1,
+                	21 => 1,
+                	28 => 1,
+                	29 => 1,
+                	31 => 1,
+                	39 => 1,
+            	];
+            	break;
+        	case 'FunctionClosingBraceSpaceUnitTest.js':
+            	return [
+                	13  => 1,
+                	25  => 1,
+                	32  => 1,
+                	53  => 1,
+                	59  => 1,
+                	67  => 1,
+                	84  => 1,
+                	128 => 1,
+            	];
+            	break;
+        	default:
+            	return [];
+            	break;
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

		@@ -157,8 +157,8 @@
		block discarded – undo
157	157
158	158	switch($foo) {
159	159
160		- case 'bar':
161		- break;
	160	+ case 'bar':
	161	+ break;
162	162
163	163	}
164	164

		@@ -26,14 +26,14 @@ discard block
		block discarded – undo
26	26	$case = '1';
27	27	return '1';
28	28
29		-case '2':
	29	+ case '2':
30	30	case '3':
31	31	$case = '5';
32	32	break;
33	33
34		-case '4':
35		- $case = '4';
36		-break;
	34	+ case '4':
	35	+ $case = '4';
	36	+ break;
37	37
38	38	default:
39	39	$case = null;
		@@ -49,19 +49,19 @@ discard block
		block discarded – undo
49	49
50	50	// PEAR style.
51	51	switch ($something) {
52		-case '1':
53		- $case = '1';
54		- break;
55		-case '2':
56		-case '3':
57		- $case = '5';
58		- break;
59		-case '4':
60		- $case = '4';
61		- break;
62		-default:
63		- $case = null;
64		- break;
	52	+ case '1':
	53	+ $case = '1';
	54	+ break;
	55	+ case '2':
	56	+ case '3':
	57	+ $case = '5';
	58	+ break;
	59	+ case '4':
	60	+ $case = '4';
	61	+ break;
	62	+ default:
	63	+ $case = null;
	64	+ break;
65	65	}
66	66
67	67	// Valid, but missing BREAKS.
		@@ -85,7 +85,7 @@ discard block
		block discarded – undo
85	85	Case '1' :
86	86	$case = '1';
87	87
88		-case '2':
	88	+ case '2':
89	89	case '3' :
90	90	$case = '5';
91	91
		@@ -140,21 +140,21 @@ discard block
		block discarded – undo
140	140	}
141	141
142	142	switch ($var) {
143		- case 'one':
144		- case 'two':
145		- break;
	143	+ case 'one':
	144	+ case 'two':
	145	+ break;
146	146
147		- case 'three':
148		- // Nothing to do.
149		- break;
	147	+ case 'three':
	148	+ // Nothing to do.
	149	+ break;
150	150
151		- case 'four':
152		- echo $hi;
153		- break;
	151	+ case 'four':
	152	+ echo $hi;
	153	+ break;
154	154
155		- default:
156		- // No default.
157		- break;
	155	+ default:
	156	+ // No default.
	157	+ break;
158	158	}
159	159
160	160	switch ($var) {
		@@ -170,8 +170,8 @@ discard block
		block discarded – undo
170	170	}
171	171
172	172	switch ($name) {
173		- case "1":
174		- switch ($name2) {
	173	+ case "1":
	174	+ switch ($name2) {
175	175	case "1":
176	176	return true;
177	177	break;
		@@ -183,24 +183,24 @@ discard block
		block discarded – undo
183	183	default:
184	184	// No default.
185	185	break;
186		- }
	186	+ }
187	187	break;
188	188
189		- case "2":
	189	+ case "2":
190	190	switch ($name2) {
191		- case "1":
192		- return true;
193		- break;
	191	+ case "1":
	192	+ return true;
	193	+ break;
194	194
195		- case "2":
196		- return true;
197		- break;
	195	+ case "2":
	196	+ return true;
	197	+ break;
198	198
199		- default:
200		- // No default.
201		- break;
	199	+ default:
	200	+ // No default.
	201	+ break;
202	202	}
203		- break;
	203	+ break;
204	204	}
205	205
206	206	switch ($name) {
		@@ -221,9 +221,9 @@ discard block
		block discarded – undo
221	221	}
222	222
223	223	switch ($name2) {
224		- default:
225		- // No default.
226		- break;
	224	+ default:
	225	+ // No default.
	226	+ break;
227	227	}
228	228
229	229	switch ($foo) {
		@@ -258,15 +258,15 @@ discard block
		block discarded – undo
258	258	}
259	259
260	260	switch ($something) {
261		- case '1':
262		- $case = '1';
263		- break;
	261	+ case '1':
	262	+ $case = '1';
	263	+ break;
264	264
265		- case '2':
266		- throw new Exception('message');
	265	+ case '2':
	266	+ throw new Exception('message');
267	267
268		- default:
269		- throw new Exception('message');
	268	+ default:
	269	+ throw new Exception('message');
270	270	}
271	271
272	272	switch ($something) {
		@@ -280,9 +280,9 @@ discard block
		block discarded – undo
280	280	}
281	281
282	282	switch ($foo) {
283		- case '1':
284		- return; // comment
285		- break;
	283	+ case '1':
	284	+ return; // comment
	285	+ break;
286	286
287	287	}
288	288
		@@ -305,13 +305,13 @@ discard block
		block discarded – undo
305	305	}
306	306
307	307	switch ($foo) {
308		- case 'bar':
309		- throw new \Exception(
310		- 'bar'
311		- );
312		-
313		- default:
314		- throw new \Exception(
315		- 'bar'
316		- );
	308	+ case 'bar':
	309	+ throw new \Exception(
	310	+ 'bar'
	311	+ );
	312	+
	313	+ default:
	314	+ throw new \Exception(
	315	+ 'bar'
	316	+ );
317	317	}

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

		@@ -28,68 +28,68 @@
		block discarded – undo
28	28	public function getErrorList($testFile='SuperfluousWhitespaceUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'SuperfluousWhitespaceUnitTest.1.inc':
32		- return [
33		- 2 => 1,
34		- 4 => 1,
35		- 5 => 1,
36		- 6 => 1,
37		- 7 => 1,
38		- 16 => 1,
39		- 23 => 1,
40		- 28 => 1,
41		- 33 => 1,
42		- 49 => 1,
43		- 62 => 1,
44		- 65 => 1,
45		- 73 => 1,
46		- ];
47		- break;
48		- case 'SuperfluousWhitespaceUnitTest.2.inc':
49		- return [
50		- 2 => 1,
51		- 8 => 1,
52		- ];
53		- break;
54		- case 'SuperfluousWhitespaceUnitTest.3.inc':
55		- return [
56		- 6 => 1,
57		- 10 => 1,
58		- ];
59		- break;
60		- case 'SuperfluousWhitespaceUnitTest.4.inc':
61		- case 'SuperfluousWhitespaceUnitTest.5.inc':
62		- return [
63		- 1 => 1,
64		- 4 => 1,
65		- ];
66		- break;
67		- case 'SuperfluousWhitespaceUnitTest.1.js':
68		- return [
69		- 1 => 1,
70		- 3 => 1,
71		- 4 => 1,
72		- 5 => 1,
73		- 6 => 1,
74		- 15 => 1,
75		- 22 => 1,
76		- 29 => 1,
77		- 38 => 1,
78		- 56 => 1,
79		- ];
80		- break;
81		- case 'SuperfluousWhitespaceUnitTest.1.css':
82		- return [
83		- 1 => 1,
84		- 8 => 1,
85		- 9 => 1,
86		- 11 => 1,
87		- 25 => 1,
88		- ];
89		- break;
90		- default:
91		- return [];
92		- break;
	31	+ case 'SuperfluousWhitespaceUnitTest.1.inc':
	32	+ return [
	33	+ 2 => 1,
	34	+ 4 => 1,
	35	+ 5 => 1,
	36	+ 6 => 1,
	37	+ 7 => 1,
	38	+ 16 => 1,
	39	+ 23 => 1,
	40	+ 28 => 1,
	41	+ 33 => 1,
	42	+ 49 => 1,
	43	+ 62 => 1,
	44	+ 65 => 1,
	45	+ 73 => 1,
	46	+ ];
	47	+ break;
	48	+ case 'SuperfluousWhitespaceUnitTest.2.inc':
	49	+ return [
	50	+ 2 => 1,
	51	+ 8 => 1,
	52	+ ];
	53	+ break;
	54	+ case 'SuperfluousWhitespaceUnitTest.3.inc':
	55	+ return [
	56	+ 6 => 1,
	57	+ 10 => 1,
	58	+ ];
	59	+ break;
	60	+ case 'SuperfluousWhitespaceUnitTest.4.inc':
	61	+ case 'SuperfluousWhitespaceUnitTest.5.inc':
	62	+ return [
	63	+ 1 => 1,
	64	+ 4 => 1,
	65	+ ];
	66	+ break;
	67	+ case 'SuperfluousWhitespaceUnitTest.1.js':
	68	+ return [
	69	+ 1 => 1,
	70	+ 3 => 1,
	71	+ 4 => 1,
	72	+ 5 => 1,
	73	+ 6 => 1,
	74	+ 15 => 1,
	75	+ 22 => 1,
	76	+ 29 => 1,
	77	+ 38 => 1,
	78	+ 56 => 1,
	79	+ ];
	80	+ break;
	81	+ case 'SuperfluousWhitespaceUnitTest.1.css':
	82	+ return [
	83	+ 1 => 1,
	84	+ 8 => 1,
	85	+ 9 => 1,
	86	+ 11 => 1,
	87	+ 25 => 1,
	88	+ ];
	89	+ break;
	90	+ default:
	91	+ return [];
	92	+ break;
93	93	}//end switch
94	94
95	95	}//end getErrorList()

		@@ -28,31 +28,31 @@
		block discarded – undo
28	28	public function getErrorList($testFile='FunctionClosingBraceSpaceUnitTest.inc')
29	29	{
30	30	switch ($testFile) {
31		- case 'FunctionClosingBraceSpaceUnitTest.inc':
32		- return [
33		- 10 => 1,
34		- 21 => 1,
35		- 28 => 1,
36		- 29 => 1,
37		- 31 => 1,
38		- 39 => 1,
39		- ];
40		- break;
41		- case 'FunctionClosingBraceSpaceUnitTest.js':
42		- return [
43		- 13 => 1,
44		- 25 => 1,
45		- 32 => 1,
46		- 53 => 1,
47		- 59 => 1,
48		- 67 => 1,
49		- 84 => 1,
50		- 128 => 1,
51		- ];
52		- break;
53		- default:
54		- return [];
55		- break;
	31	+ case 'FunctionClosingBraceSpaceUnitTest.inc':
	32	+ return [
	33	+ 10 => 1,
	34	+ 21 => 1,
	35	+ 28 => 1,
	36	+ 29 => 1,
	37	+ 31 => 1,
	38	+ 39 => 1,
	39	+ ];
	40	+ break;
	41	+ case 'FunctionClosingBraceSpaceUnitTest.js':
	42	+ return [
	43	+ 13 => 1,
	44	+ 25 => 1,
	45	+ 32 => 1,
	46	+ 53 => 1,
	47	+ 59 => 1,
	48	+ 67 => 1,
	49	+ 84 => 1,
	50	+ 128 => 1,
	51	+ ];
	52	+ break;
	53	+ default:
	54	+ return [];
	55	+ break;
56	56	}//end switch
57	57
58	58	}//end getErrorList()

gravityview / GravityView

Pull Request — develop (#1492)

Category

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

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

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

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

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

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

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

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

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