[DeadCode] Skip key variable used in throw stmts in catch on RemoveUn…

…usedForeachKeyRector
rectorphp · Dec 18, 2024 · 6eda64b · 6eda64b
1 parent 3baf881
commit 6eda64b
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/NodeManipulator/StmtsManipulator.php b/src/NodeManipulator/StmtsManipulator.php
@@ -10,6 +10,7 @@
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Expression;
+use PhpParser\Node\Stmt\Finally_;
 use PhpParser\Node\Stmt\TryCatch;
 use Rector\Contract\PhpParser\Node\StmtsAwareInterface;
 use Rector\DeadCode\NodeAnalyzer\ExprUsedInNodeAnalyzer;
@@ -96,6 +97,10 @@ public function isVariableUsedInNextStmt(
                 $stmts,
                 $stmtsAware->catches
             );
+
+            if ($stmtsAware->finally instanceof Finally_) {
+                $stmts = array_merge($stmts, $stmtsAware->finally->stmts);
+            }
         }
 
         $variable = new Variable($variableName);