Inspection of "Changelog for PR #1032" - Symplify/PHP7_CodeSniffer - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (d87ed9)

by Tomáš

created 2016-06-24 23:04 UTC

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

@@ -53,7 +53,7 @@  discard block
 block discarded – undo
                     29 => 1,
                     31 => 1,
                     39 => 1,
-                   );
+                    );
             break;
         case 'FunctionClosingBraceSpaceUnitTest.js':
             return array(
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
                     67  => 1,
                     84  => 1,
                     128 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Please login to merge, or discard this patch.

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

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

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -42,7 +42,7 @@
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='FunctionClosingBraceSpaceUnitTest.inc')
+    public function getErrorList($testFile = 'FunctionClosingBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
         case 'FunctionClosingBraceSpaceUnitTest.inc':

Please login to merge, or discard this patch.

Standards/Squiz/Tests/WhiteSpace/SuperfluousWhitespaceUnitTest.php 3 patches

Indentation +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -57,7 +57,7 @@  discard block
 block discarded – undo
                     28 => 1,
                     33 => 1,
                     53 => 1,
-                   );
+                    );
             break;
         case 'SuperfluousWhitespaceUnitTest.1.js':
             return array(
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
                     29 => 1,
                     38 => 1,
                     56 => 1,
-                   );
+                    );
             break;
         case 'SuperfluousWhitespaceUnitTest.1.css':
             return array(
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
                     9  => 1,
                     11 => 1,
                     25 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Please login to merge, or discard this patch.

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

@@ -45,46 +45,46 @@
 block discarded – undo
     public function getErrorList($testFile='SuperfluousWhitespaceUnitTest.inc')
     {
         switch ($testFile) {
-        case 'SuperfluousWhitespaceUnitTest.inc':
-            return array(
-                    2  => 1,
-                    4  => 1,
-                    5  => 1,
-                    6  => 1,
-                    7  => 1,
-                    16 => 1,
-                    23 => 1,
-                    28 => 1,
-                    33 => 1,
-                    53 => 1,
-                   );
-            break;
-        case 'SuperfluousWhitespaceUnitTest.1.js':
-            return array(
-                    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 array(
-                    1  => 1,
-                    8  => 1,
-                    9  => 1,
-                    11 => 1,
-                    25 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'SuperfluousWhitespaceUnitTest.inc':
+                return array(
+                        2  => 1,
+                        4  => 1,
+                        5  => 1,
+                        6  => 1,
+                        7  => 1,
+                        16 => 1,
+                        23 => 1,
+                        28 => 1,
+                        33 => 1,
+                        53 => 1,
+                       );
+                break;
+            case 'SuperfluousWhitespaceUnitTest.1.js':
+                return array(
+                        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 array(
+                        1  => 1,
+                        8  => 1,
+                        9  => 1,
+                        11 => 1,
+                        25 => 1,
+                       );
+                break;
+            default:
+                return array();
+                break;
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -42,7 +42,7 @@
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='SuperfluousWhitespaceUnitTest.inc')
+    public function getErrorList($testFile = 'SuperfluousWhitespaceUnitTest.inc')
     {
         switch ($testFile) {
         case 'SuperfluousWhitespaceUnitTest.inc':

Please login to merge, or discard this patch.

Standards/Squiz/Tests/WhiteSpace/PropertyLabelSpacingUnitTest.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

		@@ -49,7 +49,7 @@
		block discarded – undo
49	49	10 => 1,
50	50	12 => 1,
51	51	18 => 1,
52		- );
	52	+ );
53	53
54	54	}//end getErrorList()
55	55

Please login to merge, or discard this patch.

Standards/Squiz/Tests/WhiteSpace/FunctionOpeningBraceSpaceUnitTest.php 3 patches

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

@@ -49,7 +49,7 @@  discard block
 block discarded – undo
             return array(
                     10 => 1,
                     25 => 1,
-                   );
+                    );
             break;
         case 'FunctionOpeningBraceSpaceUnitTest.js':
             return array(
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
                     31 => 1,
                     38 => 1,
                     88 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Please login to merge, or discard this patch.

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

@@ -45,23 +45,23 @@
 block discarded – undo
     public function getErrorList($testFile='FunctionOpeningBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
-        case 'FunctionOpeningBraceSpaceUnitTest.inc':
-            return array(
-                    10 => 1,
-                    25 => 1,
-                   );
-            break;
-        case 'FunctionOpeningBraceSpaceUnitTest.js':
-            return array(
-                    11 => 1,
-                    31 => 1,
-                    38 => 1,
-                    88 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'FunctionOpeningBraceSpaceUnitTest.inc':
+                return array(
+                        10 => 1,
+                        25 => 1,
+                       );
+                break;
+            case 'FunctionOpeningBraceSpaceUnitTest.js':
+                return array(
+                        11 => 1,
+                        31 => 1,
+                        38 => 1,
+                        88 => 1,
+                       );
+                break;
+            default:
+                return array();
+                break;
         }//end switch
         
 

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -42,7 +42,7 @@
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='FunctionOpeningBraceSpaceUnitTest.inc')
+    public function getErrorList($testFile = 'FunctionOpeningBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
         case 'FunctionOpeningBraceSpaceUnitTest.inc':

Please login to merge, or discard this patch.

Standards/Squiz/Tests/WhiteSpace/LanguageConstructSpacingUnitTest.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

		@@ -58,7 +58,7 @@
		block discarded – undo
58	58	27 => 1,
59	59	31 => 1,
60	60	32 => 1,
61		- );
	61	+ );
62	62
63	63	}//end getErrorList()
64	64

Please login to merge, or discard this patch.

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/ScopeClosingBraceUnitTest.inc 2 patches

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

@@ -8,7 +8,7 @@  discard block
 block discarded – undo
 
     function test1()
     {
-     }
+        }
 
     function test2() {}
 
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
 
 class Test2
 {
- }
+    }
 
 
 public function test2()

