View GitHub Repository PHP7_CodeSniffer

Schedule Inspection
Subscribe
Completed
Push — master ( 8cd077 )
by Tomáš
09:25
created
Category
Toggle Patches
Standards/Squiz/Tests/Functions/MultiLineFunctionDeclarationUnitTest.php 2 patches
Indentation   +34 added lines, -34 removed lines patch added patch discarded remove patch 
@@ -44,42 +44,42 @@
 
 block discarded – undo
44 44 
     {
45 45 
         if ($testFile === 'MultiLineFunctionDeclarationUnitTest.inc') {
46 46 
             $errors = array(
47 
-                       2   => 1,
48 
-                       3   => 1,
49 
-                       4   => 2,
50 
-                       5   => 1,
51 
-                       7   => 1,
52 
-                       11  => 1,
53 
-                       12  => 1,
54 
-                       13  => 1,
55 
-                       16  => 1,
56 
-                       36  => 1,
57 
-                       43  => 2,
58 
-                       48  => 1,
59 
-                       81  => 1,
60 
-                       82  => 2,
61 
-                       88  => 1,
62 
-                       102 => 2,
63 
-                       137 => 1,
64 
-                       141 => 2,
65 
-                       142 => 1,
66 
-                      );
47 
+                        2   => 1,
48 
+                        3   => 1,
49 
+                        4   => 2,
50 
+                        5   => 1,
51 
+                        7   => 1,
52 
+                        11  => 1,
53 
+                        12  => 1,
54 
+                        13  => 1,
55 
+                        16  => 1,
56 
+                        36  => 1,
57 
+                        43  => 2,
58 
+                        48  => 1,
59 
+                        81  => 1,
60 
+                        82  => 2,
61 
+                        88  => 1,
62 
+                        102 => 2,
63 
+                        137 => 1,
64 
+                        141 => 2,
65 
+                        142 => 1,
66 
+                        );
67 67 
         } else {
68 68 
             $errors = array(
69 
-                       2  => 1,
70 
-                       3  => 1,
71 
-                       4  => 2,
72 
-                       5  => 1,
73 
-                       7  => 1,
74 
-                       11 => 1,
75 
-                       12 => 1,
76 
-                       13 => 1,
77 
-                       16 => 1,
78 
-                       26 => 1,
79 
-                       36 => 1,
80 
-                       43 => 2,
81 
-                       48 => 1,
82 
-                      );
69 
+                        2  => 1,
70 
+                        3  => 1,
71 
+                        4  => 2,
72 
+                        5  => 1,
73 
+                        7  => 1,
74 
+                        11 => 1,
75 
+                        12 => 1,
76 
+                        13 => 1,
77 
+                        16 => 1,
78 
+                        26 => 1,
79 
+                        36 => 1,
80 
+                        43 => 2,
81 
+                        48 => 1,
82 
+                        );
83 83 
         }//end if
