[CodingStyle] Skip by reference required params on call inside on ArrowFunctionDelegatingCallToFirstClassCallableRector

Author: samsonasik

rules-tests/CodingStyle/Rector/ArrowFunction/ArrowFunctionDelegatingCallToFirstClassCallableRector/Fixture/skip_with_reset.php.inc

@@ -0,0 +1,12 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\ArrowFunction\ArrowFunctionDelegatingCallToFirstClassCallableRector\Fixture;
+
+final class SkipWithReset
+{
+    public function run()
+    {
+        $data = [['a']];
+        return array_map(reset(...), $data);
+    }
+}

rules/CodingStyle/Guard/ArrowFunctionAndClosureFirstClassCallableGuard.php

@@ -21,6 +21,7 @@
 use PHPStan\Reflection\MethodReflection;
 use Rector\NodeNameResolver\NodeNameResolver;
 use Rector\NodeTypeResolver\Node\AttributeKey;
+use Rector\NodeTypeResolver\PHPStan\ParametersAcceptorSelectorVariantsWrapper;
 use Rector\PhpDocParser\NodeTraverser\SimpleCallableNodeTraverser;
 use Rector\PhpParser\AstResolver;
 use Rector\PhpParser\Comparing\NodeComparator;
@@ -102,6 +103,14 @@ public function shouldSkip(
             return true;
         }
 
+        // check if args require by reference
+        $parameters = ParametersAcceptorSelectorVariantsWrapper::select($reflection, $callLike, $scope)->getParameters();
+        foreach ($parameters as $parameter) {
+            if ($parameter->passedByReference()->yes()) {
+                return true;
+            }
+        }
+
         $functionLike = $this->astResolver->resolveClassMethodOrFunctionFromCall($callLike);
         if (! $functionLike instanceof FunctionLike) {
             return false;