View GitHub Repository PHP7_CodeSniffer

Schedule Inspection
Subscribe
Completed
Branch — master (d87ed9)
by Tomáš
06:31
created
Category
Toggle Patches
Standards/Squiz/Tests/Commenting/FunctionCommentThrowTagUnitTest.inc 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -239,7 +239,7 @@  discard block
 
 block discarded – undo
239 239 
      * @return array
240 240 
      * @throws ChannelException If the project is invalid.
241 241 
      */
242 
-    public static function getTestInfo($projectid=NULL)
242 
+    public static function getTestInfo($projectid = NULL)
243 243 
     {
244 244 
         try {
245 245 
             DAL::beginTransaction();
 
@@ -264,7 +264,7 @@  discard block
 
 block discarded – undo
264 264 
     {
265 265 
         parent::setUp();
266 266
267 
-        if ( !strlen($this->folderName) ) {
267 
+        if (!strlen($this->folderName)) {
268 268 
             throw new \RuntimeException('The $this->folderName must be specified before proceeding.');
269 269 
         }
270 270 
     }
Please login to merge, or discard this patch.
Standards/Squiz/Tests/Commenting/PostStatementCommentUnitTest.php 3 patches
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='PostStatementCommentUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 
-        case 'PostStatementCommentUnitTest.inc':
49 
-            return array(
50 
-                    6  => 1,
51 
-                    10 => 1,
52 
-                    18 => 1,
53 
-                   );
54 
-            break;
55 
-        case 'PostStatementCommentUnitTest.js':
56 
-            return array(
57 
-                    1 => 1,
58 
-                    4 => 1,
59 
-                    9 => 1,
60 
-                   );
61 
-            break;
62 
-        default:
63 
-            return array();
64 
-            break;
48 
+            case 'PostStatementCommentUnitTest.inc':
49 
+                return array(
50 
+                        6  => 1,
51 
+                        10 => 1,
52 
+                        18 => 1,
53 
+                       );
54 
+                break;
55 
+            case 'PostStatementCommentUnitTest.js':
56 
+                return array(
57 
+                        1 => 1,
58 
+                        4 => 1,
59 
+                        9 => 1,
60 
+                       );
61 
+                break;
62 
+            default:
63 
+                return array();
64 
+                break;
65 65 
         }//end switch
66 66
67 67 
     }//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='PostStatementCommentUnitTest.inc')
45 
+    public function getErrorList($testFile = 'PostStatementCommentUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'PostStatementCommentUnitTest.inc':
Please login to merge, or discard this patch.
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.
Standards/Squiz/Tests/Operators/ComparisonOperatorUsageUnitTest.php 3 patches
Switch Indentation   +43 added lines, -43 removed lines patch added patch discarded remove patch 
@@ -45,49 +45,49 @@
 
 block discarded – undo
45 45 
     public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 
-        case 'ComparisonOperatorUsageUnitTest.inc':
49 
-            return array(
50 
-                    6  => 1,
51 
-                    7  => 1,
52 
-                    10 => 1,
53 
-                    11 => 1,
54 
-                    18 => 1,
55 
-                    19 => 1,
56 
-                    22 => 1,
57 
-                    23 => 1,
58 
-                    29 => 2,
59 
-                    32 => 2,
60 
-                    38 => 4,
61 
-                    47 => 2,
62 
-                    69 => 1,
63 
-                    72 => 1,
64 
-                    75 => 1,
65 
-                    78 => 1,
66 
-                    80 => 1,
67 
-                    82 => 1,
68 
-                    83 => 1,
69 
-                    89 => 1,
70 
-                    92 => 1,
71 
-                    100 => 1,
72 
-                    106 => 1,
73 
-                    112 => 1,
74 
-                   );
75 
-            break;
76 
-        case 'ComparisonOperatorUsageUnitTest.js':
77 
-            return array(
78 
-                    5  => 1,
79 
-                    6  => 1,
80 
-                    17 => 1,
81 
-                    18 => 1,
82 
-                    28 => 2,
83 
-                    40 => 1,
84 
-                    47 => 1,
85 
-                    52 => 1,
86 
-                   );
87 
-            break;
88 
-        default:
89 
-            return array();
90 
-            break;
48 
+            case 'ComparisonOperatorUsageUnitTest.inc':
49 
+                return array(
50 
+                        6  => 1,
51 
+                        7  => 1,
52 
+                        10 => 1,
53 
+                        11 => 1,
54 
+                        18 => 1,
55 
+                        19 => 1,
56 
+                        22 => 1,
57 
+                        23 => 1,
58 
+                        29 => 2,
59 
+                        32 => 2,
60 
+                        38 => 4,
61 
+                        47 => 2,
62 
+                        69 => 1,
63 
+                        72 => 1,
64 
+                        75 => 1,
65 
+                        78 => 1,
66 
+                        80 => 1,
67 
+                        82 => 1,
68 
+                        83 => 1,
69 
+                        89 => 1,
70 
+                        92 => 1,
71 
+                        100 => 1,
72 
+                        106 => 1,
73 
+                        112 => 1,
74 
+                       );
75 
+                break;
76 
+            case 'ComparisonOperatorUsageUnitTest.js':
77 
+                return array(
78 
+                        5  => 1,
79 
+                        6  => 1,
80 
+                        17 => 1,
81 
+                        18 => 1,
82 
+                        28 => 2,
83 
+                        40 => 1,
84 
+                        47 => 1,
85 
+                        52 => 1,
86 
+                       );
87 
+                break;
88 
+            default:
89 
+                return array();
90 
+                break;
91 91 
         }//end switch
92 92
93 93 
     }//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='ComparisonOperatorUsageUnitTest.inc')
