Symplify / PHP7_CodeSniffer

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/FunctionOpeningBraceSpaceUnitTest.php

Switch Indentation +17 added lines, -17 removed lines

@@ -45,23 +45,23 @@
     public function getErrorList($testFile='FunctionOpeningBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
-        case 'FunctionOpeningBraceSpaceUnitTest.inc':
-            return array(
-                    10 => 1,
-                    25 => 1,
-                   );
-            break;
-        case 'FunctionOpeningBraceSpaceUnitTest.js':
-            return array(
-                    11 => 1,
-                    31 => 1,
-                    38 => 1,
-                    88 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'FunctionOpeningBraceSpaceUnitTest.inc':
+                return array(
+                        10 => 1,
+                        25 => 1,
+                       );
+                break;
+            case 'FunctionOpeningBraceSpaceUnitTest.js':
+                return array(
+                        11 => 1,
+                        31 => 1,
+                        38 => 1,
+                        88 => 1,
+                       );
+                break;
+            default:
+                return array();
+                break;
         }//end switch
         
 

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='FunctionOpeningBraceSpaceUnitTest.inc')
+    public function getErrorList($testFile = 'FunctionOpeningBraceSpaceUnitTest.inc')
     {
         switch ($testFile) {
         case 'FunctionOpeningBraceSpaceUnitTest.inc':

Indentation +2 added lines, -2 removed lines

@@ -47,13 +47,13 @@
             return array(
                     4 => 1,
                     5 => 1,
-                   );
+                    );
         } else if (version_compare(PHP_VERSION, '5.4.0RC1') >= 0) {
             // Shorthand echo is always available from PHP 5.4.0 but needed the
             // short_open_tag ini var to be set for versions before this.
             return array(
                     4 => 1,
-                   );
+                    );
         }
 
         return array();

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/ScopeClosingBraceUnitTest.inc

Spacing +3 added lines, -3 removed lines

@@ -33,8 +33,8 @@ 
     if (!class_exists($class_name)) {
         echo $error;
     }
-    $this->{$property} =& new $class_name($this->db_index);
-    $this->modules[$module] =& $this->{$property};
+    $this->{$property} = & new $class_name($this->db_index);
+    $this->modules[$module] = & $this->{$property};
 }
 
 foreach ($elements as $element) {
@@ -62,7 +62,7 @@ 
 
 ?>
 
-<?php while($row = $data->getRow()): ?>
+<?php while ($row = $data->getRow()): ?>
     <p><?= $val ?></p>
 <?php endwhile; ?>
 

Indentation +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
 
     function test1()
     {
-     }
+        }
 
     function test2() {}
 
@@ -21,7 +21,7 @@ 
 
 class Test2
 {
- }
+    }
 
 
 public function test2()

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/LogicalOperatorSpacingUnitTest.inc

Spacing +4 added lines, -4 removed lines

@@ -1,9 +1,9 @@ 
 <?php
 
 if ($foo || $bar) {}
-if ($foo||$bar && $baz) {}
-if ($foo|| $bar&&$baz) {}
-if ($foo  ||   $bar   && $baz) {}
+if ($foo || $bar && $baz) {}
+if ($foo || $bar && $baz) {}
+if ($foo || $bar && $baz) {}
 
 // Spacing after || below is ignored as it is EOL whitespace.
 if ($foo ||    
@@ -12,7 +12,7 @@ 
 ) {
 }
 
