Symplify / PHP7_CodeSniffer

src/Standards/Generic/Tests/WhiteSpace/DisallowTabIndentUnitTest.inc

Indentation +10 added lines, -10 removed lines

@@ -2,25 +2,25 @@ 
 
 class ExampleClass
 {
-	function exampleFunction()	{}
+    function exampleFunction()	{}
 
 }
 
-	$o = <<<EOF
+    $o = <<<EOF
 	this is some text
 	this is some text
 EOF;
 
     $correctVar = true;
-	$correctVar = false;
+    $correctVar = false;
 
 class MyClass
 {
     /**
-	 * Short	description.
-	 *
-	 * @return	void
-	 */
+     * Short	description.
+     *
+     * @return	void
+     */
     public function 	myFunction() {	}
 }
 
@@ -28,9 +28,9 @@ 
 		there';
 
 $foo = array(
-	'Česká republika'   => 'Czech republic',
-	'România'			=> 'Romania',
-	'Magyarország'	   	=> 'Hungary',
+    'Česká republika'   => 'Czech republic',
+    'România'			=> 'Romania',
+    'Magyarország'	   	=> 'Hungary',
 );
 
 $var = "$hello	$there";

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 class ExampleClass
 {
-	function exampleFunction()	{}
+	function exampleFunction() {}
 
 }
 

src/Standards/Generic/Tests/WhiteSpace/DisallowSpaceIndentUnitTest.inc

Indentation +12 added lines, -12 removed lines

@@ -11,29 +11,29 @@
     this is some text
 EOF;
 
-	$correctVar = false;
+    $correctVar = false;
     $correctVar = true;
 
 // Indent with spaces is not allowed
- $hello = array();
-   $world = '';
+    $hello = array();
+    $world = '';
 // here the indention is mixed with tabs and spaces
 // [tab][space][space][space][tab]return "<$name$xmlns$type_str $atts xsi:nil=\"true\"/>";
-	   	return "<$name$xmlns$type_str $atts xsi:nil=\"true\"/>";
+            return "<$name$xmlns$type_str $atts xsi:nil=\"true\"/>";
 // [space][space][space][tab]return "<$name$xmlns$type_str $atts xsi:nil=\"true\"/>";
-   	return "<$name$xmlns$type_str $atts xsi:nil=\"true\"/>";
+        return "<$name$xmlns$type_str $atts xsi:nil=\"true\"/>";
 // Doc comments are indent with tabs and one space
 //[tab]/**
 //[tab][space]*
-	/**
-	 * CVS revision for HTTP headers.
+    /**
+     * CVS revision for HTTP headers.
      *
-	  * @var string
-	 * @access private
-	 */
-	/**
+     * @var string
+     * @access private
+     */
+    /**
      *
-	*/
+     */
 
 $str = 'hello
         there';

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.1.inc

Indentation +44 added lines, -44 removed lines