45 
+    public function getErrorList($testFile = 'ComparisonOperatorUsageUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'ComparisonOperatorUsageUnitTest.inc':
Please login to merge, or discard this patch.
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.
Standards/Squiz/Tests/Operators/IncrementDecrementUsageUnitTest.inc 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -22,7 +22,7 @@
 
 block discarded – undo
22 22 
 $var -= $var - 1;
23 23 
 $var += $var - 1;
24 24
25 
-$id = $id.'_'.($i-- - $x--).'_'.$x;
25 
+$id = $id.'_'.($i-- -$x--).'_'.$x;
26 26 
 $id = $id.'_'.(++$i - $x--).'_'.$x;
27 27 
 $id = $id.'_'.(--$i - $x++).'_'.$x;
28 28
Please login to merge, or discard this patch.
CodeSniffer/Standards/Squiz/Tests/PHP/DisallowInlineIfUnitTest.php 3 patches
Switch Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch 
@@ -43,19 +43,19 @@
 
 block discarded – undo
43 43 
     public function getErrorList($testFile='DisallowInlineIfUnitTest.inc')
44 44 
     {
45 45 
         switch ($testFile) {
46 
-        case 'DisallowInlineIfUnitTest.inc':
47 
-            return array(
48 
-                    8 => 1,
49 
-                   );
50 
-            break;
51 
-        case 'DisallowInlineIfUnitTest.js':
52 
-            return array(
53 
-                    1 => 1,
54 
-                   );
55 
-            break;
56 
-        default:
57 
-            return array();
58 
-            break;
46 
+            case 'DisallowInlineIfUnitTest.inc':
47 
+                return array(
48 
+                        8 => 1,
49 
+                       );
50 
+                break;
51 
+            case 'DisallowInlineIfUnitTest.js':
52 
+                return array(
53 
+                        1 => 1,
54 
+                       );
55 
+                break;
56 
+            default:
57 
+                return array();
58 
+                break;
59 59 
         }//end switch
60 60
61 61 
     }//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='DisallowInlineIfUnitTest.inc')
43 
+    public function getErrorList($testFile = 'DisallowInlineIfUnitTest.inc')
44 44 
     {
45 45 
         switch ($testFile) {
46 46 
         case 'DisallowInlineIfUnitTest.inc':
Please login to merge, or discard this patch.
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.
Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.inc 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -52,7 +52,7 @@
 
 block discarded – undo
52 52 
 do {
53 53 
     echo $a->count;
54 54 
     $a->count--;
55 
-} while($a->count);
55 
+} while ($a->count);
56 56
57 57 
 for ($i = 0; $i < $a->count; $i++) {}
58 58 
 ?>
Please login to merge, or discard this patch.
CodeSniffer/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.inc 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -16,10 +16,10 @@
 
 block discarded – undo
16 16 
     echo $moreBody;
17 17 
     ?>
18 18 
     <?php  echo 'hi'; ?>
19 
-    <?php  echo 'hi';   ?>
19 
+    <?php  echo 'hi'; ?>
20 20 
     <?php echo 'hi;' ?>
21 21 
     <?php echo 'hi'; echo 'hi;'; ?>
22 
-    <?php  echo 'hi'; echo 'hi;';   ?>
22 
+    <?php  echo 'hi'; echo 'hi;'; ?>
23 23
24 24 
     <?php
25 25 
     ?>
Please login to merge, or discard this patch.
CodeSniffer/Standards/Squiz/Tests/PHP/InnerFunctionsUnitTest.inc 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -10,7 +10,7 @@
 
 block discarded – undo
10 10 
 // Closures are allowed.
11 11 
 function myFunc($foo)
12 12 
 {
13 
-    $callback = function ($bar) use ($foo)
13 
+    $callback = function($bar) use ($foo)
14 14 
                 {
15 15 
                     $bar += $foo;
16 16 
                 };
Please login to merge, or discard this patch.
Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.php 3 patches
Switch Indentation   +24 added lines, -24 removed lines patch added patch discarded remove patch 
@@ -45,30 +45,30 @@
 
 block discarded – undo
45 45 
     public function getErrorList($testFile='DisallowSizeFunctionsInLoopsUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 
-        case 'DisallowSizeFunctionsInLoopsUnitTest.inc':
49 
-            return array(
50 
-                    2  => 1,
51 
-                    7  => 1,
52 
-                    11 => 1,
53 
-                    13 => 1,
54 
-                    18 => 1,
55 
-                    23 => 1,
56 
-                    27 => 1,
57 
-                    29 => 1,
58 
-                    35 => 1,
59 
-                    40 => 1,
60 
-                    44 => 1,
61 
-                    46 => 1,
62 
-                   );
63 
-            break;
64 
-        case 'DisallowSizeFunctionsInLoopsUnitTest.js':
65 
-            return array(
66 
-                    1 => 1,
67 
-                   );
68 
-            break;
69 
-        default:
70 
-            return array();
71 
-            break;
48 
+            case 'DisallowSizeFunctionsInLoopsUnitTest.inc':
49 
+                return array(
50 
+                        2  => 1,
51 
+                        7  => 1,
52 
+                        11 => 1,
53 
+                        13 => 1,
54 
+                        18 => 1,
55 
+                        23 => 1,
56 
+                        27 => 1,
57 
+                        29 => 1,
58 
+                        35 => 1,
59 
+                        40 => 1,
60 
+                        44 => 1,
61 
+                        46 => 1,
62 
+                       );
63 
+                break;
64 
+            case 'DisallowSizeFunctionsInLoopsUnitTest.js':
65 
+                return array(
66 
+                        1 => 1,
67 
+                       );
68 
+                break;
69 
+            default:
70 
+                return array();
71 
+                break;
72 72 
         }//end switch
73 73
74 74 
     }//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='DisallowSizeFunctionsInLoopsUnitTest.inc')
45 
+    public function getErrorList($testFile = 'DisallowSizeFunctionsInLoopsUnitTest.inc')
46 46 
     {
47 47 
         switch ($testFile) {
48 48 
         case 'DisallowSizeFunctionsInLoopsUnitTest.inc':
Please login to merge, or discard this patch.
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.