Adds support for array closures in Squiz.Functions.FunctionDeclaratio…

…nArgumentSpacing Related to #2523
squizlabs · Nov 26, 2019 · 62ae7e7 · 62ae7e7
1 parent 80ebd4a
commit 62ae7e7
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 0 deletions.
diff --git a/src/Standards/Squiz/Sniffs/Functions/FunctionDeclarationArgumentSpacingSniff.php b/src/Standards/Squiz/Sniffs/Functions/FunctionDeclarationArgumentSpacingSniff.php
@@ -48,6 +48,7 @@ public function register()
         return [
             T_FUNCTION,
             T_CLOSURE,
+            T_FN,
         ];
 
     }//end register()

diff --git a/src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc b/src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc
@@ -107,3 +107,5 @@ $a = function ($var1, $var2=false) use (
     $longVar2 ,  &$longerVar2,
     $muchLongerVar3
 ) {};
+
+fn ($a,$b = null) => $a($b);
diff --git a/src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc.fixed b/src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc.fixed
@@ -107,3 +107,5 @@ $a = function ($var1, $var2=false) use (
     $longVar2, &$longerVar2,
     $muchLongerVar3
 ) {};
+
+fn ($a, $b=null) => $a($b);
diff --git a/src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.php b/src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.php
@@ -62,6 +62,7 @@ public function getErrorList()
             102 => 2,
             106 => 1,
             107 => 2,
+            111 => 3,
         ];
 
     }//end getErrorList()