Symplify / PHP7_CodeSniffer

CodeSniffer/Standards/Generic/Sniffs/PHP/LowerCaseKeywordSniff.php

Indentation +4 added lines, -4 removed lines

@@ -103,7 +103,7 @@ 
                 T_USE,
                 T_VAR,
                 T_WHILE,
-               );
+                );
 
     }//end register()
 
@@ -130,9 +130,9 @@ 
 
             $error = 'PHP keywords must be lowercase; expected "%s" but found "%s"';
             $data  = array(
-                      strtolower($keyword),
-                      $keyword,
-                     );
+                        strtolower($keyword),
+                        $keyword,
+                        );
 
             $fix = $phpcsFile->addFixableError($error, $stackPtr, 'Found', $data);
             if ($fix === true) {

CodeSniffer/Standards/Generic/Sniffs/PHP/ForbiddenFunctionsSniff.php

Indentation +16 added lines, -16 removed lines

@@ -40,9 +40,9 @@ 
      * @var array(string => string|null)
      */
     public $forbiddenFunctions = array(
-                                  'sizeof' => 'count',
-                                  'delete' => 'unset',
-                                 );
+                                    'sizeof' => 'count',
+                                    'delete' => 'unset',
+                                    );
 
     /**
      * A cache of forbidden function names, for faster lookups.
@@ -121,19 +121,19 @@ 
         $tokens = $phpcsFile->getTokens();
 
         $ignore = array(
-                   T_DOUBLE_COLON    => true,
-                   T_OBJECT_OPERATOR => true,
-                   T_FUNCTION        => true,
-                   T_CONST           => true,
-                   T_PUBLIC          => true,
-                   T_PRIVATE         => true,
-                   T_PROTECTED       => true,
-                   T_AS              => true,
-                   T_NEW             => true,
-                   T_INSTEADOF       => true,
-                   T_NS_SEPARATOR    => true,
-                   T_IMPLEMENTS      => true,
-                  );
+                    T_DOUBLE_COLON    => true,
+                    T_OBJECT_OPERATOR => true,
+                    T_FUNCTION        => true,
+                    T_CONST           => true,
+                    T_PUBLIC          => true,
+                    T_PRIVATE         => true,
+                    T_PROTECTED       => true,
+                    T_AS              => true,
+                    T_NEW             => true,
+                    T_INSTEADOF       => true,
+                    T_NS_SEPARATOR    => true,
+                    T_IMPLEMENTS      => true,
+                    );
 
         $prevToken = $phpcsFile->findPrevious(T_WHITESPACE, ($stackPtr - 1), null, true);
 

Spacing +1 added lines, -1 removed lines

@@ -204,7 +204,7 @@
      *
      * @return void
      */
-    protected function addError($phpcsFile, $stackPtr, $function, $pattern=null)
+    protected function addError($phpcsFile, $stackPtr, $function, $pattern = null)
     {
         $data  = array($function);
         $error = 'The use of function %s() is ';

CodeSniffer/Standards/Generic/Sniffs/PHP/DisallowShortOpenTagSniff.php

Indentation +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
         return array(
                 T_OPEN_TAG,
                 T_OPEN_TAG_WITH_ECHO,
-               );
+                );
 
     }//end register()
 
@@ -76,7 +76,7 @@ 
                         $nextVar['content'],
                         $openTag['content'],
                         $nextVar['content'],
-                       );
+                        );
             $phpcsFile->addError($error, $stackPtr, 'EchoFound', $data);
         }
 

CodeSniffer/Standards/Generic/Sniffs/PHP/UpperCaseConstantSniff.php

Indentation +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
                 T_TRUE,
                 T_FALSE,
                 T_NULL,
-               );
+                );
 
     }//end register()
 
@@ -70,9 +70,9 @@ 
 
             $error = 'TRUE, FALSE and NULL must be uppercase; expected "%s" but found "%s"';
             $data  = array(
-                      $expected,
-                      $keyword,
-                     );
+                        $expected,
+                        $keyword,
+                        );
 
             $fix = $phpcsFile->addFixableError($error, $stackPtr, 'Found', $data);
             if ($fix === true) {

CodeSniffer/Standards/Generic/Sniffs/Metrics/NestingLevelSniff.php

Indentation +4 added lines, -4 removed lines

@@ -93,16 +93,16 @@
         if ($nestingLevel > $this->absoluteNestingLevel) {
             $error = 'Function\'s nesting level (%s) exceeds allowed maximum of %s';
             $data  = array(
-                      $nestingLevel,
-                      $this->absoluteNestingLevel,
-                     );
+                        $nestingLevel,
+                        $this->absoluteNestingLevel,
+                        );
             $phpcsFile->addError($error, $stackPtr, 'MaxExceeded', $data);
         } else if ($nestingLevel > $this->nestingLevel) {
             $warning = 'Function\'s nesting level (%s) exceeds %s; consider refactoring the function';
             $data    = array(
                         $nestingLevel,
                         $this->nestingLevel,
-                       );
+                        );
             $phpcsFile->addWarning($warning, $stackPtr, 'TooHigh', $data);
         }
 

CodeSniffer/Standards/Generic/Tests/Classes/DuplicateClassNameUnitTest.php

Indentation +4 added lines, -4 removed lines

