[EarlyReturn] Do not move up variable up next foreach on ChangeAndIfT…

…oEarlyReturnRector (#852)

* [EarlyReturn] Do not move up variable up next foreach

* update fixture

* Fixed 🎉

* cs

rules-tests/EarlyReturn/Rector/If_/ChangeAndIfToEarlyReturnRector/Fixture/do_not_move_variable_up_foreach.php.inc

@@ -0,0 +1,45 @@
+<?php
+
+namespace Rector\Tests\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
+
+class DoNotMoveVariableUpForeach
+{
+    public function run(array $data, $a, $b)
+    {
+        foreach ($data as $value) {
+            if ($a && $b) {
+                unset($value);
+            }
+        }
+
+        $targets = [];
+        return $targets;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
+
+class DoNotMoveVariableUpForeach
+{
+    public function run(array $data, $a, $b)
+    {
+        foreach ($data as $value) {
+            if (!$a) {
+                continue;
+            }
+            if (!$b) {
+                continue;
+            }
+            unset($value);
+        }
+
+        $targets = [];
+        return $targets;
+    }
+}
+
+?>

rules/EarlyReturn/Rector/If_/ChangeAndIfToEarlyReturnRector.php

@@ -147,7 +147,9 @@ private function processReplaceIfs(If_ $node, array $conditions, Return_ $ifNext
 
         $this->removeNode($node);
 
-        if (! $node->stmts[0] instanceof Return_ && $ifNextReturnClone->expr instanceof Expr) {
+        if (! $node->stmts[0] instanceof Return_ && $ifNextReturnClone->expr instanceof Expr && ! $this->contextAnalyzer->isInLoop(
+            $node
+        )) {
             return [$node, $ifNextReturnClone];
         }