View GitHub Repository PHP7_CodeSniffer

Schedule Inspection
Subscribe
Completed
Push — master ( 8cd077 )
by Tomáš
09:25
created
Category
Toggle Patches
src/Standards/Squiz/Tests/CSS/SemicolonSpacingUnitTest.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/Files/FileExtensionUnitTest.php 2 patches
Switch Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -28,10 +28,10 @@
 
 block discarded – undo
28 28 
     public function getErrorList($testFile='')
29 29 
     {
30 30 
         switch ($testFile) {
31 
-        case 'FileExtensionUnitTest.1.inc':
32 
-            return array(1 => 1);
33 
-        default:
34 
-            return array();
31 
+            case 'FileExtensionUnitTest.1.inc':
32 
+                return array(1 => 1);
33 
+            default:
34 
+                return array();
35 35 
         }
36 36
37 37 
     }//end getErrorList()
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='')
43 
+    public function getErrorList($testFile = '')
44 44 
     {
45 45 
         switch ($testFile) {
46 46 
         case 'CharacterBeforePHPOpeningTagUnitTest.1.inc':
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc 3 patches
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -123,13 +123,13 @@
 
 block discarded – undo
123 123 
 }
124 124
125 125 
 $expectedPermission = array(
126 
-                       'granted'        => 4,
127 
-                       'denied'         => 1,
128 
-                       'cascade'        => TRUE,
129 
-                       'blockergranted' => 2,
130 
-                       'blockerdenied'  => - 3,
131 
-                       'effective'      => TRUE,
132 
-                      );
126 
+                        'granted'        => 4,
127 
+                        'denied'         => 1,
128 
+                        'cascade'        => TRUE,
129 
+                        'blockergranted' => 2,
130 
+                        'blockerdenied'  => - 3,
131 
+                        'effective'      => TRUE,
132 
+                        );
133 133
134 134 
 $value = (int) isset($blah) + 2;
135 135 
 $value = (int) isset($blah) + (int) isset($foo) + (int) isset($bar);
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -60,7 +60,7 @@  discard block
 
 block discarded – undo
60 60
61 61 
 $line  = substr($line, 0, -2);
62 62 
 $words = preg_split('|(\s+)|', $line."\n", -1, $flags);
63 
-if (!isset($this->words[$wordPos-1]) || $this->words[$wordPos-1] !== ' ') {
63 
+if (!isset($this->words[$wordPos - 1]) || $this->words[$wordPos - 1] !== ' ') {
64 64 
 } else if ($this->tokens[$pos + 1] === "\n") {
65 65 
 }
66 66 
 
@@ -127,7 +127,7 @@  discard block
 
 block discarded – undo
127 127 
                        'denied'         => 1,
128 128 
                        'cascade'        => TRUE,
129 129 
                        'blockergranted' => 2,
130 
-                       'blockerdenied'  => - 3,
130 
+                       'blockerdenied'  => -3,
131 131 
                        'effective'      => TRUE,
132 132 
                       );
133 133
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -105,8 +105,12 @@
 
 block discarded – undo
105 105
106 106 
 }//end class
107 107
108 
-if ($index < -1) $index = 0;
109 
-if ($index < - 1) $index = 0;
108 
+if ($index < -1) {
109 
+    $index = 0;
110 
+}
111 
+if ($index < - 1) {
112 
+    $index = 0;
113 
+}
110 114
111 115 
 $three = ceil($one / $two);
112 116 
 $three = ceil(($one / $two) / $four);
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.php 3 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -78,7 +78,7 @@  discard block
 
 block discarded – undo
78 78 
                     135 => 2,
79 79 
                     137 => 1,
80 80 
                     139 => 1,
81 
-                   );
81 
+                    );
82 82 
             break;
83 83 
         case 'OperatorBracketUnitTest.js':
84 84 
             return array(
 
@@ -95,8 +95,8 @@  discard block
 
 block discarded – undo
95 95 
                     47  => 1,
96 96 
                     63  => 1,
97 97 
                     108 => 1,
98 
-                   );
99 
-             break;
98 
+                    );
99 
+                break;
100 100 
         default:
101 101 
             return array();
102 102 
             break;
Please login to merge, or discard this patch.
Switch Indentation   +55 added lines, -55 removed lines patch added patch discarded remove patch 
@@ -45,61 +45,61 @@
 
 block discarded – undo
45 45 
     public function getErrorList($testFile='OperatorBracketUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 
-        case 'OperatorBracketUnitTest.inc':
49 
-            return array(
50 
-                    3   => 1,
51 
-                    6   => 1,
52 
-                    9   => 1,
53 
-                    12  => 1,
54 
-                    15  => 1,
55 
-                    18  => 2,
56 
-                    20  => 1,
57 
-                    25  => 1,
58 
-                    28  => 1,
59 
-                    31  => 1,
60 
-                    34  => 1,
61 
-                    37  => 1,
62 
-                    40  => 1,
63 
-                    43  => 2,
64 
-                    45  => 1,
65 
-                    47  => 5,
66 
-                    48  => 1,
67 
-                    50  => 2,
68 
-                    55  => 2,
69 
-                    56  => 1,
70 
-                    63  => 2,
71 
-                    64  => 1,
72 
-                    67  => 1,
73 
-                    86  => 1,
74 
-                    90  => 1,
75 
-                    109 => 1,
76 
-                    130 => 1,
77 
-                    134 => 1,
78 
-                    135 => 2,
79 
-                    137 => 1,
80 
-                    139 => 1,
81 
-                   );
82 
-            break;
83 
-        case 'OperatorBracketUnitTest.js':
84 
-            return array(
85 
-                    5   => 1,
86 
-                    8   => 1,
87 
-                    11  => 1,
88 
-                    14  => 1,
89 
-                    24  => 1,
90 
-                    30  => 1,
91 
-                    33  => 1,
92 
-                    36  => 1,
93 
-                    39  => 1,
94 
-                    46  => 1,
95 
-                    47  => 1,
96 
-                    63  => 1,
97 
-                    108 => 1,
98 
-                   );
99 
-             break;
100 
-        default:
101 
-            return array();
102 
-            break;
48 
+            case 'OperatorBracketUnitTest.inc':
49 
+                return array(
50 
+                        3   => 1,
51 
+                        6   => 1,
52 
+                        9   => 1,
53 
+                        12  => 1,
54 
+                        15  => 1,
55 
+                        18  => 2,
56 
+                        20  => 1,
57 
+                        25  => 1,
58 
+                        28  => 1,
59 
+                        31  => 1,
60 
+                        34  => 1,
61 
+                        37  => 1,
62 
+                        40  => 1,
63 
+                        43  => 2,
64 
+                        45  => 1,
65 
+                        47  => 5,
66 
+                        48  => 1,
67 
+                        50  => 2,
68 
+                        55  => 2,
69 
+                        56  => 1,
70 
+                        63  => 2,
71 
+                        64  => 1,
72 
+                        67  => 1,
73 
+                        86  => 1,
74 
+                        90  => 1,
75 
+                        109 => 1,
76 
+                        130 => 1,
77 
+                        134 => 1,
78 
+                        135 => 2,
79 
+                        137 => 1,
80 
+                        139 => 1,
81 
+                       );
82 
+                break;
83 
+            case 'OperatorBracketUnitTest.js':
84 
+                return array(
85 
+                        5   => 1,
86 
+                        8   => 1,
87 
+                        11  => 1,
88 
+                        14  => 1,
89 
+                        24  => 1,
90 
+                        30  => 1,
91 
+                        33  => 1,
92 
+                        36  => 1,
93 
+                        39  => 1,
94 
+                        46  => 1,
95 
+                        47  => 1,
96 
+                        63  => 1,
97 
+                        108 => 1,
98 
+                       );
99 
+                 break;
100 
+            default:
101 
+                return array();
102 
+                break;
103 103 
         }//end switch
