Skip first class callables in NullToStrictStringFuncCallArgRector (#2919

)
rectorphp · Sep 7, 2022 · 2975a1b · 2975a1b
1 parent 90adbf0
commit 2975a1b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/...or/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_first_class_callable.php.inc b/...or/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_first_class_callable.php.inc
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\Php81\Rector\FuncCall\NullToStrictStringFuncCallArgRector\Fixture;
+
+final class SkipFirstClassCallable
+{
+    public function run()
+    {
+        array_map(trim(...), [' ']);
+    }
+}
diff --git a/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php b/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php
@@ -500,6 +500,6 @@ private function resolveOriginalPositions(FuncCall $funcCall): array
     private function shouldSkip(FuncCall $funcCall): bool
     {
         $functionNames = array_keys(self::ARG_POSITION_NAME_NULL_TO_STRICT_STRING);
-        return ! $this->nodeNameResolver->isNames($funcCall, $functionNames);
+        return ! $this->nodeNameResolver->isNames($funcCall, $functionNames) || $funcCall->isFirstClassCallable();
     }
 }