MethodArgumentSpaceFixer - support for anonymous classses

PHP-CS-Fixer · Oct 27, 2021 · fafb79b · fafb79b
1 parent 05ede95
commit fafb79b
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 1 deletion.
diff --git a/src/Fixer/FunctionNotation/MethodArgumentSpaceFixer.php b/src/Fixer/FunctionNotation/MethodArgumentSpaceFixer.php
@@ -136,7 +136,7 @@ public function getPriority(): int
      */
     protected function applyFix(\SplFileInfo $file, Tokens $tokens): void
     {
-        $expectedTokens = [T_LIST, T_FUNCTION, CT::T_USE_LAMBDA];
+        $expectedTokens = [T_LIST, T_FUNCTION, CT::T_USE_LAMBDA, T_CLASS];
 
         if (\PHP_VERSION_ID >= 70400) {
             $expectedTokens[] = T_FN;

diff --git a/tests/Fixer/FunctionNotation/MethodArgumentSpaceFixerTest.php b/tests/Fixer/FunctionNotation/MethodArgumentSpaceFixerTest.php
@@ -182,6 +182,19 @@ public function provideFixCases(): array
                 '<?php new Foo($a=10 , $b=20 ,          $c=30);',
                 ['keep_multiple_spaces_after_comma' => true],
             ],
+            'test anonymous class constructor call' => [
+                '<?php new class ($a=10, $b=20, $this->foo(), $c=30) {};',
+                '<?php new class ($a=10,$b=20 ,$this->foo() ,$c=30) {};',
+            ],
+            'test anonymous class constructor call with multiple spaces' => [
+                '<?php new class ($a=10, $b=20, $c=30) extends Foo {};',
+                '<?php new class ($a=10 , $b=20 ,          $c=30) extends Foo {};',
+            ],
+            'test anonymous class constructor call with multiple spaces (kmsac)' => [
+                '<?php new class ($a=10, $b=20,          $c=30) {};',
+                '<?php new class ($a=10 , $b=20 ,          $c=30) {};',
+                ['keep_multiple_spaces_after_comma' => true],
+            ],
             'test receiving data in list context with omitted values' => [
                 '<?php list($a, $b, , , $c) = foo();',
                 '<?php list($a, $b,, ,$c) = foo();',
@@ -241,6 +254,20 @@ public function provideFixCases(): array
                 );
                 }',
             ],
+            'multi line anonymous class constructor call' => [
+                '<?php if (1) {
+                new class (
+                    $a=10,
+                    $b=20,
+                    $c=30
+                ) {};
+                }',
+                '<?php if (1) {
+                new class (
+                    $a=10 ,
+                $b=20,$c=30) {};
+                }',
+            ],
             'skip arrays but replace arg methods' => [
                 '<?php fnc(1, array(2, func2(6, 7) ,4), 5);',
                 '<?php fnc(1,array(2, func2(6,    7) ,4),    5);',