-if ($foo||
+if ($foo ||
     $bar
     &&$baz
 ) {

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/OperatorSpacingUnitTest.php

Switch Indentation +104 added lines, -104 removed lines

@@ -45,110 +45,110 @@
     public function getErrorList($testFile='OperatorSpacingUnitTest.inc')
     {
         switch ($testFile) {
-        case 'OperatorSpacingUnitTest.inc':
-            return array(
-                    4   => 1,
-                    5   => 2,
-                    6   => 1,
-                    7   => 1,
-                    8   => 2,
-                    11  => 1,
-                    12  => 2,
-                    13  => 1,
-                    14  => 1,
-                    15  => 2,
-                    18  => 1,
-                    19  => 2,
-                    20  => 1,
-                    21  => 1,
-                    22  => 2,
-                    25  => 1,
-                    26  => 2,
-                    27  => 1,
-                    28  => 1,
-                    29  => 2,
-                    32  => 1,
-                    33  => 2,
-                    34  => 1,
-                    35  => 1,
-                    36  => 2,
-                    40  => 2,
-                    42  => 2,
-                    44  => 2,
-                    45  => 1,
-                    46  => 2,
-                    53  => 4,
-                    54  => 3,
-                    59  => 10,
-                    64  => 1,
-                    77  => 4,
-                    78  => 1,
-                    79  => 1,
-                    80  => 2,
-                    81  => 1,
-                    84  => 6,
-                    85  => 6,
-                    87  => 4,
-                    88  => 5,
-                    90  => 4,
-                    91  => 5,
-                    128 => 4,
-                    132 => 1,
-                    133 => 1,
-                    135 => 1,
-                    136 => 1,
-                    140 => 1,
-                    141 => 1,
-                    174 => 1,
-                    177 => 1,
-                    178 => 1,
-                    184 => 2,
-                   );
-            break;
-        case 'OperatorSpacingUnitTest.js':
-            return array(
-                    4  => 1,
-                    5  => 2,
-                    6  => 1,
-                    7  => 1,
-                    8  => 2,
-                    11 => 1,
-                    12 => 2,
-                    13 => 1,
-                    14 => 1,
-                    15 => 2,
-                    18 => 1,
-                    19 => 2,
-                    20 => 1,
-                    21 => 1,
-                    22 => 2,
-                    25 => 1,
-                    26 => 2,
-                    27 => 1,
-                    28 => 1,
-                    29 => 2,
-                    32 => 1,
-                    33 => 2,
-                    34 => 1,
-                    35 => 1,
-                    36 => 2,
-                    40 => 2,
-                    42 => 2,
-                    44 => 2,
-                    45 => 1,
-                    46 => 2,
-                    55 => 4,
-                    65 => 1,
-                    66 => 1,
-                    68 => 1,
-                    69 => 1,
-                    73 => 1,
-                    74 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'OperatorSpacingUnitTest.inc':
+                return array(
+                        4   => 1,
+                        5   => 2,
+                        6   => 1,
+                        7   => 1,
+                        8   => 2,
+                        11  => 1,
+                        12  => 2,
+                        13  => 1,
+                        14  => 1,
+                        15  => 2,
+                        18  => 1,
+                        19  => 2,
+                        20  => 1,
+                        21  => 1,
+                        22  => 2,
+                        25  => 1,
+                        26  => 2,
+                        27  => 1,
+                        28  => 1,
+                        29  => 2,
+                        32  => 1,
+                        33  => 2,
+                        34  => 1,
+                        35  => 1,
+                        36  => 2,
+                        40  => 2,
+                        42  => 2,
+                        44  => 2,
+                        45  => 1,
+                        46  => 2,
+                        53  => 4,
+                        54  => 3,
+                        59  => 10,
+                        64  => 1,
+                        77  => 4,
+                        78  => 1,
+                        79  => 1,
+                        80  => 2,
+                        81  => 1,
+                        84  => 6,
+                        85  => 6,
+                        87  => 4,
+                        88  => 5,
+                        90  => 4,
+                        91  => 5,
+                        128 => 4,
+                        132 => 1,
+                        133 => 1,
+                        135 => 1,
+                        136 => 1,
+                        140 => 1,
+                        141 => 1,
+                        174 => 1,
+                        177 => 1,
+                        178 => 1,
+                        184 => 2,
+                       );
+                break;
+            case 'OperatorSpacingUnitTest.js':
+                return array(
+                        4  => 1,
+                        5  => 2,
+                        6  => 1,
+                        7  => 1,
+                        8  => 2,
+                        11 => 1,
+                        12 => 2,
+                        13 => 1,
+                        14 => 1,
+                        15 => 2,
+                        18 => 1,
+                        19 => 2,
+                        20 => 1,
+                        21 => 1,
+                        22 => 2,
+                        25 => 1,
+                        26 => 2,
+                        27 => 1,
+                        28 => 1,
+                        29 => 2,
+                        32 => 1,
+                        33 => 2,
+                        34 => 1,
+                        35 => 1,
+                        36 => 2,
+                        40 => 2,
+                        42 => 2,
+                        44 => 2,
+                        45 => 1,
+                        46 => 2,
+                        55 => 4,
+                        65 => 1,
+                        66 => 1,
+                        68 => 1,
+                        69 => 1,
+                        73 => 1,
+                        74 => 1,
+                       );
+                break;
+            default:
+                return array();
+                break;
         }//end switch
 
     }//end getErrorList()

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='OperatorSpacingUnitTest.inc')
+    public function getErrorList($testFile = 'OperatorSpacingUnitTest.inc')
     {
         switch ($testFile) {
         case 'OperatorSpacingUnitTest.inc':

Indentation +2 added lines, -2 removed lines

@@ -47,13 +47,13 @@
             return array(
                     4 => 1,
                     5 => 1,
-                   );
+                    );
         } else if (version_compare(PHP_VERSION, '5.4.0RC1') >= 0) {
             // Shorthand echo is always available from PHP 5.4.0 but needed the
             // short_open_tag ini var to be set for versions before this.
             return array(
                     4 => 1,
-                   );
+                    );
         }
 
         return array();

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/SemicolonSpacingUnitTest.inc

Spacing +6 added lines, -6 removed lines

@@ -1,12 +1,12 @@
 <?php
 $test = $this->testThis();
-$test = $this->testThis() ;
-$test = $this->testThis()           ;
-for ($var = 1   ; $var < 10  ; $var++) {
-    echo $var ;
+$test = $this->testThis();
+$test = $this->testThis();
+for ($var = 1; $var < 10; $var++) {
+    echo $var;
 }
 $test = $this->testThis()  /* comment here */;
-$test = $this->testThis()  /* comment here */  ;
+$test = $this->testThis()  /* comment here */;
 
-$hello ='foo';
+$hello = 'foo';
 ;
\ No newline at end of file

CodeSniffer/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc

Indentation +4 added lines, -4 removed lines

@@ -58,10 +58,10 @@
 function myFunc( ) {}
 
 function multiLineFunction(
-	array $flatList,
-	$markup,
-	array $otherList,
-	$lastOffset=0
+    array $flatList,
+    $markup,
+    array $otherList,
+    $lastOffset=0
 ) {
 }
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing requiredSpacesAfterOpen 0

Spacing +25 added lines, -25 removed lines

@@ -2,38 +2,38 @@ 
 function myFunc() {}
 function myFunc( ) {}
 function myFunc($blah) {}
-function myFunc( $blah ) {}
+function myFunc($blah) {}
+function myFunc($blah, $blah2, $blah3) {}
+function myFunc($blah, $blah2, $blah3) {}
 function myFunc($blah, $blah2, $blah3) {}
-function myFunc($blah , $blah2, $blah3 ) {}
-function myFunc($blah,$blah2,$blah3) {}
-function myFunc($blah,   $blah2,  $blah3) {}
-function myFunc($blah='hello') {}
+function myFunc($blah, $blah2, $blah3) {}
+function myFunc($blah = 'hello') {}
 function myFunc($blah = 'hello') {}
-function myFunc(PHP_CodeSniffer $object, array $array=array(), $blah3='yo') {}
-function myFunc(  $blah='hello'   , $blah2=   'hi',  $blah3   = 'yo' ) {}
-function myFunc(PHP_CodeSniffer   $object, array  $array=array(), $blah3='yo') {}
-function myFunc( PHP_CodeSniffer $object,  array $array=array(), $blah3='yo') {}
-function myFunc( array  &$one,  array  &$two) {}
+function myFunc(PHP_CodeSniffer $object, array $array = array(), $blah3 = 'yo') {}
+function myFunc($blah = 'hello', $blah2 = 'hi', $blah3 = 'yo') {}
+function myFunc(PHP_CodeSniffer   $object, array  $array = array(), $blah3 = 'yo') {}
+function myFunc(PHP_CodeSniffer $object, array $array = array(), $blah3 = 'yo') {}
+function myFunc(array  &$one, array  &$two) {}
+function myFunc(&$blah) {}
 function myFunc(&$blah) {}
-function myFunc( &$blah ) {}
 
 function multiLineFunction(
     array $flatList,
     $markup,
     array $otherList,
-    $lastOffset=0
+    $lastOffset = 0
 ) {
 }
 
 function multiLineFunction(
     $markup,
     array $otherList,
-    $lastOffset=0
+    $lastOffset = 0
 ) {
 }
 
-$noArgs_longVars = function ($longVar1,  $longerVar2= false) use (
-    $longVar1 ,  $longerVar2= false,
+$noArgs_longVars = function($longVar1, $longerVar2 = false) use (
+    $longVar1, $longerVar2 = false,
     $muchLongerVar3
 )  {
     // body
@@ -41,19 +41,19 @@ 
 
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing equalsSpacing 1
 function myFunc($blah = 'hello') {}
-function myFunc($blah  =  'hello') {}
-function myFunc($blah =  'hello') {}
+function myFunc($blah  = 'hello') {}
+function myFunc($blah = 'hello') {}
 function myFunc($blah  = 'hello') {}
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing equalsSpacing 0
 
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing requiredSpacesAfterOpen 1
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing requiredSpacesBeforeClose 1
 function myFunc($blah) {}
-function myFunc( $blah ) {}
-function myFunc(  $blah  ) {}
-function myFunc( array $blah ) {}
-function myFunc(array $blah ) {}
-function myFunc(  array $blah ) {}
+function myFunc($blah) {}
+function myFunc($blah) {}
+function myFunc(array $blah) {}
+function myFunc(array $blah) {}
+function myFunc(array $blah) {}
 function myFunc() {}
 function myFunc( ) {}
 
@@ -61,16 +61,16 @@ 
 	array $flatList,
 	$markup,
 	array $otherList,
-	$lastOffset=0
+	$lastOffset = 0
 ) {
 }
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing requiredSpacesAfterOpen 0
 // @codingStandardsChangeSetting Squiz.Functions.FunctionDeclarationArgumentSpacing requiredSpacesBeforeClose 0
 
-function myFunc($req, $opt=null, ...$params) {}
+function myFunc($req, $opt = null, ...$params) {}
 function myFunc($param, &...$moreParams) {}
 
 function MissingParamTypeInDocBlock(array$a = null, callable$c, \ArrayObject$o, $foo = []) {}
 
 function myFunc(...$args) {}
-function myFunc( ...$args) {}
+function myFunc(...$args) {}

CodeSniffer/Standards/Squiz/Tests/Functions/FunctionDeclarationUnitTest.inc

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
     function func1();
 
 
-    function func2() ;
+    function func2();
 
 
 }//end interface
@@ -60,12 +60,12 @@ 
 function recurseCreateTree(
     array $flatList,
     $markup,
-    $lastOffset=0
+    $lastOffset = 0
 ) {
 }
 
 
-function test ( ){}
+function test( ) {}
 
 function parent() {}
 function self() {}

CodeSniffer/Standards/Squiz/Tests/Functions/MultiLineFunctionDeclarationUnitTest.inc

Indentation +19 added lines, -19 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 public function someFunctionWithAVeryLongName($firstParameter='something',
-   $secondParameter='booooo',
-   $third=null, $fourthParameter=false,
-     $fifthParameter=123.12,
+    $secondParameter='booooo',
+    $third=null, $fourthParameter=false,
+        $fifthParameter=123.12,
     $sixthParam=true
 ){
 }
@@ -39,22 +39,22 @@ 
     ) {
     }
 
-     public function someFunctionWithAVeryLongName3(
-         $firstParameter, $secondParameter, $third=null
-     ) {
-     }
-
-     public function someFunctionWithAVeryLongName4(
-         $firstParameter, $secondParameter
-     ) {
-     }
-
-     public function someFunctionWithAVeryLongName5(
-         $firstParameter,
-         $secondParameter=array(1,2,3),
-         $third=null
-     ) {
-     }
+        public function someFunctionWithAVeryLongName3(
+            $firstParameter, $secondParameter, $third=null
+        ) {
+        }
+
+        public function someFunctionWithAVeryLongName4(
+            $firstParameter, $secondParameter
+        ) {
+        }
+
+        public function someFunctionWithAVeryLongName5(
+            $firstParameter,
+            $secondParameter=array(1,2,3),
+            $third=null
+        ) {
+        }
 
 }
 