Please login to merge, or discard this patch.

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -33,8 +33,8 @@  discard block
 block discarded – undo
     if (!class_exists($class_name)) {
         echo $error;
     }
-    $this->{$property} =& new $class_name($this->db_index);
-    $this->modules[$module] =& $this->{$property};
+    $this->{$property} = & new $class_name($this->db_index);
+    $this->modules[$module] = & $this->{$property};
 }
 
 foreach ($elements as $element) {
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
 
 ?>
 
-<?php while($row = $data->getRow()): ?>
+<?php while ($row = $data->getRow()): ?>
     <p><?= $val ?></p>
 <?php endwhile; ?>
 

Please login to merge, or discard this patch.

Standards/Squiz/Tests/WhiteSpace/LogicalOperatorSpacingUnitTest.inc 1 patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -1,9 +1,9 @@  discard block
 block discarded – undo
 <?php
 
 if ($foo || $bar) {}
-if ($foo||$bar && $baz) {}
-if ($foo|| $bar&&$baz) {}
-if ($foo  ||   $bar   && $baz) {}
+if ($foo || $bar && $baz) {}
+if ($foo || $bar && $baz) {}
+if ($foo || $bar && $baz) {}
 
 // Spacing after || below is ignored as it is EOL whitespace.
 if ($foo ||    
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
 ) {
 }
 
-if ($foo||
+if ($foo ||
     $bar
     &&$baz
 ) {

Please login to merge, or discard this patch.

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/OperatorSpacingUnitTest.php 3 patches

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

@@ -103,7 +103,7 @@  discard block
 block discarded – undo
                     177 => 1,
                     178 => 1,
                     184 => 2,
-                   );
+                    );
             break;
         case 'OperatorSpacingUnitTest.js':
             return array(
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
                     69 => 1,
                     73 => 1,
                     74 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Please login to merge, or discard this patch.

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

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

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -42,7 +42,7 @@
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='OperatorSpacingUnitTest.inc')
+    public function getErrorList($testFile = 'OperatorSpacingUnitTest.inc')
     {
         switch ($testFile) {
         case 'OperatorSpacingUnitTest.inc':

Please login to merge, or discard this patch.

Standards/Squiz/Tests/WhiteSpace/ScopeKeywordSpacingUnitTest.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

		@@ -49,7 +49,7 @@
		block discarded – undo
49	49	14 => 1,
50	50	15 => 1,
51	51	17 => 2,
52		- );
	52	+ );
53	53
54	54	}//end getErrorList()
55	55