104 104
105 105 
     }//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='OperatorBracketUnitTest.inc')
45 
+    public function getErrorList($testFile = 'OperatorBracketUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'OperatorBracketUnitTest.inc':
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Scope/StaticThisUsageUnitTest.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/ScopeClosingBraceUnitTest.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/FunctionClosingBraceSpaceUnitTest.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   +25 added lines, -25 removed lines patch added patch discarded remove patch 
@@ -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()
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='FunctionClosingBraceSpaceUnitTest.inc')
45 
+    public function getErrorList($testFile = 'FunctionClosingBraceSpaceUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'FunctionClosingBraceSpaceUnitTest.inc':
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/ControlStructureSpacingUnitTest.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   +44 added lines, -44 removed lines patch added patch discarded remove patch 
@@ -45,50 +45,50 @@
 
 block discarded – undo
45 45 
     public function getErrorList($testFile='ControlStructureSpacingUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 
-        case 'ControlStructureSpacingUnitTest.inc':
49 
-            return array(
50 
-                    3   => 1,
51 
-                    5   => 1,
52 
-                    8   => 1,
53 
-                    15  => 1,
54 
-                    23  => 1,
55 
-                    74  => 1,
56 
-                    79  => 1,
57 
-                    82  => 1,
58 
-                    83  => 1,
59 
-                    87  => 1,
60 
-                    103 => 1,
61 
-                    113 => 2,
62 
-                    114 => 2,
63 
-                    118 => 1,
64 
-                    150 => 1,
65 
-                    153 => 1,
66 
-                    154 => 1,
67 
-                    157 => 1,
68 
-                    170 => 1,
69 
-                    176 => 2,
70 
-                    179 => 1,
71 
-                    189 => 1,
72 
-                   );
73 
-            break;
74 
-        case 'ControlStructureSpacingUnitTest.js':
75 
-            return array(
76 
-                    3  => 1,
77 
-                    9  => 1,
78 
-                    15 => 1,
79 
-                    21 => 1,
80 
-                    56 => 1,
81 
-                    61 => 1,
82 
-                    64 => 1,
83 
-                    65 => 1,
84 
-                    68 => 1,
85 
-                    74 => 2,
86 
-                    75 => 2,
87 
-                   );
88 
-            break;
89 
-        default:
90 
-            return array();
91 
-            break;
48 
+            case 'ControlStructureSpacingUnitTest.inc':
49 
+                return array(
50 
+                        3   => 1,
51 
+                        5   => 1,
52 
+                        8   => 1,
53 
+                        15  => 1,
54 
+                        23  => 1,
55 
+                        74  => 1,
56 
+                        79  => 1,
57 
+                        82  => 1,
58 
+                        83  => 1,
59 
+                        87  => 1,
60 
+                        103 => 1,
61 
+                        113 => 2,
62 
+                        114 => 2,
63 
+                        118 => 1,
64 
+                        150 => 1,
65 
+                        153 => 1,
66 
+                        154 => 1,
67 
+                        157 => 1,
68 
+                        170 => 1,
69 
+                        176 => 2,
70 
+                        179 => 1,
71 
+                        189 => 1,
72 
+                       );
73 
+                break;
74 
+            case 'ControlStructureSpacingUnitTest.js':
75 
+                return array(
76 
+                        3  => 1,
77 
+                        9  => 1,
78 
+                        15 => 1,
79 
+                        21 => 1,
80 
+                        56 => 1,
81 
+                        61 => 1,
82 
+                        64 => 1,
83 
+                        65 => 1,
84 
+                        68 => 1,
85 
+                        74 => 2,
86 
+                        75 => 2,
87 
+                       );
88 
+                break;
89 
+            default:
90 
+                return array();
91 
+                break;
92 92 
         }//end switch
93 93
94 94 
     }//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='ControlStructureSpacingUnitTest.inc')
45 
+    public function getErrorList($testFile = 'ControlStructureSpacingUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'ControlStructureSpacingUnitTest.inc':
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/WhiteSpace/CastSpacingUnitTest.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.