Symplify / PHP7_CodeSniffer

CodeSniffer/Standards/Squiz/Tests/Classes/ValidClassNameUnitTest.inc

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 class ValidCamelCaseClassEndingWithNumber5 extends MyClass {}
 class 12345 extends MyClass {}
 
-class Testing{}
+class Testing {}
 
 class Base
 {

CodeSniffer/Standards/Squiz/Tests/Classes/ClassDeclarationUnitTest.inc

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 
 
- class ClassDeclaration
+    class ClassDeclaration
 {
 
 }

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
 {
 }//end class
 
-class ClassOne implements ClassFour   ,ClassFive,  ClassSix
+class ClassOne implements ClassFour, ClassFive, ClassSix
 {
 }//end class
 
@@ -82,7 +82,7 @@ 
 
 }//end class
 
-class File implements \Zend_Auth_Storage_Interface,\Zend_Auth_Storage,  \Zend_Foo
+class File implements \Zend_Auth_Storage_Interface, \Zend_Auth_Storage, \Zend_Foo
 {
 }
 
@@ -107,4 +107,4 @@ 
 class IncorrectCodeBeforeClosingBrace
 {
 
-echo phpinfo();}
+echo phpinfo(); }

CodeSniffer/Standards/Squiz/Tests/Classes/ClassFileNameUnitTest.php

Indentation +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
                 16 => 1,
                 17 => 1,
                 18 => 1,
-               );
+                );
 
     }//end getErrorList()
 

CodeSniffer/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.php

Indentation +4 added lines, -4 removed lines

@@ -56,8 +56,8 @@ 
                 30 => 1,
                 37 => 2,
                 39 => 2,
-               );
-             break;
+                );
+                break;
         case 'ForLoopDeclarationUnitTest.js':
             return array(
                 6  => 2,
@@ -69,8 +69,8 @@ 
                 36 => 1,
                 43 => 2,
                 45 => 2,
-               );
-             break;
+                );
+                break;
         default:
             return array();
             break;

Switch Indentation +29 added lines, -29 removed lines