84 84
85 85 
         return $errors;
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='MultiLineFunctionDeclarationUnitTest.inc')
43 
+    public function getErrorList($testFile = 'MultiLineFunctionDeclarationUnitTest.inc')
44 44 
     {
45 45 
         if ($testFile === 'MultiLineFunctionDeclarationUnitTest.inc') {
46 46 
             $errors = array(
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Functions/LowercaseFunctionKeywordsUnitTest.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/Functions/FunctionDeclarationUnitTest.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/Functions/FunctionDeclarationUnitTest.inc 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -52,7 +52,7 @@  discard block
 
 block discarded – undo
52 52 
     function func1();
53 53
54 54
55 
-    function func2() ;
55 
+    function func2();
56 56
57 57
58 58 
 }//end interface
 
@@ -60,12 +60,12 @@  discard block
 
 block discarded – undo
60 60 
 function recurseCreateTree(
61 61 
     array $flatList,
62 62 
     $markup,
63 
-    $lastOffset=0
63 
+    $lastOffset = 0
64 64 
 ) {
65 65 
 }
66 66
67 67
68 
-function test ( ){}
68 
+function test( ) {}
69 69
70 70 
 function parent() {}
71 71 
 function self() {}
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Functions/FunctionDuplicateArgumentUnitTest.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/ControlStructures/InlineIfDeclarationUnitTest.inc 2 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -14,11 +14,11 @@
 
 block discarded – undo
14 14 
                     : 'positive');
15 15
16 16 
 $args = array(
17 
-         '"'.$this->id.'"',
18 
-         '"'.$this->stepInfo['title'].'"',
19 
-         '"'.((isset($this->stepInfo['description']) === TRUE) ? $this->stepInfo['description'] : '').'"',
20 
-         '"'.(isset($this->stepInfo['description']) === TRUE ? $this->stepInfo['description'] : '').'"',
21 
-         '"'.$this->stepInfo['title'].'"',
17 
+            '"'.$this->id.'"',
18 
+            '"'.$this->stepInfo['title'].'"',
19 
+            '"'.((isset($this->stepInfo['description']) === TRUE) ? $this->stepInfo['description'] : '').'"',
20 
+            '"'.(isset($this->stepInfo['description']) === TRUE ? $this->stepInfo['description'] : '').'"',
21 
+            '"'.$this->stepInfo['title'].'"',
22 22 
         );
23 23
24 24 
 echo (TRUE)?'Hello':'Bye';
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -2,12 +2,12 @@  discard block
 
 block discarded – undo
2 2
3 3 
 echo 'var is '.(($var < 0) ? 'negative' : 'positive');
4 4 
 echo 'var is '.($var < 0 ? 'negative' : 'positive');
5 
-echo 'var is '.(($var < 0) ? 'negative' :'positive');
6 
-echo 'var is '.(($var < 0) ? 'negative': 'positive');
7 
-echo 'var is '.(($var < 0) ? 'negative' :   'positive');
8 
-echo 'var is '.(($var < 0) ? 'negative'   : 'positive');
9 
-echo 'var is '.(($var < 0) ?   'negative' : 'positive');
10 
-echo 'var is '.(($var < 0)  ? 'negative' : 'positive');
5 
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
6 
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
7 
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
8 
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
9 
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
10 
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
11 11
12 12 
 echo 'var is '.(($var < 0)
13 13 
                     ? 'negative'
 
@@ -21,7 +21,7 @@  discard block
 
 block discarded – undo
21 21 
          '"'.$this->stepInfo['title'].'"',
22 22 
         );
23 23
24 
-echo (TRUE)?'Hello':'Bye';
24 
+echo (TRUE) ? 'Hello' : 'Bye';
25 25
26 26 
 $array = array(
27 27 
     'one' => ($test == 1) ? true : false,
 
@@ -29,7 +29,7 @@  discard block
 
 block discarded – undo
29 29 
     'three' => (($test == 1) ? true : false)
30 30 
 );
31 31 
 $var = ($test == 1) ? true : false;
32 
-$var = (myFunc(1,2,3) == 1) ? true : false;
32 
+$var = (myFunc(1, 2, 3) == 1) ? true : false;
33 33
34 34 
 set('config', function() {
35 35 
     $foo = ($bar === "on") ? "1" : "2";
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.php 3 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -39,8 +39,8 @@  discard block
 
 block discarded – undo
39 39 
                     30 => 1,
40 40 
                     37 => 2,
41 41 
                     39 => 2,
42 
-                   );
43 
-             break;
42 
+                    );
43 
+                break;
44 44 
         case 'ForLoopDeclarationUnitTest.js':
45 45 
             return array(
46 46 
                     6  => 2,
 
@@ -52,8 +52,8 @@  discard block
 
 block discarded – undo
52 52 
                     36 => 1,
53 53 
                     43 => 2,
54 54 
                     45 => 2,
55 
-                   );
56 
-             break;
55 
+                    );
56 
+                break;
57 57 
         default:
