Symplify / PHP7_CodeSniffer

CodeSniffer/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.inc

Switch Indentation +33 added lines, -33 removed lines

@@ -49,13 +49,13 @@ 
 }
 
 switch ($lowerVarType) {
-    case 'bool':
-        return 'boolean';
-        echo 'hi';
-    case 'double':
-    case 'real':
-        return 'float';
-        echo 'hi';
+        case 'bool':
+            return 'boolean';
+            echo 'hi';
+        case 'double':
+        case 'real':
+            return 'float';
+            echo 'hi';
 }
 
 while ($line=fgets($fp,2*1024*1024))
@@ -66,14 +66,14 @@ 
 }
 
 switch ($var) {
-    case 1:
-        echo '1';
-    break;
+        case 1:
+            echo '1';
+        break;
 
-        echo 'non-executable';
-    default:
-        echo '2';
-    break;
+            echo 'non-executable';
+        default:
+            echo '2';
+        break;
 }
 
 switch (0) {
@@ -124,13 +124,13 @@ 
 }//end myFunction()
 
 switch ($var) {
-    case 1: {
-        return '1';
-    }
+        case 1: {
+            return '1';
+        }
 
-    case 2: {
-        return '2';
-    }
+        case 2: {
+            return '2';
+        }
 }
 
 defined('FOO') or die('error');
@@ -202,12 +202,12 @@ 
 }
 
 switch ($foo) {
-    case 'foo':
-        if ($foo)
-            return $foo;
-        return $bar;
-    default:
-        return $bar;
+        case 'foo':
+            if ($foo)
+                return $foo;
+            return $bar;
+        default:
+            return $bar;
 }
 
 function foo()
@@ -216,13 +216,13 @@ 
 }
 
 switch (true) {
-    case 1:
-        return foo(
-            function () {
-                $foo = $bar;  // when this is removed it works ok
-                return false; // from here on it reports unreachable
-            }
-        );
+        case 1:
+            return foo(
+                function () {
+                    $foo = $bar;  // when this is removed it works ok
+                    return false; // from here on it reports unreachable
+                }
+            );
 }
 
 exit();

Spacing +8 added lines, -8 removed lines

@@ -58,9 +58,9 @@ 
         echo 'hi';
 }
 
-while ($line=fgets($fp,2*1024*1024))
+while ($line = fgets($fp, 2 * 1024 * 1024))
 {
-    if (!preg_match("/^<([a-z0-9_]+)/",$line,$matches))
+    if (!preg_match("/^<([a-z0-9_]+)/", $line, $matches))
     continue;
     print $line;
 }
@@ -109,7 +109,7 @@ 
 }//end myFunction()
 
 public static function thisCausesAnError() {
-return new foo(function() {return $foo;}
+return new foo(function() {return $foo; }
 );
 }
 
@@ -180,8 +180,8 @@ 
     echo $foo;
 }
 
-function test(){
-    switch($a) {
+function test() {
+    switch ($a) {
         case 1:
             if (empty($b))
                 return 0;
@@ -212,14 +212,14 @@ 
 
 function foo()
 {
-    return $bar->{$action . 'JsonAction'}();
+    return $bar->{$action.'JsonAction'}();
 }
 
 switch (true) {
     case 1:
         return foo(
-            function () {
-                $foo = $bar;  // when this is removed it works ok
+            function() {
+                $foo = $bar; // when this is removed it works ok
                 return false; // from here on it reports unreachable
             }
         );

Braces +16 added lines, -12 removed lines

@@ -60,8 +60,9 @@ 
 
 while ($line=fgets($fp,2*1024*1024))
 {
-    if (!preg_match("/^<([a-z0-9_]+)/",$line,$matches))
-    continue;
+    if (!preg_match("/^<([a-z0-9_]+)/",$line,$matches)) {
+        continue;
+    }
     print $line;
 }
 
@@ -183,19 +184,21 @@ 
 function test(){
     switch($a) {
         case 1:
-            if (empty($b))
-                return 0;
+            if (empty($b)) {
+                            return 0;
+            }
             break;
         default:
             return 2;
     }
 
-    if (empty($a))
-        echo '1';
-    elseif ($empty($b))
-        return 0;
-    else
-        return 1;
+    if (empty($a)) {
+            echo '1';
+    } elseif ($empty($b)) {
+            return 0;
+    } else {
+            return 1;
+    }
 
     echo "oi";
     return 1;
@@ -203,8 +206,9 @@ 
 
 switch ($foo) {
     case 'foo':
-        if ($foo)
-            return $foo;
+        if ($foo) {
+                    return $foo;
+        }
         return $bar;
     default:
         return $bar;

CodeSniffer/Standards/Squiz/Tests/PHP/DisallowMultipleAssignmentsUnitTest.inc

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-function getVar($varname='var', $var='var')
+function getVar($varname = 'var', $var = 'var')
 {
     $var = $var2 = getVar();
     $var = $var2 = $var3 = '';
@@ -38,5 +38,5 @@ 
 
 $obj->$classVar = $prefix.'-'.$type;
 
-$closureWithDefaultParamter = function(array $testArray=array()) {};
+$closureWithDefaultParamter = function(array $testArray = array()) {};
 ?>

CodeSniffer/Standards/Squiz/Tests/PHP/CommentedOutCodeUnitTest.php

Switch Indentation +18 added lines, -18 removed lines

@@ -58,24 +58,24 @@
     public function getWarningList($testFile='CommentedOutCodeUnitTest.inc')
     {
         switch ($testFile) {
-        case 'CommentedOutCodeUnitTest.inc':
-            return array(
-                    6  => 1,
-                    8  => 1,
-                    15 => 1,
-                    19 => 1,
-                    87 => 1,
-                   );
-            break;
-        case 'CommentedOutCodeUnitTest.css':
-            return array(
-                    7  => 1,
-                    16 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'CommentedOutCodeUnitTest.inc':
+                return array(
+                        6  => 1,
+                        8  => 1,
+                        15 => 1,
+                        19 => 1,
+                        87 => 1,
+                       );
+                break;
+            case 'CommentedOutCodeUnitTest.css':
+                return array(
+                        7  => 1,
+                        16 => 1,
+                       );
+                break;
+            default:
+                return array();
+                break;
         }//end switch
 
     }//end getWarningList()

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='CommentedOutCodeUnitTest.inc')
+    public function getWarningList($testFile = 'CommentedOutCodeUnitTest.inc')
     {
         switch ($testFile) {
         case 'CommentedOutCodeUnitTest.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/Objects/DisallowObjectStringIndexUnitTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='DisallowObjectStringIndexUnitTest.js')
+    public function getErrorList($testFile = 'DisallowObjectStringIndexUnitTest.js')
     {
         if ($testFile !== 'DisallowObjectStringIndexUnitTest.js') {
             return array();

Indentation +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
                 48 => 1,
                 70 => 1,
                 71 => 1,
-               );
+                );
 
     }//end getErrorList()
 

CodeSniffer/Standards/Squiz/Tests/Objects/ObjectInstantiationUnitTest.inc

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 $obj = new MyClass();
-$obj =& new MyClass();
+$obj = & new MyClass();
 $obj = &new MyClass();
 new MyClass();
 

CodeSniffer/Standards/Squiz/Tests/Objects/ObjectMemberCommaUnitTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='ObjectMemberCommaUnitTest.js')
+    public function getErrorList($testFile = 'ObjectMemberCommaUnitTest.js')
     {
         if ($testFile !== 'ObjectMemberCommaUnitTest.js') {
             return array();

Indentation +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
                 48 => 1,
                 70 => 1,
                 71 => 1,
-               );
+                );
 
     }//end getErrorList()
 

CodeSniffer/Standards/PSR1/Sniffs/Files/SideEffectsSniff.php

Indentation +9 added lines, -9 removed lines

@@ -59,9 +59,9 @@ 
         if ($result['symbol'] !== null && $result['effect'] !== null) {
             $error = 'A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line %s and the first side effect is on line %s.';
             $data  = array(
-                      $tokens[$result['symbol']]['line'],
-                      $tokens[$result['effect']]['line'],
-                     );
+                        $tokens[$result['symbol']]['line'],
+                        $tokens[$result['effect']]['line'],
+                        );
             $phpcsFile->addWarning($error, 0, 'FoundWithSymbols', $data);
             $phpcsFile->recordMetric($stackPtr, 'Declarations and side effects mixed', 'yes');
         } else {
@@ -96,13 +96,13 @@ 
                     T_INTERFACE => T_INTERFACE,
                     T_TRAIT     => T_TRAIT,
                     T_FUNCTION  => T_FUNCTION,
-                   );
+                    );
 
         $conditions = array(
-                       T_IF     => T_IF,
-                       T_ELSE   => T_ELSE,
-                       T_ELSEIF => T_ELSEIF,
-                      );
+                        T_IF     => T_IF,
+                        T_ELSE   => T_ELSE,
+                        T_ELSEIF => T_ELSEIF,
+                        );
 
         $firstSymbol = null;
         $firstEffect = null;
@@ -230,7 +230,7 @@ 
         return array(
                 'symbol' => $firstSymbol,
                 'effect' => $firstEffect,
-               );
+                );
 
     }//end _searchForConflict()
 

CodeSniffer/Standards/PSR1/Tests/Files/SideEffectsUnitTest.php

Switch Indentation +8 added lines, -8 removed lines

@@ -60,14 +60,14 @@
     public function getWarningList($testFile='')
     {
         switch ($testFile) {
-        case 'SideEffectsUnitTest.3.inc':
-        case 'SideEffectsUnitTest.4.inc':
-        case 'SideEffectsUnitTest.5.inc':
-            return array(
-                    1 => 1,
-                   );
-        default:
-            return array();
+            case 'SideEffectsUnitTest.3.inc':
+            case 'SideEffectsUnitTest.4.inc':
+            case 'SideEffectsUnitTest.5.inc':
+                return array(
+                        1 => 1,
+                       );
+            default:
+                return array();
         }//end switch
 
     }//end getWarningList()

Indentation +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         case 'CharacterBeforePHPOpeningTagUnitTest.1.inc':
             return array(
                     2 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
+    public function getErrorList($testFile = '')
     {
         switch ($testFile) {
         case 'EndFileNoNewlineUnitTest.1.inc':
@@ -75,7 +75,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='')
+    public function getWarningList($testFile = '')
     {
         return array();
 

CodeSniffer/Standards/PSR1/Tests/Files/SideEffectsUnitTest.1.inc

Spacing +2 added lines, -2 removed lines

@@ -7,9 +7,9 @@
 use Something;
 use SomethingElse;
 
-declare(ticks=1);
+declare(ticks = 1);
 
-declare(ticks=1) {
+declare(ticks = 1) {
     // Code.
 }