View GitHub Repository PHP7_CodeSniffer

Schedule Inspection
Subscribe
Completed
Push — master ( 8cd077 )
by Tomáš
09:25
created
Category
Toggle Patches
src/Standards/Squiz/Tests/WhiteSpace/FunctionSpacingUnitTest.inc 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -259,13 +259,13 @@
 
 block discarded – undo
259 259 
 // Closures should be ignored.
260 260 
 preg_replace_callback(
261 261 
     '~-([a-z])~',
262 
-    function ($match) {
262 
+    function($match) {
263 263 
         return strtoupper($match[1]);
264 264 
     },
265 265 
     'hello-world'
266 266 
 );
267 267
268 
-$callback = function ($bar) use ($foo)
268 
+$callback = function($bar) use ($foo)
269 269 
             {
270 270 
                 $bar += $foo;
271 271 
             };
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/OperatorSpacingUnitTest.php 3 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -47,13 +47,13 @@
 
 block discarded – undo
47 47 
             return array(
48 48 
                     4 => 1,
49 49 
                     5 => 1,
50 
-                   );
50 
+                    );
51 51 
         } else if (version_compare(PHP_VERSION, '5.4.0RC1') >= 0) {
52 52 
             // Shorthand echo is always available from PHP 5.4.0 but needed the
53 53 
             // short_open_tag ini var to be set for versions before this.
54 54 
             return array(
55 55 
                     4 => 1,
56 
-                   );
56 
+                    );
57 57 
         }
58 58
59 59 
         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
42 42 
      *
43 43 
      * @return array<int, int>
44 44 
      */
45 
-    public function getErrorList($testFile='OperatorSpacingUnitTest.inc')
45 
+    public function getErrorList($testFile = 'OperatorSpacingUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'OperatorSpacingUnitTest.inc':
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/LanguageConstructSpacingUnitTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -57,7 +57,7 @@
 
 block discarded – undo
57 57 
                 48 => 1,
58 58 
                 70 => 1,
59 59 
                 71 => 1,
60 
-               );
60 
+                );
61 61
62 62 
     }//end getErrorList()
63 63
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/ObjectOperatorSpacingUnitTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -57,7 +57,7 @@
 
 block discarded – undo
57 57 
                 48 => 1,
58 58 
                 70 => 1,
59 59 
                 71 => 1,
60 
-               );
60 
+                );
61 61
62 62 
     }//end getErrorList()
63 63
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/SemicolonSpacingUnitTest.inc 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -1,12 +1,12 @@
 
 block discarded – undo
1 1 
 <?php
2 2 
 $test = $this->testThis();
3 
-$test = $this->testThis() ;
4 
-$test = $this->testThis()           ;
5 
-for ($var = 1   ; $var < 10  ; $var++) {
6 
-    echo $var ;
3 
+$test = $this->testThis();
4 
+$test = $this->testThis();
5 
+for ($var = 1; $var < 10; $var++) {
6 
+    echo $var;
7 7 
 }
8 8 
 $test = $this->testThis()  /* comment here */;
9 
-$test = $this->testThis()  /* comment here */  ;
9 
+$test = $this->testThis()  /* comment here */;
10 10
11 
-$hello ='foo';
11 
+$hello = 'foo';
12 12 
 ;
13 13 
\ No newline at end of file
Please login to merge, or discard this patch.
src/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
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 
 ) {
Please login to merge, or discard this patch.
src/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
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()
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
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
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/FunctionOpeningBraceSpaceUnitTest.php 3 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -47,13 +47,13 @@
 
 block discarded – undo
47 47 
             return array(
48 48 
                     4 => 1,
49 49 
                     5 => 1,
50 
-                   );
50 
+                    );
51 51 
         } else if (version_compare(PHP_VERSION, '5.4.0RC1') >= 0) {
52 52 
             // Shorthand echo is always available from PHP 5.4.0 but needed the
53 53 
             // short_open_tag ini var to be set for versions before this.
54 54 
             return array(
55 55 
                     4 => 1,
56 
-                   );
56 
+                    );
57 57 
         }
58 58
59 59 
         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
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
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
42 42 
      *
43 43 
      * @return array<int, int>
44 44 
      */
45 
-    public function getErrorList($testFile='FunctionOpeningBraceSpaceUnitTest.inc')
45 
+    public function getErrorList($testFile = 'FunctionOpeningBraceSpaceUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'FunctionOpeningBraceSpaceUnitTest.inc':
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/LogicalOperatorSpacingUnitTest.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -57,7 +57,7 @@
 
 block discarded – undo
57 57 
                 48 => 1,
58 58 
                 70 => 1,
59 59 
                 71 => 1,
60 
-               );
60 
+                );
61 61
62 62 
     }//end getErrorList()
63 63
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -40,7 +40,7 @@
 
 block discarded – undo
40 40 
      *
41 41 
      * @return array<int, int>
42 42 
      */
43 
-    public function getErrorList($testFile='LogicalOperatorSpacingUnitTest.inc')
43 
+    public function getErrorList($testFile = 'LogicalOperatorSpacingUnitTest.inc')
44 44 
     {
45 45 
         return array(
46 46 
                 4  => 2,
Please login to merge, or discard this patch.