58 58 
             return array();
59 59 
             break;
Please login to merge, or discard this patch.
Switch Indentation   +29 added lines, -29 removed lines patch added patch discarded remove patch 
@@ -28,35 +28,35 @@
 
 block discarded – undo
28 28 
     public function getErrorList($testFile='ForLoopDeclarationUnitTest.inc')
29 29 
     {
30 30 
         switch ($testFile) {
31 
-        case 'ForLoopDeclarationUnitTest.inc':
32 
-            return array(
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 
-             break;
44 
-        case 'ForLoopDeclarationUnitTest.js':
45 
-            return array(
46 
-                    6  => 2,
47 
-                    9  => 2,
48 
-                    12 => 2,
49 
-                    15 => 2,
50 
-                    19 => 6,
51 
-                    33 => 1,
52 
-                    36 => 1,
53 
-                    43 => 2,
54 
-                    45 => 2,
55 
-                   );
56 
-             break;
57 
-        default:
58 
-            return array();
59 
-            break;
31 
+            case 'ForLoopDeclarationUnitTest.inc':
32 
+                return array(
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 
+                 break;
44 
+            case 'ForLoopDeclarationUnitTest.js':
45 
+                return array(
46 
+                        6  => 2,
47 
+                        9  => 2,
48 
+                        12 => 2,
49 
+                        15 => 2,
50 
+                        19 => 6,
51 
+                        33 => 1,
52 
+                        36 => 1,
53 
+                        43 => 2,
54 
+                        45 => 2,
55 
+                       );
56 
+                 break;
57 
+            default:
58 
+                return array();
59 
+                break;
60 60 
         }//end switch
61 61
62 62 
     }//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='ForLoopDeclarationUnitTest.inc')
45 
+    public function getErrorList($testFile = 'ForLoopDeclarationUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'ForLoopDeclarationUnitTest.inc':
Please login to merge, or discard this patch.
Standards/Squiz/Tests/ControlStructures/ForEachLoopDeclarationUnitTest.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.
Standards/Squiz/Tests/ControlStructures/ForEachLoopDeclarationUnitTest.inc 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -30,7 +30,7 @@
 
 block discarded – undo
30 30 
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForEachLoopDeclaration requiredSpacesBeforeClose 0
31 31
32 32 
 foreach ([
33 
-          'foo'    => 'bar',
34 
-          'foobaz' => 'bazzy',
35 
-         ] as $key => $value) {
33 
+            'foo'    => 'bar',
34 
+            'foobaz' => 'bazzy',
35 
+            ] as $key => $value) {
36 36 
 }
37 37 
\ No newline at end of file
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -5,7 +5,7 @@  discard block
 
 block discarded – undo
5 5 
 }
6 6
7 7 
 // Invalid.
8 
-foreach ( $something as $blah => $that ) {
8 
+foreach ($something as $blah => $that) {
9 9 
 }
10 10
11 11 
 foreach ($something   as   $blah => $that) {
 
@@ -18,14 +18,14 @@  discard block
 
 block discarded – undo
18 18 
 }
19 19
20 20 
 // The works.
21 
-foreach (  $something   aS   $blah    =>   $that   ) {
21 
+foreach ($something   aS   $blah    =>   $that) {
22 22 
 }
23 23
24 24 
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForEachLoopDeclaration requiredSpacesAfterOpen 1
25 25 
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForEachLoopDeclaration requiredSpacesBeforeClose 1
26 26 
 foreach ($something as $blah => $that) {}
27 
-foreach ( $something as $blah => $that ) {}
28 
-foreach (  $something as $blah => $that  ) {}
27 
+foreach ($something as $blah => $that) {}
28 
+foreach ($something as $blah => $that) {}
29 29 
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForEachLoopDeclaration requiredSpacesAfterOpen 0
30 30 
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForEachLoopDeclaration requiredSpacesBeforeClose 0
31 31
Please login to merge, or discard this patch.