[DeadCode] Skip used in Closure use on RemoveUnusedConstructorParamRe…

…ctor (#2341)

rules-tests/DeadCode/Rector/ClassMethod/RemoveUnusedConstructorParamRector/Fixture/skip_used_in_closure_use.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveUnusedConstructorParamRector\Fixture;
+
+final class SkipUsedInClosureUse
+{
+    public function __construct($hey, $man)
+    {
+        echo $hey . ' ';
+
+        (function () use ($man) {
+            echo $man;
+        })();
+    }
+}

src/NodeAnalyzer/ParamAnalyzer.php

@@ -5,6 +5,7 @@
 namespace Rector\Core\NodeAnalyzer;
 
 use PhpParser\Node;
+use PhpParser\Node\Expr\Closure;
 use PhpParser\Node\Expr\ConstFetch;
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\NullableType;
@@ -28,11 +29,21 @@ public function isParamUsedInClassMethod(ClassMethod $classMethod, Param $param)
         return (bool) $this->betterNodeFinder->findFirstInFunctionLikeScoped($classMethod, function (Node $node) use (
             $param
         ): bool {
-            if (! $node instanceof Variable) {
+            if (! $node instanceof Variable && ! $node instanceof Closure) {
                 return false;
             }
 
-            return $this->nodeComparator->areNodesEqual($node, $param->var);
+            if ($node instanceof Variable) {
+                return $this->nodeComparator->areNodesEqual($node, $param->var);
+            }
+
+            foreach ($node->uses as $use) {
+                if ($this->nodeComparator->areNodesEqual($use->var, $param->var)) {
+                    return true;
+                }
+            }
+
+            return false;
         });
     }