@@ -4,30 +4,30 @@ 
 class Test {
     function __construct()
     {
-       $this->hello();
+        $this->hello();
     }
 
-   function hello()
+    function hello()
     {
         echo 'hello';
 }//end hello()
 
     function hello2()
     {
-       if (TRUE) {
+        if (TRUE) {
             echo 'hello'; // no error here as its more than 4 spaces.
         } else {
         echo 'bye';
         }
 
         while (TRUE) {
-           echo 'hello';
-         }
+            echo 'hello';
+            }
 
-       do {
-         echo 'hello';
-       } while (TRUE);
-   }
+        do {
+            echo 'hello';
+        } while (TRUE);
+    }
 
     function hello3()
     {
@@ -123,7 +123,7 @@ 
    */
 
     /** This is a T_DOC_COMMENT
-   */
+     */
 
     /*
       This T_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) {
@@ -383,18 +383,18 @@ 
 );
 
 some_function(
-   function() {
-      $a = 403;
-       if ($a === 404) {
-           $a = 403;
-       }
+    function() {
+        $a = 403;
+        if ($a === 404) {
+            $a = 403;
+        }
     }
 );
 
 $myFunction = function() {
     $a = 403;
     if ($a === 404) {
-       $a = 403;
+        $a = 403;
     }
 }
 
@@ -416,8 +416,8 @@ 
     public static function myFunction()
     {
         if (empty($keywords) === FALSE) {
-    	    $keywords = 'foo';
-    	    $existing = 'foo';
+            $keywords = 'foo';
+            $existing = 'foo';
         }
 
         return $keywords;
@@ -657,7 +657,7 @@ 
 ];
 
 if ($foo) {
- ?>
+    ?>
  </p>
   <?php
 }
@@ -665,12 +665,12 @@ 
 function foo()
 {
     $failingTests[$testName][] = array(
-                                  'comment'    => self::_replaceKeywords($failingComment, $result),
-                                  'screenshot' => Test::getScreenshotPath(
-                                      $projectid,
-                                      $result['class_name'],
-                                  ),
-                                 );
+                                    'comment'    => self::_replaceKeywords($failingComment, $result),
+                                    'screenshot' => Test::getScreenshotPath(
+                                        $projectid,
+                                        $result['class_name'],
+                                    ),
+                                    );
 
 }
 
@@ -737,13 +737,13 @@ 
 
     public $scopeOpeners = array(
                             T_CASE => array(
-                                       'end'    => array(
+                                        'end'    => array(
                                                     T_BREAK    => T_BREAK,
                                                     T_RETURN   => T_RETURN,
-                                                   ),
-                                       'strict' => true,
-                                      ),
-                           );
+                                                    ),
+                                        'strict' => true,
+                                        ),
+                            );
 }
 
 echo $string->
@@ -827,12 +827,12 @@ 
 foo();
 
 array(
- 'key1' => function ($bar) {
+    'key1' => function ($bar) {
     return $bar;
- },
- 'key2' => function ($foo) {
+    },
+    'key2' => function ($foo) {
     return $foo;
- },
+    },
 );
 
 ?>
@@ -855,7 +855,7 @@ 
 
 class C0
 {
-     public function m0()
+        public function m0()
     {
     }
 

Switch Indentation +66 added lines, -66 removed lines

@@ -254,11 +254,11 @@ 
 }
 
 switch ($foo) {
-    case 1:
-        switch ($bar) {
+        case 1:
+            switch ($bar) {
             default:
                 echo $string{1};
-        }
+            }
     break;
 }
 
@@ -274,25 +274,25 @@ 
 }
 
 switch ($foo) {
-    case 1:
-        switch ($bar) {
-        default:
-            if ($something) {
-                echo $string{1};
-            } else if ($else) {
-                switch ($else) {
+        case 1:
+            switch ($bar) {
                 default:
+                    if ($something) {
+                        echo $string{1};
+                    } else if ($else) {
+                        switch ($else) {
+                            default:
+                        }
                 }
-            }
         }
     break;
 }
 
 switch ($name) {
-    case "1":
-    case "2":
-    case "3":
-        return true;
+        case "1":
+        case "2":
+        case "3":
+            return true;
 }
 
 switch ($name) {
@@ -314,16 +314,16 @@ 
 }
 
 switch ($name) {
-    case "1":
-        switch ($name2) {
+        case "1":
+            switch ($name2) {
             case "1":
             break;
             case "2":
             break;
-        }
-    break;
-    case "2":
+            }
     break;
+        case "2":
+        break;
 }
 
 switch (true) {
@@ -339,17 +339,17 @@ 
 }
 
 switch (true) {
-    case true: {
-        echo 'hi';
-    }
-    // No break here.
-    case false:
-    case true:
-    case null:{
-    echo 'hi';
+        case true: {
+            echo 'hi';
+        }
+        // No break here.
+        case false:
+        case true:
+        case null:{
         echo 'hi';
-    }
-    break;
+            echo 'hi';
+        }
+        break;
 }
 
 switch (true) {
@@ -910,29 +910,29 @@ 
 }
 
 switch ($foo) {
-    case 1:
-        return array();
-    case 2:
-        return '';
-    case 3:
-        return $function();
-    case 4:
-        return $functionCall($param[0]);
-    case 5:
-        return array() + array(); // Array Merge
-    case 6:
-        // String connect
-        return $functionReturningString('') . $functionReturningString(array());
-    case 7:
-        return functionCall(
-            $withMultiLineParam[0],
-            array(),
-            $functionReturningString(
-                $withMultiLineParam[1]
-            )
-        );
-    case 8:
-        return $param[0][0];
+        case 1:
+            return array();
+        case 2:
+            return '';
+        case 3:
+            return $function();
+        case 4:
+            return $functionCall($param[0]);
+        case 5:
+            return array() + array(); // Array Merge
+        case 6:
+            // String connect
+            return $functionReturningString('') . $functionReturningString(array());
+        case 7:
+            return functionCall(
+                $withMultiLineParam[0],
+                array(),
+                $functionReturningString(
+                    $withMultiLineParam[1]
+                )
+            );
+        case 8:
+            return $param[0][0];
 }
 
 class Test {
@@ -948,10 +948,10 @@ 
 }
 
 switch ($x) {
-    case 1:
-        return [1];
-    default:
-        return [2];
+        case 1:
+            return [1];
+        default:
+            return [2];
 }
 
 switch ($foo) {
@@ -1024,16 +1024,16 @@ 
 }
 
 switch ($parameter) {
-    case null:
-        return [
-            'foo' => in_array(
-                'foo',
-                []
-            ),
-        ];
+        case null:
+            return [
+                'foo' => in_array(
+                    'foo',
+                    []
+                ),
+            ];
 
-    default:
-        return [];
+        default:
+            return [];
 }
 
 class SomeClass

Spacing +29 added lines, -29 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) {
@@ -366,7 +366,7 @@ 
     function c()
     {
         $this->doSomething(
-            function () {
+            function() {
                 echo 123;
             }
         );
@@ -427,7 +427,7 @@ 
 }//end class
 
 $var = call_user_func(
-    $new_var = function () use (&$a) {
+    $new_var = function() use (&$a) {
         if ($a > 0) {
             return $a++;
         } else {
@@ -442,7 +442,7 @@ 
     {
         return array(
             'functions' => Array(
-                'function1' => function ($a, $b, $c) {
+                'function1' => function($a, $b, $c) {
                     $a = $b + $c;
                     $b = $c / 2;
                     return Array($a, $b, $c);
@@ -512,7 +512,7 @@ 
 <?php }
 
 $list = [
-    'fn' => function ($a) {
+    'fn' => function($a) {
         if ($a === true) {
             echo 'hi';
         }
@@ -520,7 +520,7 @@ 
 ];
 
 $list = [
-    'fn' => function ($a) {
+    'fn' => function($a) {
                 if ($a === true) {
                     echo 'hi';
                 }
@@ -576,11 +576,11 @@ 
     $array
 );
 
-function foo(){return function(){};}
+function foo() {return function() {};}
 
 $mockedDatabase->expects($this->at(2))
     ->with($this->callback(
-            function ($subject)
+            function($subject)
             {
             }
         )
@@ -593,7 +593,7 @@ 
             ->appaend('bar')
             ->appaend('baz')
             ->outputUsing(
-                function ()
+                function()
                 {
                 }
             );
@@ -643,13 +643,13 @@ 
 $var = [
     [
         '1' =>
-            function () {
+            function() {
                 return true;
             },
     ],
     [
         '1' =>
-            function () {
+            function() {
                 return true;
             },
         '2' => true,
@@ -677,7 +677,7 @@ 
 $this->mockedDatabase
     ->with(
         $this->callback(
-            function () {
+            function() {
                 return;
             }
         )
@@ -692,7 +692,7 @@ 
             static::where($c)
                 ->where($c)
                 ->where(
-                    function ($d) {
+                    function($d) {
                         $d->whereNull();
                         $d->orWhere();
                     }
@@ -714,7 +714,7 @@ 
 
 $foo->load(
     array(
-        'bar' => function ($baz) {
+        'bar' => function($baz) {
             $baz->call();
         }
     )
@@ -724,7 +724,7 @@ 
 
 $foo = array_unique(
     array_map(
-        function ($entry) {
+        function($entry) {
             return $entry * 2;
         },
         array()
@@ -751,12 +751,12 @@ 
         appaend('bar')->
         appaend('baz')->
         outputUsing(
-            function ()
+            function()
             {
             }
         );
 
-$str = 'the items I want to show are: ' .
+$str = 'the items I want to show are: '.
     implode(
         ', ',
         array('a', 'b', 'c')
@@ -771,14 +771,14 @@ 
 echo $str;
 
 bar([
-    'foo' => foo(function () {
+    'foo' => foo(function() {
         return 'foo';
     })
 ]);
 
 $domains = array_unique(
     array_map(
-        function ($url) {
+        function($url) {
             $urlObject = new \Purl\Url($url);
             return $urlObject->registerableDomain;
         },
@@ -827,10 +827,10 @@ 
 foo();
 
 array(
- 'key1' => function ($bar) {
+ 'key1' => function($bar) {
     return $bar;
  },
- 'key2' => function ($foo) {
+ 'key2' => function($foo) {
     return $foo;
  },
 );
@@ -875,7 +875,7 @@ 
 
 }
 
-declare(ticks=1) {
+declare(ticks = 1) {
 echo 'foo';
 }
 
@@ -922,7 +922,7 @@ 
         return array() + array(); // Array Merge
     case 6:
         // String connect
-        return $functionReturningString('') . $functionReturningString(array());
+        return $functionReturningString('').$functionReturningString(array());
     case 7:
         return functionCall(
             $withMultiLineParam[0],
@@ -940,7 +940,7 @@ 
     public
         $foo
         ,$bar
-        ,$baz = [ ]
+        ,$baz = []
         ;
 
     public function wtfindent() {
@@ -956,7 +956,7 @@ 
 
 switch ($foo) {
     case self::FOO:
-        return $this->bar($gfoo, function ($id) {
+        return $this->bar($gfoo, function($id) {
             return FOO::bar($id);
         }, $values);
     case self::BAR:
@@ -1040,9 +1040,9 @@ 
 {
     public function someFunc()
     {
-        a(function () {
+        a(function() {
             echo "a";
-        })->b(function () {
+        })->b(function() {
             echo "b";
         });
 
@@ -1084,7 +1084,7 @@ 
 }
 
 $foo = foo(
-    function () {
+    function() {
         $foo->debug(
             $a,
             $b
@@ -1097,7 +1097,7 @@ 
 );
 
 $foo = foo(
-    function () {
+    function() {
         $foo->debug(
             $a,
             $b

Braces +14 added lines, -5 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;
+        }
     }
 
     /**
@@ -794,13 +798,18 @@ 
 elseif ($a == 6) :
     echo "a equals 6";
     echo "!!!";
-else :
+else {
+    :
     echo "a is neither 5 nor 6";
+}
 endif;
 
 if ($foo):
-if ($bar) $foo = 1;
-elseif ($baz) $foo = 2;
+if ($bar) {
+    $foo = 1;
+} elseif ($baz) {
+    $foo = 2;
+}
 endif;
 
 $this

src/Standards/Generic/Tests/WhiteSpace/DisallowTabIndentUnitTest.php

Indentation +3 added lines, -3 removed lines

@@ -79,20 +79,20 @@
                     46 => 1,
                     47 => 1,
                     48 => 1,
-                   );
+                    );
             break;
         case 'DisallowTabIndentUnitTest.js':
             return array(
                     3 => 1,
                     5 => 1,
                     6 => 1,
-                   );
+                    );
             break;
         case 'DisallowTabIndentUnitTest.css':
             return array(
                     1 => 1,
                     2 => 1,
-                   );
+                    );
             break;
         default:
             return array();

Switch Indentation +39 added lines, -39 removed lines

@@ -58,45 +58,45 @@
     public function getErrorList($testFile='DisallowTabIndentUnitTest.inc')
     {
         switch ($testFile) {
-        case 'DisallowTabIndentUnitTest.inc':
-            return array(
-                    5  => 2,
-                    9  => 1,
-                    15 => 1,
-                    20 => 2,
-                    21 => 1,
-                    22 => 2,
-                    23 => 1,
-                    24 => 2,
-                    31 => 1,
-                    32 => 2,
-                    33 => 2,
-                    41 => 1,
-                    42 => 1,
-                    43 => 1,
-                    44 => 1,
-                    45 => 1,
-                    46 => 1,
-                    47 => 1,
-                    48 => 1,
-                   );
-            break;
-        case 'DisallowTabIndentUnitTest.js':
-            return array(
-                    3 => 1,
-                    5 => 1,
-                    6 => 1,
-                   );
-            break;
-        case 'DisallowTabIndentUnitTest.css':
-            return array(
-                    1 => 1,
-                    2 => 1,
-                   );
-            break;
-        default:
-            return array();
-            break;
+            case 'DisallowTabIndentUnitTest.inc':
+                return array(
+                        5  => 2,
+                        9  => 1,
+                        15 => 1,
+                        20 => 2,
+                        21 => 1,
+                        22 => 2,
+                        23 => 1,
+                        24 => 2,
+                        31 => 1,
+                        32 => 2,
+                        33 => 2,
+                        41 => 1,
+                        42 => 1,
+                        43 => 1,
+                        44 => 1,
+                        45 => 1,
+                        46 => 1,
+                        47 => 1,
+                        48 => 1,
+                       );
+                break;
+            case 'DisallowTabIndentUnitTest.js':
+                return array(
+                        3 => 1,
+                        5 => 1,
+                        6 => 1,
+                       );
+                break;
+            case 'DisallowTabIndentUnitTest.css':
+                return array(
+                        1 => 1,
+                        2 => 1,
+                       );
+                break;
+            default:
+                return array();
+                break;
         }//end switch
 
     }//end getErrorList()

Spacing +1 added lines, -1 removed lines

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

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.3.inc

Indentation +3 added lines, -3 removed lines

@@ -3,11 +3,11 @@
 <?php
 function test()
 {
-     echo 'test';
-     echo 'test2';
+        echo 'test';
+        echo 'test2';
     echo 'test3';
     if (true) {
-         echo 'test3';
+            echo 'test3';
     }
     echo 'test3';
     $x = f1(

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.php

Indentation +3 added lines, -3 removed lines

@@ -87,7 +87,7 @@ 
                     156 => 1,
                     168 => 1,
                     184 => 1,
-                   );
+                    );
         }
 
         if ($testFile === 'ScopeIndentUnitTest.3.inc') {
@@ -95,7 +95,7 @@ 
                     6  => 1,
                     7  => 1,
                     10 => 1,
-                   );
+                    );
         }
 
         return array(
@@ -159,7 +159,7 @@ 
                 1117 => 1,
                 1119 => 1,
                 1122 => 1,
-               );
+                );
 
     }//end getErrorList()
 

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='ScopeIndentUnitTest.inc')
+    public function getErrorList($testFile = 'ScopeIndentUnitTest.inc')
     {
         if ($testFile === 'ScopeIndentUnitTest.1.js') {
             return array(

src/Standards/Generic/Tests/Commenting/DocCommentUnitTest.php

Indentation +1 added lines, -1 removed lines

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

src/Standards/Generic/Tests/Commenting/TodoUnitTest.php

Indentation +1 added lines, -1 removed lines

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

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='TodoUnitTest.inc')
+    public function getErrorList($testFile = 'TodoUnitTest.inc')
     {
         return array();
 
@@ -59,7 +59,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='TodoUnitTest.inc')
+    public function getWarningList($testFile = 'TodoUnitTest.inc')
     {
         return array(
                 3  => 1,

src/Standards/Generic/Tests/Commenting/FixmeUnitTest.php

Indentation +1 added lines, -1 removed lines

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

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='FixmeUnitTest.inc')
+    public function getErrorList($testFile = 'FixmeUnitTest.inc')
     {
         return array(
                 3  => 1,
@@ -70,7 +70,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='FixmeUnitTest.inc')
+    public function getWarningList($testFile = 'FixmeUnitTest.inc')
     {
         return array();