Please login to merge, or discard this patch.

		@@ -53,7 +53,7 @@ discard block
		block discarded – undo
53	53	29 => 1,
54	54	31 => 1,
55	55	39 => 1,
56		- );
	56	+ );
57	57	break;
58	58	case 'FunctionClosingBraceSpaceUnitTest.js':
59	59	return array(
		@@ -65,7 +65,7 @@ discard block
		block discarded – undo
65	65	67 => 1,
66	66	84 => 1,
67	67	128 => 1,
68		- );
	68	+ );
69	69	break;
70	70	default:
71	71	return array();

		@@ -45,31 +45,31 @@
		block discarded – undo
45	45	public function getErrorList($testFile='FunctionClosingBraceSpaceUnitTest.inc')
46	46	{
47	47	switch ($testFile) {
48		- case 'FunctionClosingBraceSpaceUnitTest.inc':
49		- return array(
50		- 10 => 1,
51		- 21 => 1,
52		- 28 => 1,
53		- 29 => 1,
54		- 31 => 1,
55		- 39 => 1,
56		- );
57		- break;
58		- case 'FunctionClosingBraceSpaceUnitTest.js':
59		- return array(
60		- 13 => 1,
61		- 25 => 1,
62		- 32 => 1,
63		- 53 => 1,
64		- 59 => 1,
65		- 67 => 1,
66		- 84 => 1,
67		- 128 => 1,
68		- );
69		- break;
70		- default:
71		- return array();
72		- break;
	48	+ case 'FunctionClosingBraceSpaceUnitTest.inc':
	49	+ return array(
	50	+ 10 => 1,
	51	+ 21 => 1,
	52	+ 28 => 1,
	53	+ 29 => 1,
	54	+ 31 => 1,
	55	+ 39 => 1,
	56	+ );
	57	+ break;
	58	+ case 'FunctionClosingBraceSpaceUnitTest.js':
	59	+ return array(
	60	+ 13 => 1,
	61	+ 25 => 1,
	62	+ 32 => 1,
	63	+ 53 => 1,
	64	+ 59 => 1,
	65	+ 67 => 1,
	66	+ 84 => 1,
	67	+ 128 => 1,
	68	+ );
	69	+ break;
	70	+ default:
	71	+ return array();
	72	+ break;
73	73	}//end switch
74	74
75	75	}//end getErrorList()

		@@ -42,7 +42,7 @@
		block discarded – undo
42	42	*
43	43	* @return array<int, int>
44	44	*/
45		- public function getErrorList($testFile='FunctionClosingBraceSpaceUnitTest.inc')
	45	+ public function getErrorList($testFile = 'FunctionClosingBraceSpaceUnitTest.inc')
46	46	{
47	47	switch ($testFile) {
48	48	case 'FunctionClosingBraceSpaceUnitTest.inc':

		@@ -57,7 +57,7 @@ discard block
		block discarded – undo
57	57	28 => 1,
58	58	33 => 1,
59	59	53 => 1,
60		- );
	60	+ );
61	61	break;
62	62	case 'SuperfluousWhitespaceUnitTest.1.js':
63	63	return array(
		@@ -71,7 +71,7 @@ discard block
		block discarded – undo
71	71	29 => 1,
72	72	38 => 1,
73	73	56 => 1,
74		- );
	74	+ );
75	75	break;
76	76	case 'SuperfluousWhitespaceUnitTest.1.css':
77	77	return array(
		@@ -80,7 +80,7 @@ discard block
		block discarded – undo
80	80	9 => 1,
81	81	11 => 1,
82	82	25 => 1,
83		- );
	83	+ );
84	84	break;
85	85	default:
86	86	return array();

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

		@@ -49,7 +49,7 @@ discard block
		block discarded – undo
49	49	return array(
50	50	10 => 1,
51	51	25 => 1,
52		- );
	52	+ );
53	53	break;
54	54	case 'FunctionOpeningBraceSpaceUnitTest.js':
55	55	return array(
		@@ -57,7 +57,7 @@ discard block
		block discarded – undo
57	57	31 => 1,
58	58	38 => 1,
59	59	88 => 1,
60		- );
	60	+ );