@@ -62,24 +62,24 @@
             return array(
                     6 => 1,
                     7 => 1,
-                   );
+                    );
             break;
         case 'DuplicateClassNameUnitTest.2.inc':
             return array(
                     2 => 1,
                     3 => 1,
-                   );
+                    );
             break;
         case 'DuplicateClassNameUnitTest.5.inc':
             return array(
                     3 => 1,
                     7 => 1,
-                   );
+                    );
             break;
         case 'DuplicateClassNameUnitTest.6.inc':
             return array(
                     10 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Switch Indentation +26 added lines, -26 removed lines

@@ -58,32 +58,32 @@
     public function getWarningList($testFile='')
     {
         switch ($testFile) {
-        case 'DuplicateClassNameUnitTest.1.inc':
-            return array(
-                    6 => 1,
-                    7 => 1,
-                   );
-            break;
-        case 'DuplicateClassNameUnitTest.2.inc':
-            return array(
-                    2 => 1,
-                    3 => 1,
-                   );
-            break;
-        case 'DuplicateClassNameUnitTest.5.inc':
-            return array(
-                    3 => 1,
-                    7 => 1,
-                   );
-            break;
-        case 'DuplicateClassNameUnitTest.6.inc':
-            return array(
-                    10 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'DuplicateClassNameUnitTest.1.inc':
+                return array(
+                        6 => 1,
+                        7 => 1,
+                       );
+                break;
+            case 'DuplicateClassNameUnitTest.2.inc':
+                return array(
+                        2 => 1,
+                        3 => 1,
+                       );
+                break;
+            case 'DuplicateClassNameUnitTest.5.inc':
+                return array(
+                        3 => 1,
+                        7 => 1,
+                       );
+                break;
+            case 'DuplicateClassNameUnitTest.6.inc':
+                return array(
+                        10 => 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='')
+    public function getWarningList($testFile = '')
     {
         switch ($testFile) {
         case 'DuplicateClassNameUnitTest.1.inc':

CodeSniffer/Standards/Generic/Tests/Classes/DuplicateClassNameUnitTest.6.inc

Indentation +2 added lines, -2 removed lines

@@ -1,10 +1,10 @@
 <?php
 class Test{
     public function testOne(){
-     ?>
+        ?>
      <p>some html here</p>
     <?php 
-   }
+    }
 }
 
 class Test{

Spacing +3 added lines, -3 removed lines

@@ -1,12 +1,12 @@
 <?php
-class Test{
-    public function testOne(){
+class Test {
+    public function testOne() {
      ?>
      <p>some html here</p>
     <?php 
    }
 }
 
-class Test{
+class Test {
 
 }

CodeSniffer/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.php

Indentation +2 added lines, -2 removed lines

@@ -74,7 +74,7 @@ 
                     162 => 1,
                     163 => 1,
                     164 => 1,
-                   );
+                    );
             break;
         case 'InlineControlStructureUnitTest.js':
             return array(
@@ -84,7 +84,7 @@ 
                     13 => 1,
                     15 => 1,
                     21 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Switch Indentation +44 added lines, -44 removed lines

@@ -45,50 +45,50 @@
     public function getErrorList($testFile='InlineControlStructureUnitTest.inc')
     {
         switch ($testFile) {
-        case 'InlineControlStructureUnitTest.inc':
-            return array(
-                    3   => 1,
-                    7   => 1,
-                    11  => 1,
-                    13  => 1,
-                    15  => 1,
-                    17  => 1,
-                    23  => 1,
-                    42  => 1,
-                    43  => 1,
-                    45  => 1,
-                    46  => 1,
-                    49  => 1,
-                    62  => 1,
-                    66  => 1,
-                    78  => 1,
-                    120 => 1,
-                    128 => 1,
-                    134 => 1,
-                    142 => 1,
-                    143 => 1,
-                    144 => 1,
-                    150 => 1,
-                    158 => 1,
-                    159 => 1,
-                    162 => 1,
-                    163 => 1,
-                    164 => 1,
-                   );
-            break;
-        case 'InlineControlStructureUnitTest.js':
-            return array(
-                    3  => 1,
-                    7  => 1,
-                    11 => 1,
-                    13 => 1,
-                    15 => 1,
-                    21 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'InlineControlStructureUnitTest.inc':
+                return array(
+                        3   => 1,
+                        7   => 1,
+                        11  => 1,
+                        13  => 1,
+                        15  => 1,
+                        17  => 1,
+                        23  => 1,
+                        42  => 1,
+                        43  => 1,
+                        45  => 1,
+                        46  => 1,
+                        49  => 1,
+                        62  => 1,
+                        66  => 1,
+                        78  => 1,
+                        120 => 1,
+                        128 => 1,
+                        134 => 1,
+                        142 => 1,
+                        143 => 1,
+                        144 => 1,
+                        150 => 1,
+                        158 => 1,
+                        159 => 1,
+                        162 => 1,
+                        163 => 1,
+                        164 => 1,
+                       );
+                break;
+            case 'InlineControlStructureUnitTest.js':
+                return array(
+                        3  => 1,
+                        7  => 1,
+                        11 => 1,
+                        13 => 1,
+                        15 => 1,
+                        21 => 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='InlineControlStructureUnitTest.inc')
+    public function getErrorList($testFile = 'InlineControlStructureUnitTest.inc')
     {
         switch ($testFile) {
         case 'InlineControlStructureUnitTest.inc':

CodeSniffer/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.inc

Indentation +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 } while ($something);
 
 if(true)
-  $someObject->{$name};
+    $someObject->{$name};
 
 if (true) :
     $foo = true;

Switch Indentation +4 added lines, -4 removed lines

@@ -109,11 +109,11 @@
 switch($response = \Bar::baz('bat', function ($foo) {
     return 'bar';
 })) {
-    case 1:
-        return 'test';
+        case 1:
+            return 'test';
 
-    case 2:
-        return 'other';
+        case 2:
+            return 'other';
 }
 
 $stuff = [1,2,3];

Spacing +13 added lines, -13 removed lines

@@ -20,7 +20,7 @@ 
     $i--;
 } while ($something);
 
-if(true)
+if (true)
   $someObject->{$name};
 
 if (true) :
@@ -60,16 +60,16 @@ 
 function test()
 {
     if ($a)
-        $a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
+        $a .= ' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
 }
 
 if ($a)
     foreach ($b as $c) {
         if ($d) {
-            $e=$f;
-            $g=$h;
-        } elseif ($i==0) {
-            $j=$k;
+            $e = $f;
+            $g = $h;
+        } elseif ($i == 0) {
+            $j = $k;
         }
     }
 
@@ -106,7 +106,7 @@ 
     } else {
     }
 
-switch($response = \Bar::baz('bat', function ($foo) {
+switch ($response = \Bar::baz('bat', function($foo) {
     return 'bar';
 })) {
     case 1:
@@ -116,22 +116,22 @@ 
         return 'other';
 }
 
-$stuff = [1,2,3];
-foreach($stuff as $num)
-    if ($num %2 ) {
+$stuff = [1, 2, 3];
+foreach ($stuff as $num)
+    if ($num % 2) {
         echo "even";
     } else {
         echo "odd";
     }
 
 $i = 0;
-foreach($stuff as $num)
+foreach ($stuff as $num)
     do {
         echo $i;
         $i++;
     } while ($i < 5);
 
-foreach($stuff as $num)
+foreach ($stuff as $num)
     if (true) {
         echo "true1\n";
     }
@@ -156,7 +156,7 @@ 
 }
 
 foreach ($sql as $s)
-        if (!$this->execute) echo "<pre>",$s.";\n</pre>";
+        if (!$this->execute) echo "<pre>", $s.";\n</pre>";
         else {
             $ok = $this->connDest->Execute($s);
             if (!$ok)

Braces +64 added lines, -30 removed lines

@@ -1,27 +1,40 @@ 
 <?php
 
-if ($something) echo 'hello';
+if ($something) {
+    echo 'hello';
+}
 
 if ($something) {
     echo 'hello';
-} else echo 'hi';
+} else {
+    echo 'hi';
+}
 
 if ($something) {
     echo 'hello';
-} else if ($else) echo 'hi';
+} else if ($else) {
+    echo 'hi';
+}
 
-foreach ($something as $thing) echo 'hello';
+foreach ($something as $thing) {
+    echo 'hello';
+}
 
-for ($i; $i > 0; $i--) echo 'hello';
+for ($i; $i > 0; $i--) {
+    echo 'hello';
+}
 
-while ($something) echo 'hello';
+while ($something) {
+    echo 'hello';
+}
 
 do {
     $i--;
 } while ($something);
 
-if(true)
+if(true) {
   $someObject->{$name};
+}
 
 if (true) :
     $foo = true;
@@ -42,31 +55,38 @@ 
 while (!$this->readLine($tokens, $tag));
 while (!$this->readLine($tokens, $tag)); //skip to end of file
 
-foreach ($cookies as $cookie)
+foreach ($cookies as $cookie) {
     if ($cookie->match($uri, $matchSessionCookies, $now))
         $ret[] = $cookie;
+}
 
-foreach ($stringParade as $hit)
-    $hitParade[] = $hit + 0; //cast to integer
+foreach ($stringParade as $hit) {
+    $hitParade[] = $hit + 0;
+}
+//cast to integer
 
 if ($foo) :
     echo 'true';
 elseif ($something) :
     echo 'foo';
-else:
+else {
+    :
     echo 'false';
+}
 endif;
 
 function test()
 {
-    if ($a)
-        $a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
-}
+    if ($a) {
+            $a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
+    }
+    }
 
-if ($a)
+if ($a) {
     foreach ($b as $c) {
         if ($d) {
             $e=$f;
+}
             $g=$h;
         } elseif ($i==0) {
             $j=$k;
@@ -75,7 +95,8 @@ 
 
 ?>
 <div style="text-align: right;">
-    <?php if ($model->scenario == 'simple') $widget->renderPager() ?>
+    <?php if ($model->scenario == 'simple') {
+    $widget->renderPager() ?>
 </div>
 
 <?php
@@ -83,6 +104,7 @@ 
     case Shop_Customer :: ERROR_INVALID_GENDER: ?>
         Ung&uuml;ltiges Geschlecht!
     <?php break;
+}
     case Shop_Customer :: ERROR_EMAIL_IN_USE: ?>
         Die eingetragene E-Mail-Adresse ist bereits registriert.
     <?php break;
@@ -95,8 +117,10 @@ 
             echo 'foo';
         endswitch;
     endif;
-else:
+else {
+    :
     echo 'foo';
+}
 endif;
 
 // ELSE IF split over multiple lines (not inline)
@@ -117,49 +141,59 @@ 
 }
 
 $stuff = [1,2,3];
-foreach($stuff as $num)
+foreach($stuff as $num) {
     if ($num %2 ) {
         echo "even";
+}
     } else {
         echo "odd";
     }
 
 $i = 0;
-foreach($stuff as $num)
+foreach($stuff as $num) {
     do {
         echo $i;
+}
         $i++;
     } while ($i < 5);
 
-foreach($stuff as $num)
+foreach($stuff as $num) {
     if (true) {
         echo "true1\n";
+}
     }
     if (true) {
         echo "true2\n";
     }
 
-if ($foo) echo 'foo';
-elseif ($bar) echo 'bar';
-else echo 'baz';
+if ($foo) {
+    echo 'foo';
+} elseif ($bar) {
+    echo 'bar';
+} else {
+    echo 'baz';
+}
 
 switch ($type) {
     case 1:
         if ($foo) {
             return true;
-        } elseif ($baz)
-            return true;
-        else {
+        } elseif ($baz) {
+                    return true;
+        } else {
             echo 'else';
         }
     break;
 }
 
-foreach ($sql as $s)
+foreach ($sql as $s) {
         if (!$this->execute) echo "<pre>",$s.";\n</pre>";
+}
         else {
             $ok = $this->connDest->Execute($s);
-            if (!$ok)
-                if ($this->neverAbort) $ret = false;
-                else return false;
+            if (!$ok) {
+                            if ($this->neverAbort) $ret = false;
+            } else {
+                    return false;
+                }
         }