Spacing +30 added lines, -30 removed lines

@@ -1,16 +1,16 @@ 
 <?php
-public function someFunctionWithAVeryLongName($firstParameter='something',
-   $secondParameter='booooo',
-   $third=null, $fourthParameter=false,
-     $fifthParameter=123.12,
-    $sixthParam=true
-){
+public function someFunctionWithAVeryLongName($firstParameter = 'something',
+   $secondParameter = 'booooo',
+   $third = null, $fourthParameter = false,
+     $fifthParameter = 123.12,
+    $sixthParam = true
+) {
 }
 
 function someFunctionWithAVeryLongName2(
-$firstParameter='something',
-$secondParameter='booooo',
-)  {
+$firstParameter = 'something',
+$secondParameter = 'booooo',
+) {
 }
 
 function blah() {
@@ -24,23 +24,23 @@ 
 {
 
     public function someFunctionWithAVeryLongName(
-        $firstParameter='something',
-        $secondParameter='booooo',
-        $third=null,
-        $fourthParameter=false,
-        $fifthParameter=123.12,
-        $sixthParam=true
+        $firstParameter = 'something',
+        $secondParameter = 'booooo',
+        $third = null,
+        $fourthParameter = false,
+        $fifthParameter = 123.12,
+        $sixthParam = true
     ) /** w00t */ {
     }
 
-    public function someFunctionWithAVeryLongName2($firstParameter='something',
-        $secondParameter='booooo',
-        $third=null
+    public function someFunctionWithAVeryLongName2($firstParameter = 'something',
+        $secondParameter = 'booooo',
+        $third = null
     ) {
     }
 
      public function someFunctionWithAVeryLongName3(
-         $firstParameter, $secondParameter, $third=null
+         $firstParameter, $secondParameter, $third = null
      ) {
      }
 
@@ -51,14 +51,14 @@ 
 
      public function someFunctionWithAVeryLongName5(
          $firstParameter,
-         $secondParameter=array(1,2,3),
-         $third=null
+         $secondParameter = array(1, 2, 3),
+         $third = null
      ) {
      }
 
 }
 
-$noArgs_longVars = function () use (
+$noArgs_longVars = function() use (
     $longVar1,
     $longerVar2,
     $muchLongerVar3
@@ -66,7 +66,7 @@ 
     // body
 };
 
-$longArgs_longVars = function (
+$longArgs_longVars = function(
     $longArgument,
     $longerArgument,
     $muchLongerArgument
@@ -78,14 +78,14 @@ 
     // body
 };
 
-$longArgs_longVars = function ($longArgument,
+$longArgs_longVars = function($longArgument,
     $muchLongerArgument) use ($muchLongerVar3
 ) {
     // body
 };
 
-$noArgs_longVars = function () use (
-    $longVar1,  $longerVar2,
+$noArgs_longVars = function() use (
+    $longVar1, $longerVar2,
     $muchLongerVar3
 ) {
     // body
@@ -93,19 +93,19 @@ 
 
 usort(
     $data,
-    function ($a, $b) {
+    function($a, $b) {
         // body
     }
 );
 
 function myFunction(
-    $firstParameter,$secondParameter=[1,2,3],$third=null
+    $firstParameter, $secondParameter = [1, 2, 3], $third = null
 ) {
 }
 
 if (array_filter(
     $commands,
-    function ($cmd) use ($commandName) {
+    function($cmd) use ($commandName) {
         return ($cmd['name'] == $commandName);
     }
 )) {
@@ -127,7 +127,7 @@ 
     // body
 }
 
-$foo = function ($bar = [
+$foo = function($bar = [
     1,
     2,
 ]) use ($longVar1, $longerVar2) {

CodeSniffer/Standards/Squiz/Tests/Functions/FunctionDuplicateArgumentUnitTest.inc

Spacing +1 added lines, -1 removed lines

@@ -2,5 +2,5 @@
 function myFunc($blah, $x, $blah) {}
 function myFunc($a, $b, $c) {}
 function myFunc($a, $a, $a) {}
-function myFunc($a='1', $b='1', $a='2') {}
+function myFunc($a = '1', $b = '1', $a = '2') {}
 ?>