Skip to content

Commit 600de1b

Browse files
committed
Fix
1 parent dc19c4d commit 600de1b

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

rules/DeadCode/Rector/Concat/RemoveUnusedClosureVariableUseRector.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66

77
use PhpParser\Node;
88
use PhpParser\Node\Expr\Closure;
9+
use Rector\DeadCode\NodeAnalyzer\ExprUsedInNodeAnalyzer;
910
use Rector\PhpParser\Node\BetterNodeFinder;
1011
use Rector\Rector\AbstractRector;
1112
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -17,7 +18,8 @@
1718
final class RemoveUnusedClosureVariableUseRector extends AbstractRector
1819
{
1920
public function __construct(
20-
private readonly BetterNodeFinder $betterNodeFinder
21+
private readonly BetterNodeFinder $betterNodeFinder,
22+
private readonly ExprUsedInNodeAnalyzer $exprUsedInNodeAnalyzer
2123
) {
2224
}
2325

@@ -72,7 +74,10 @@ public function refactor(Node $node): ?Node
7274
continue;
7375
}
7476

75-
$isUseUsed = (bool) $this->betterNodeFinder->findVariableOfName($node->stmts, $useVariableName);
77+
$isUseUsed = (bool) $this->betterNodeFinder->findFirst(
78+
$node->stmts,
79+
fn (Node $subNode): bool => $this->exprUsedInNodeAnalyzer->isUsed($subNode, $useVariable->var)
80+
);
7681
if ($isUseUsed) {
7782
continue;
7883
}

0 commit comments

Comments
 (0)