Symplify / PHP7_CodeSniffer

src/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php

Indentation +1 added lines, -1 removed lines

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

Doc Comments +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {
@@ -35,7 +35,7 @@ 
     /**
      * Processes this test, when one of its tokens is encountered.
      *
-     * @param PHP_CodeSniffer_File $phpcsFile The current file being processed.
+     * @param File $phpcsFile The current file being processed.
      * @param int                  $stackPtr  The position of the current token in the
      *                                        stack passed in $tokens.
      *

src/Standards/PEAR/Sniffs/Classes/ClassDeclarationSniff.php

Indentation +8 added lines, -8 removed lines

@@ -49,7 +49,7 @@ 
                 T_CLASS,
                 T_INTERFACE,
                 T_TRAIT,
-               );
+                );
 
     }//end register()
 
@@ -99,10 +99,10 @@ 
             if ($braceLine > ($classLine + 1)) {
                 $error = 'Opening brace of a %s must be on the line following the %s declaration; found %s line(s)';
                 $data  = array(
-                          $tokens[$stackPtr]['content'],
-                          $tokens[$stackPtr]['content'],
-                          ($braceLine - $classLine - 1),
-                         );
+                            $tokens[$stackPtr]['content'],
+                            $tokens[$stackPtr]['content'],
+                            ($braceLine - $classLine - 1),
+                            );
                 $fix   = $phpcsFile->addFixableError($error, $curlyBrace, 'OpenBraceWrongLine', $data);
                 if ($fix === true) {
                     $phpcsFile->fixer->beginChangeset();
@@ -142,9 +142,9 @@ 
             if ($spaces !== $expected) {
                 $error = 'Expected %s spaces before opening brace; %s found';
                 $data  = array(
-                          $expected,
-                          $spaces,
-                         );
+                            $expected,
+                            $spaces,
+                            );
 
                 $fix = $phpcsFile->addFixableError($error, $curlyBrace, 'SpaceBeforeBrace', $data);
                 if ($fix === true) {

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
                           $tokens[$stackPtr]['content'],
                           ($braceLine - $classLine - 1),
                          );
-                $fix   = $phpcsFile->addFixableError($error, $curlyBrace, 'OpenBraceWrongLine', $data);
+                $fix = $phpcsFile->addFixableError($error, $curlyBrace, 'OpenBraceWrongLine', $data);
                 if ($fix === true) {
                     $phpcsFile->fixer->beginChangeset();
                     for ($i = ($curlyBrace - 1); $i > $lastContent; $i--) {

src/Standards/PEAR/Tests/Files/IncludingFileUnitTest.php

Indentation +1 added lines, -1 removed lines

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

src/Standards/PEAR/Tests/Formatting/MultiLineAssignmentUnitTest.php

Indentation +1 added lines, -1 removed lines

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

src/Standards/PEAR/Tests/Formatting/MultiLineAssignmentUnitTest.inc

Indentation +2 added lines, -2 removed lines

@@ -1,9 +1,9 @@
 <?php
 $GLOBALS['TSFE']->additionalHeaderData[$this->strApplicationName]
-     = $this->xajax->getJavascript(t3lib_extMgm::siteRelPath('nr_xajax'));
+        = $this->xajax->getJavascript(t3lib_extMgm::siteRelPath('nr_xajax'));
 
 $GLOBALS['TSFE']->additionalHeaderData[$this->strApplicationName]
-   = $this->xajax->getJavascript(t3lib_extMgm::siteRelPath('nr_xajax'));
+    = $this->xajax->getJavascript(t3lib_extMgm::siteRelPath('nr_xajax'));
 
 $GLOBALS['TSFE']->additionalHeaderData[$this->strApplicationName] = 
     $this->xajax->getJavascript(t3lib_extMgm::siteRelPath('nr_xajax'));

Spacing +3 added lines, -3 removed lines

@@ -12,11 +12,11 @@
     = $this->xajax->getJavascript(t3lib_extMgm::siteRelPath('nr_xajax'));
 $GLOBALS['TSFE']->additionalHeaderData[$this->strApplicationName] = 'boo'
 
-$var='string';
+$var = 'string';
 
 function getInstalledStandards(
-    $includeGeneric=false,
-    $standardsDir=''
+    $includeGeneric = false,
+    $standardsDir = ''
 ) {
 }
 ?>

src/Standards/PEAR/Tests/WhiteSpace/ScopeClosingBraceUnitTest.php

Indentation +1 added lines, -1 removed lines

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

src/Standards/PEAR/Tests/WhiteSpace/ScopeIndentUnitTest.inc

Indentation +20 added lines, -20 removed lines

@@ -4,30 +4,30 @@ 
 {
     function __construct()
     {
-       $this->hello(); // error here
+        $this->hello(); // error here
     }
 
-   function hello() // error here
+    function hello() // error here
     { // no error here as brackets can be put anywhere in the pear standard
         echo 'hello';
     }
 
     function hello2()
     {
-       if (TRUE) { // error here
+        if (TRUE) { // error here
             echo 'hello'; // no error here as its more than 4 spaces.
         } else {
         echo 'bye'; // error here
         }
 
         while (TRUE) {
-           echo 'hello'; // error here
-         }
+            echo 'hello'; // error here
+            }
 
-       do { // error here
-         echo 'hello'; // error here
-       } while (TRUE);
-   }
+        do { // error here
+            echo 'hello'; // error here
+        } while (TRUE);
+    }
 
     function hello3()
     {
@@ -123,7 +123,7 @@ 
    */
 
     /**
-   */
+     */
 
     /*
       This comment has a newline in it.
@@ -136,14 +136,14 @@ 
 
         // no errors below.
         $array = array(
-                  'this',
-                  'that' => array(
-                             'hello',
-                             'hello again' => array(
-                                               'hello',
-                                              ),
+                    'this',
+                    'that' => array(
+                                'hello',
+                                'hello again' => array(
+                                                'hello',
+                                                ),
                             ),
-                 );
+                    );
     }
 }
 
@@ -221,8 +221,8 @@ 
 {
     /* taken from http://de3.php.net/manual/en/reserved.php */
     # $m[] = 'declare';
-   /* taken from http://de3.php.net/manual/en/reserved.php */
-   # $m[] = 'declare';
+    /* taken from http://de3.php.net/manual/en/reserved.php */
+    # $m[] = 'declare';
 }
 
 foreach ($elements as $element) {
@@ -308,7 +308,7 @@ 
 array_map(
     function($x)
     {
-       return trim($x);
+        return trim($x);
     },
     $array
 );
\ No newline at end of file

Switch Indentation +27 added lines, -27 removed lines

@@ -32,8 +32,8 @@ 
     function hello3()
     {
         switch ($hello) {
-        case 'hello':
-            break;
+            case 'hello':
+                break;
         }
     }
 
@@ -79,20 +79,20 @@ 
             } else if (FALSE) {
                 foreach ($tokens as $token) {
                     switch ($token) {
-                    case '1':
-                    case '2':
-                        if (true) {
-                            if (false) {
+                        case '1':
+                        case '2':
+                            if (true) {
                                 if (false) {
                                     if (false) {
-                                        echo 'hello';
+                                        if (false) {
+                                            echo 'hello';
+                                        }
                                     }
                                 }
                             }
-                        }
-                    break;
-                    case '5':
                         break;
+                        case '5':
+                            break;
                     }
                     do {
                         while (true) {
@@ -161,20 +161,20 @@ 
                 $last = count($ts) - 1;
 
                 switch ($token) {
-                case '(':
+                    case '(':
 
-                    if ($last >= 3 &&
-                        $ts[0]['token'] != T_CLASS &&
-                        $ts[$last - 2]['token'] == T_OBJECT_OPERATOR &&
-                        $ts[$last - 3]['token'] == T_VARIABLE ) {
+                        if ($last >= 3 &&
+                            $ts[0]['token'] != T_CLASS &&
+                            $ts[$last - 2]['token'] == T_OBJECT_OPERATOR &&
+                            $ts[$last - 3]['token'] == T_VARIABLE ) {
 
 
-                        if (true) {
-                            echo 'hello';
+                            if (true) {
+                                echo 'hello';
+                            }
                         }
-                    }
-                    array_push($braces, $token);
-                    break;
+                        array_push($braces, $token);
+                        break;
                 }
             }
         }
@@ -256,19 +256,19 @@ 
 switch ($foo) {
 case 1:
     switch ($bar) {
-    default:
-        echo $string{1};
+        default:
+            echo $string{1};
     }
     break;
 }
 
 function temp($foo, $bar) {
     switch ($foo) {
-    case 1:
-        switch ($bar) {
-        default:
-            return $foo;
-        }
+        case 1:
+            switch ($bar) {
+                default:
+                    return $foo;
+            }
         break;
     }
 }

Spacing +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
                     if ($last >= 3 &&
                         $ts[0]['token'] != T_CLASS &&
                         $ts[$last - 2]['token'] == T_OBJECT_OPERATOR &&
-                        $ts[$last - 3]['token'] == T_VARIABLE ) {
+                        $ts[$last - 3]['token'] == T_VARIABLE) {
 
 
                         if (true) {

Braces +6 added lines, -2 removed lines

@@ -61,8 +61,12 @@
     public function open()
     {
         // Some inline stuff that shouldn't error
-        if (TRUE) echo 'hello';
-        foreach ($tokens as $token) echo $token;
+        if (TRUE) {
+            echo 'hello';
+        }
+        foreach ($tokens as $token) {
+            echo $token;
+        }
     }
 
     /**

src/Standards/PEAR/Tests/WhiteSpace/ScopeClosingBraceUnitTest.inc

Indentation +2 added lines, -2 removed lines

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

Switch Indentation +17 added lines, -17 removed lines

@@ -48,18 +48,18 @@ 
 switch ($foo) {
 case 1:
     switch ($bar) {
-    default:
-        if ($something) {
-            echo $string{1};
-        } else if ($else) {
-            switch ($else) {
-            case 1:
-                // Do something.
-                break;
-            default:
-                // Do something.
-            break;
-            }
+        default:
+            if ($something) {
+                echo $string{1};
+            } else if ($else) {
+                switch ($else) {
+                    case 1:
+                        // Do something.
+                        break;
+                    default:
+                        // Do something.
+                    break;
+                }
         }
     }
 break;
@@ -69,11 +69,11 @@ 
 }
 
 switch ($httpResponseCode) {
-    case 100:
-    case 101:
-    case 102:
-    default:
-        return 'Unknown';
+        case 100:
+        case 101:
+        case 102:
+        default:
+            return 'Unknown';
 }
 
 switch ($httpResponseCode) {

Spacing +6 added lines, -6 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) {
@@ -85,7 +85,7 @@ 
     return 'Unknown';
 }
 
-switch($i) {
+switch ($i) {
 case 1: {}
 }
 
@@ -127,15 +127,15 @@ 
             <?php endforeach ?>
 </ul>
 <?php
-switch ( $a ) {
+switch ($a) {
     case 'foo':
         do {
             $a = 'b';
-        } while ( $a );
+        } while ($a);
     return 5;
 
     case 'bar':
-        foreach ( $a as $b ) {
+        foreach ($a as $b) {
             $e = 'b';
         }
     return 5;

src/Standards/PEAR/Tests/WhiteSpace/ObjectOperatorIndentUnitTest.php

Indentation +1 added lines, -1 removed lines

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