@@ -45,35 +45,35 @@
     public function getErrorList($testFile='ForLoopDeclarationUnitTest.inc')
     {
         switch ($testFile) {
-        case 'ForLoopDeclarationUnitTest.inc':
-            return array(
-                8  => 2,
-                11 => 2,
-                14 => 2,
-                17 => 2,
-                21 => 6,
-                27 => 1,
-                30 => 1,
-                37 => 2,
-                39 => 2,
-               );
-             break;
-        case 'ForLoopDeclarationUnitTest.js':
-            return array(
-                6  => 2,
-                9  => 2,
-                12 => 2,
-                15 => 2,
-                19 => 6,
-                33 => 1,
-                36 => 1,
-                43 => 2,
-                45 => 2,
-               );
-             break;
-        default:
-            return array();
-            break;
+            case 'ForLoopDeclarationUnitTest.inc':
+                return array(
+                    8  => 2,
+                    11 => 2,
+                    14 => 2,
+                    17 => 2,
+                    21 => 6,
+                    27 => 1,
+                    30 => 1,
+                    37 => 2,
+                    39 => 2,
+                   );
+                 break;
+            case 'ForLoopDeclarationUnitTest.js':
+                return array(
+                    6  => 2,
+                    9  => 2,
+                    12 => 2,
+                    15 => 2,
+                    19 => 6,
+                    33 => 1,
+                    36 => 1,
+                    43 => 2,
+                    45 => 2,
+                   );
+                 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='ForLoopDeclarationUnitTest.inc')
+    public function getErrorList($testFile = 'ForLoopDeclarationUnitTest.inc')
     {
         switch ($testFile) {
         case 'ForLoopDeclarationUnitTest.inc':

CodeSniffer/Standards/Squiz/Tests/ControlStructures/SwitchDeclarationUnitTest.inc

Indentation +1 added lines, -1 removed lines

@@ -292,7 +292,7 @@
     return array(
             'whiz',
             'bang',
-           );
+            );
 
     case 2:
     return helper_func(

Switch Indentation +66 added lines, -66 removed lines

@@ -26,14 +26,14 @@ 
         $case = '1';
         return '1';
 
-case '2':
+    case '2':
     case '3':
         $case = '5';
     break;
 
-case '4':
-    $case = '4';
-break;
+    case '4':
+        $case = '4';
+    break;
 
     default:
         $case = null;
@@ -49,19 +49,19 @@ 
 
 // PEAR style.
 switch ($something) {
-case '1':
-    $case = '1';
-    break;
-case '2':
-case '3':
-    $case = '5';
-    break;
-case '4':
-    $case = '4';
-    break;
-default:
-    $case = null;
-    break;
+    case '1':
+        $case = '1';
+        break;
+    case '2':
+    case '3':
+        $case = '5';
+        break;
+    case '4':
+        $case = '4';
+        break;
+    default:
+        $case = null;
+        break;
 }
 
 // Valid, but missing BREAKS.
@@ -85,7 +85,7 @@ 
     Case '1' :
         $case = '1';
 
-case  '2':
+    case  '2':
     case  '3' :
         $case = '5';
 
@@ -140,21 +140,21 @@ 
 }
 
 switch ($var) {
-    case 'one':
-    case 'two':
-    break;
+        case 'one':
+        case 'two':
+        break;
 
-    case 'three':
-        // Nothing to do.
-    break;
+        case 'three':
+            // Nothing to do.
+        break;
 
-    case 'four':
-        echo $hi;
-    break;
+        case 'four':
+            echo $hi;
+        break;
 
-    default:
-        // No default.
-    break;
+        default:
+            // No default.
+        break;
 }
 
 switch ($var) {
@@ -170,8 +170,8 @@ 
 }
 
 switch ($name) {
-    case "1":
-        switch ($name2) {
+        case "1":
+            switch ($name2) {
             case "1":
                 return true;
             break;
@@ -183,24 +183,24 @@ 
             default:
                 // No default.
             break;
-        }
+            }
     break;
 
-    case "2":
+        case "2":
 switch ($name2) {
-    case "1":
-        return true;
-    break;
+            case "1":
+                return true;
+            break;
 
-    case "2":
-    return true;
-    break;
+            case "2":
+            return true;
+            break;
 
-    default:
-        // No default.
-    break;
+            default:
+                // No default.
+            break;
 }
-    break;
+        break;
 }
 
 switch ($name) {
@@ -221,9 +221,9 @@ 
 }
 
 switch ($name2) {
-    default:
-        // No default.
-    break;
+        default:
+            // No default.
+        break;
 }
 
 switch ($foo) {
@@ -258,15 +258,15 @@ 
 }
 
 switch ($something) {
-    case '1':
-        $case = '1';
-    break;
+        case '1':
+            $case = '1';
+        break;
 
-    case '2':
-        throw new Exception('message');
+        case '2':
+            throw new Exception('message');
 
-    default:
-    throw new Exception('message');
+        default:
+        throw new Exception('message');
 }
 
 switch ($something) {
@@ -280,9 +280,9 @@ 
 }
 
 switch ($foo) {
-    case '1':
-        return; // comment
-    break;
+        case '1':
+            return; // comment
+        break;
 
 }
 
@@ -305,13 +305,13 @@ 
 }
 
 switch ($foo) {
-    case 'bar':
-    throw new \Exception(
-        'bar'
-    );
-
-    default:
-    throw new \Exception(
-        'bar'
-    );
+        case 'bar':
+        throw new \Exception(
+            'bar'
+        );
+
+        default:
+        throw new \Exception(
+            'bar'
+        );
 }

CodeSniffer/Standards/Squiz/Tests/ControlStructures/ForEachLoopDeclarationUnitTest.php

Indentation +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
                 21 => 7,
                 26 => 2,
                 28 => 2,
-               );
+                );
 
     }//end getErrorList()
 

CodeSniffer/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.inc

Spacing +9 added lines, -9 removed lines

@@ -5,37 +5,37 @@
 }
 
 // Invalid.