61	61	break;
62	62	default:
63	63	return array();

		@@ -45,23 +45,23 @@
		block discarded – undo
45	45	public function getErrorList($testFile='FunctionOpeningBraceSpaceUnitTest.inc')
46	46	{
47	47	switch ($testFile) {
48		- case 'FunctionOpeningBraceSpaceUnitTest.inc':
49		- return array(
50		- 10 => 1,
51		- 25 => 1,
52		- );
53		- break;
54		- case 'FunctionOpeningBraceSpaceUnitTest.js':
55		- return array(
56		- 11 => 1,
57		- 31 => 1,
58		- 38 => 1,
59		- 88 => 1,
60		- );
61		- break;
62		- default:
63		- return array();
64		- break;
	48	+ case 'FunctionOpeningBraceSpaceUnitTest.inc':
	49	+ return array(
	50	+ 10 => 1,
	51	+ 25 => 1,
	52	+ );
	53	+ break;
	54	+ case 'FunctionOpeningBraceSpaceUnitTest.js':
	55	+ return array(
	56	+ 11 => 1,
	57	+ 31 => 1,
	58	+ 38 => 1,
	59	+ 88 => 1,
	60	+ );
	61	+ break;
	62	+ default:
	63	+ return array();
	64	+ break;
65	65	}//end switch
66	66
67	67

		@@ -8,7 +8,7 @@ discard block
		block discarded – undo
8	8
9	9	function test1()
10	10	{
11		- }
	11	+ }
12	12
13	13	function test2() {}
14	14
		@@ -21,7 +21,7 @@ discard block
		block discarded – undo
21	21
22	22	class Test2
23	23	{
24		- }
	24	+ }
25	25
26	26
27	27	public function test2()

		@@ -33,8 +33,8 @@ discard block
		block discarded – undo
33	33	if (!class_exists($class_name)) {
34	34	echo $error;
35	35	}
36		- $this->{$property} =& new $class_name($this->db_index);
37		- $this->modules[$module] =& $this->{$property};
	36	+ $this->{$property} = & new $class_name($this->db_index);
	37	+ $this->modules[$module] = & $this->{$property};
38	38	}
39	39
40	40	foreach ($elements as $element) {
		@@ -62,7 +62,7 @@ discard block
		block discarded – undo
62	62
63	63	?>
64	64
65		-<?php while($row = $data->getRow()): ?>
	65	+<?php while ($row = $data->getRow()): ?>
66	66	<p><?= $val ?></p>
67	67	<?php endwhile; ?>
68	68

		@@ -1,9 +1,9 @@ discard block
		block discarded – undo
1	1	<?php
2	2
3	3	if ($foo \|\| $bar) {}
4		-if ($foo\|\|$bar && $baz) {}
5		-if ($foo\|\| $bar&&$baz) {}
6		-if ($foo \|\| $bar && $baz) {}
	4	+if ($foo \|\| $bar && $baz) {}
	5	+if ($foo \|\| $bar && $baz) {}
	6	+if ($foo \|\| $bar && $baz) {}
7	7
8	8	// Spacing after \|\| below is ignored as it is EOL whitespace.
9	9	if ($foo \|\|
		@@ -12,7 +12,7 @@ discard block
		block discarded – undo
12	12	) {
13	13	}
14	14
15		-if ($foo\|\|
	15	+if ($foo \|\|
16	16	$bar
17	17	&&$baz
18	18	) {

		@@ -103,7 +103,7 @@ discard block
		block discarded – undo
103	103	177 => 1,
104	104	178 => 1,
105	105	184 => 2,
106		- );
	106	+ );
107	107	break;
108	108	case 'OperatorSpacingUnitTest.js':
109	109	return array(
		@@ -144,7 +144,7 @@ discard block
		block discarded – undo
144	144	69 => 1,
145	145	73 => 1,
146	146	74 => 1,
147		- );
	147	+ );
148	148	break;
149	149	default:
150	150	return array();

Symplify / PHP7_CodeSniffer

Branch — master (d87ed9)

Category

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

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

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +3 added lines, -3 removed lines patch added patch discarded remove patch

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

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

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

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

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

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

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

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

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

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch