View GitHub Repository PHP7_CodeSniffer

Schedule Inspection
Subscribe
Completed
Push — master ( 8cd077...c823b8 )
by Tomáš
14:59 queued 07:26
created
Status
Category
Toggle Patches
Standards/Generic/Tests/NamingConventions/UpperCaseConstantNameUnitTest.inc 2 patches
Switch Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -118,8 +118,8 @@
 
 block discarded – undo
118 118
119 119 
 switch($foo)
120 120 
 {
121 
-    case $bar:
122 
-    gototarget3:
121 
+        case $bar:
122 
+        gototarget3:
123 123 
 }
124 124
125 125 
 $a = 2 * ${x} - ${minus};
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -44,7 +44,7 @@  discard block
 
 block discarded – undo
44 44 
         print 'HELLO';
45 45 
     }
46 46
47 
-    function myFunc(PHP_CodeSniffer &$blah) {}
47 
+    function myFunc(PHP_CodeSniffer & $blah) {}
48 48 
     function myFunc(PHP_CodeSniffer $blah) {}
49 49
50 50 
 }
 
@@ -116,13 +116,13 @@  discard block
 
 block discarded – undo
116 116 
     gototarget2:
117 117 
 }
118 118
119 
-switch($foo)
119 
+switch ($foo)
120 120 
 {
121 121 
     case $bar:
122 122 
     gototarget3:
123 123 
 }
124 124
125 
-$a = 2 * ${x} - ${minus};
125 
+$a = 2 * ${x} -${minus};
126 126
127 127 
 class Object implements IObject
128 128 
 {
Please login to merge, or discard this patch.
Standards/Generic/Tests/NamingConventions/UpperCaseConstantNameUnitTest.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/Generic/Tests/NamingConventions/ConstructorNameUnitTest.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/Generic/Tests/PHP/ForbiddenFunctionsUnitTest.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -43,10 +43,10 @@
 
 block discarded – undo
43 43 
     public function getErrorList()
44 44 
     {
45 45 
         $errors = array(
46 
-                   2 => 1,
47 
-                   4 => 1,
48 
-                   6 => 1,
49 
-                  );
46 
+                    2 => 1,
47 
+                    4 => 1,
48 
+                    6 => 1,
49 
+                    );
50 50
51 51 
         return $errors;
52 52
Please login to merge, or discard this patch.
src/Standards/Generic/Tests/PHP/LowerCaseConstantUnitTest.inc 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -1,13 +1,13 @@  discard block
 
 block discarded – undo
1 1 
 <?php
2 2
3 3 
 // True
4 
-function myFunction($arg1, $arg2=true)
4 
+function myFunction($arg1, $arg2 = true)
5 5 
 {
6 6 
 }
7 
-function myFunction($arg1, $arg2=TRUE)
7 
+function myFunction($arg1, $arg2 = TRUE)
8 8 
 {
9 9 
 }
10 
-function myFunction($arg1, $arg2=True)
10 
+function myFunction($arg1, $arg2 = True)
11 11 
 {
12 12 
 }
13 13 
 
@@ -17,13 +17,13 @@  discard block
 
 block discarded – undo
17 17
18 18
19 19 
 // False
20 
-function myFunction($arg1, $arg2=false)
20 
+function myFunction($arg1, $arg2 = false)
21 21 
 {
22 22 
 }
23 
-function myFunction($arg1, $arg2=FALSE)
23 
+function myFunction($arg1, $arg2 = FALSE)
24 24 
 {
25 25 
 }
26 
-function myFunction($arg1, $arg2=False)
26 
+function myFunction($arg1, $arg2 = False)
27 27 
 {
28 28 
 }
29 29 
 
@@ -33,13 +33,13 @@  discard block
 
 block discarded – undo
33 33
34 34
35 35 
 // Null
36 
-function myFunction($arg1, $arg2=null)
36 
+function myFunction($arg1, $arg2 = null)
37 37 
 {
38 38 
 }
39 
-function myFunction($arg1, $arg2=NULL)
39 
+function myFunction($arg1, $arg2 = NULL)
40 40 
 {
41 41 
 }
42 
-function myFunction($arg1, $arg2=Null)
42 
+function myFunction($arg1, $arg2 = Null)
43 43 
 {
44 44 
 }
45 45 
 
@@ -67,8 +67,8 @@  discard block
 
 block discarded – undo
67 67 
     }
68 68 
 }
69 69
70 
-$x = $f?FALSE:true;
71 
-$x = $f? FALSE:true;
70 
+$x = $f ? FALSE : true;
71 
+$x = $f ? FALSE : true;
72 72
73 73 
 class MyClass