-for ( $i = 0; $i < 10; $i++ ) {
+for ($i = 0; $i < 10; $i++) {
 }
 
-for ($i = 0;  $i < 10;  $i++) {
+for ($i = 0; $i < 10; $i++) {
 }
 
-for ($i = 0 ; $i < 10 ; $i++) {
+for ($i = 0; $i < 10; $i++) {
 }
 
-for ($i = 0;$i < 10;$i++) {
+for ($i = 0; $i < 10; $i++) {
 }
 
 // The works.
-for ( $i = 0 ;  $i < 10 ;  $i++ ) {
+for ($i = 0; $i < 10; $i++) {
 }
 
 for ($i = 0; $i < 10;) {
 }
 
-for ($i = 0; $i < 10; ) {
+for ($i = 0; $i < 10;) {
 }
 
 for ($i = 0; ; $i++) {
 }
-for ($i = 0;; $i++) {
+for ($i = 0; ; $i++) {
 }
 
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForLoopDeclaration requiredSpacesAfterOpen 1
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForLoopDeclaration requiredSpacesBeforeClose 1
 for ($i = 0; $i < 10; $i++) {}
-for ( $i = 0; $i < 10; $i++ ) {}
-for (  $i = 0; $i < 10; $i++  ) {}
+for ($i = 0; $i < 10; $i++) {}
+for ($i = 0; $i < 10; $i++) {}
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForLoopDeclaration requiredSpacesAfterOpen 0
 // @codingStandardsChangeSetting Squiz.ControlStructures.ForLoopDeclaration requiredSpacesBeforeClose 0

CodeSniffer/Standards/Squiz/Tests/ControlStructures/InlineIfDeclarationUnitTest.inc

Indentation +5 added lines, -5 removed lines

@@ -14,11 +14,11 @@
                     : 'positive');
 
 $args = array(
-         '"'.$this->id.'"',
-         '"'.$this->stepInfo['title'].'"',
-         '"'.((isset($this->stepInfo['description']) === TRUE) ? $this->stepInfo['description'] : '').'"',
-         '"'.(isset($this->stepInfo['description']) === TRUE ? $this->stepInfo['description'] : '').'"',
-         '"'.$this->stepInfo['title'].'"',
+            '"'.$this->id.'"',
+            '"'.$this->stepInfo['title'].'"',
+            '"'.((isset($this->stepInfo['description']) === TRUE) ? $this->stepInfo['description'] : '').'"',
+            '"'.(isset($this->stepInfo['description']) === TRUE ? $this->stepInfo['description'] : '').'"',
+            '"'.$this->stepInfo['title'].'"',
         );
 
 echo (TRUE)?'Hello':'Bye';

Spacing +8 added lines, -8 removed lines

@@ -2,12 +2,12 @@ 
 
 echo 'var is '.(($var < 0) ? 'negative' : 'positive');
 echo 'var is '.($var < 0 ? 'negative' : 'positive');
-echo 'var is '.(($var < 0) ? 'negative' :'positive');
-echo 'var is '.(($var < 0) ? 'negative': 'positive');
-echo 'var is '.(($var < 0) ? 'negative' :   'positive');
-echo 'var is '.(($var < 0) ? 'negative'   : 'positive');
-echo 'var is '.(($var < 0) ?   'negative' : 'positive');
-echo 'var is '.(($var < 0)  ? 'negative' : 'positive');
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
+echo 'var is '.(($var < 0) ? 'negative' : 'positive');
 
 echo 'var is '.(($var < 0)
                     ? 'negative'
@@ -21,7 +21,7 @@ 
          '"'.$this->stepInfo['title'].'"',
         );
 
-echo (TRUE)?'Hello':'Bye';
+echo (TRUE) ? 'Hello' : 'Bye';
 
 $array = array(
     'one' => ($test == 1) ? true : false,
@@ -29,7 +29,7 @@ 
     'three' => (($test == 1) ? true : false)
 );
 $var = ($test == 1) ? true : false;
-$var = (myFunc(1,2,3) == 1) ? true : false;
+$var = (myFunc(1, 2, 3) == 1) ? true : false;
 
 set('config', function() {
     $foo = ($bar === "on") ? "1" : "2";

CodeSniffer/Standards/Squiz/Tests/ControlStructures/SwitchDeclarationUnitTest.php

Indentation +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
                 266 => 1,
                 282 => 1,
                 284 => 2,
-               );
+                );
 
     }//end getErrorList()
 

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='SwitchDeclarationUnitTest.inc')
+    public function getErrorList($testFile = 'SwitchDeclarationUnitTest.inc')
     {
         return array(
                 27  => 1,
@@ -100,7 +100,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='SwitchDeclarationUnitTest.inc')
+    public function getWarningList($testFile = 'SwitchDeclarationUnitTest.inc')
     {
         if ($testFile === 'SwitchDeclarationUnitTest.js') {
             return array(273 => 1);