74 74 
 {
Please login to merge, or discard this patch.
src/Standards/Generic/Tests/PHP/CharacterBeforePHPOpeningTagUnitTest.php 2 patches
Switch Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -28,12 +28,12 @@
 
 block discarded – undo
28 28 
     public function getErrorList($testFile='')
29 29 
     {
30 30 
         switch ($testFile) {
31 
-        case 'CharacterBeforePHPOpeningTagUnitTest.1.inc':
32 
-            return array(2 => 1);
33 
-            break;
34 
-        default:
35 
-            return array();
36 
-            break;
31 
+            case 'CharacterBeforePHPOpeningTagUnitTest.1.inc':
32 
+                return array(2 => 1);
33 
+                break;
34 
+            default:
35 
+                return array();
36 
+                break;
37 37 
         }//end switch
38 38
39 39 
     }//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/Generic/Tests/PHP/LowerCaseKeywordUnitTest.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/Generic/Tests/PHP/LowerCaseConstantUnitTest.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   +32 added lines, -32 removed lines patch added patch discarded remove patch 
@@ -45,39 +45,39 @@
 
 block discarded – undo
45 45 
     public function getErrorList($testFile='LowerCaseConstantUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 
-        case 'LowerCaseConstantUnitTest.inc':
49 
-            return array(
50 
-                    7  => 1,
51 
-                    10 => 1,
52 
-                    15 => 1,
53 
-                    16 => 1,
54 
-                    23 => 1,
55 
-                    26 => 1,
56 
-                    31 => 1,
57 
-                    32 => 1,
58 
-                    39 => 1,
59 
-                    42 => 1,
60 
-                    47 => 1,
61 
-                    48 => 1,
62 
-                    70 => 1,
63 
-                    71 => 1,
64 
-                   );
65 
-        break;
66 
-        case 'LowerCaseConstantUnitTest.js':
67 
-            return array(
68 
-                    2  => 1,
69 
-                    3  => 1,
70 
-                    4  => 1,
71 
-                    7  => 1,
72 
-                    8  => 1,
73 
-                    12 => 1,
74 
-                    13 => 1,
75 
-                    14 => 1,
76 
-                   );
77 
-            break;
78 
-        default:
79 
-            return array();
48 
+            case 'LowerCaseConstantUnitTest.inc':
49 
+                return array(
50 
+                        7  => 1,
51 
+                        10 => 1,
52 
+                        15 => 1,
53 
+                        16 => 1,
54 
+                        23 => 1,
55 
+                        26 => 1,
56 
+                        31 => 1,
57 
+                        32 => 1,
58 
+                        39 => 1,
59 
+                        42 => 1,
60 
+                        47 => 1,
61 
+                        48 => 1,
62 
+                        70 => 1,
63 
+                        71 => 1,
64 
+                       );
80 65 
             break;
66 
+            case 'LowerCaseConstantUnitTest.js':
67 
+                return array(
68 
+                        2  => 1,
69 
+                        3  => 1,
70 
+                        4  => 1,
71 
+                        7  => 1,
72 
+                        8  => 1,
73 
+                        12 => 1,
74 
+                        13 => 1,
75 
+                        14 => 1,
76 
+                       );
77 
+                break;
78 
+            default:
79 
+                return array();
80 
+                break;
81 81 
         }
82 82
83 83 
     }//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='LowerCaseConstantUnitTest.inc')
45 
+    public function getErrorList($testFile = 'LowerCaseConstantUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'LowerCaseConstantUnitTest.inc':
Please login to merge, or discard this patch.
src/Standards/Generic/Tests/PHP/LowerCaseKeywordUnitTest.inc 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -13,7 +13,7 @@
 
 block discarded – undo
13 13 
 For ($var As $var) { Exit; }
14 14 
 If ($a AND $b OR $c XOR $d) { Die; } ElseIf { } Else
15 15 
 GOTO a;
16 
-function (Array $a) {}
16 
+function(Array $a) {}
17 17 
 const PRIVATE;
18 18 
 HttpStatus::CONTINUE;
19 19 
 function
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -4,15 +4,17 @@
 
 block discarded – undo
4 4 
 abstract public private protected function whatever() {}
5 5 
 const array()
6 6 
 for ($var as $var) { exit; }
7 
-if ($a and $b or $c xor $d) { die; } elseif { } else
8 
-goto a;
7 
+if ($a and $b or $c xor $d) { die; } elseif { } else {
8 
+    goto a;
9 
+}
9 10
10 11 
 Class Something EXTENDS SomethingElse implementS Whatever
11 12 
 Abstract Public Private Protected function whatever() {}
12 13 
 CONST array()
13 14 
 For ($var As $var) { Exit; }
14 
-If ($a AND $b OR $c XOR $d) { Die; } ElseIf { } Else
15 
-GOTO a;
15 
+If ($a AND $b OR $c XOR $d) { Die; } ElseIf { } Else {
16 
+    GOTO a;
17 
+}
16 18 
 function (Array $a) {}
17 19 
 const PRIVATE;
18 20 
 HttpStatus::CONTINUE;
Please login to merge, or discard this patch.