[CodeQuality] Skip throw after foreach with return in loop on Simplif…

…yForeachToCoalescingRector (#5714)

* CodeQuality] Skip throw after foreach with return in loop on SimplifyForeachToCoalescingRector

* CodeQuality] Skip throw after foreach with return in loop on SimplifyForeachToCoalescingRector

* [ci-review] Rector Rectify

---------

Co-authored-by: GitHub Action <actions@github.com>

rules-tests/CodeQuality/Rector/Foreach_/SimplifyForeachToCoalescingRector/Fixture/skip_throw_after_foreach.php.inc

@@ -0,0 +1,26 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Foreach_\SimplifyForeachToCoalescingRector\Fixture;
+
+class SkipThrowAfterForeach
+{
+    public const SHOPCONFIGKEY_FILTER = 'MY_KEY';
+
+    /**
+     * @return array<string, mixed>
+     *
+     * @throws Exception
+     */
+    private function getFilterConfig(): array
+    {
+        $shopConfig = clxMobileNet::config('shopconfig');
+
+        foreach ($shopConfig as $key => $mixedValues) {
+            if (self::SHOPCONFIGKEY_FILTER === $key) {
+                return $mixedValues;
+            }
+        }
+
+        throw new \Exception('Usage of "'.self::class.'" assumes that ShopConfig "'.self::SHOPCONFIGKEY_FILTER.'" is defined!');
+    }
+}

rules/CodeQuality/Rector/Foreach_/SimplifyForeachToCoalescingRector.php

@@ -112,6 +112,10 @@ public function refactor(Node $node): ?Node
 
                 $nextStmt = $node->stmts[$key + 1] ?? null;
 
+                if (! $nextStmt instanceof Return_) {
+                    continue;
+                }
+
                 $return = $this->processForeachNodeWithReturnInside($foreach, $foreachReturnOrAssign, $nextStmt);
 
                 if (! $return instanceof Return_) {
@@ -121,9 +125,7 @@ public function refactor(Node $node): ?Node
                 $node->stmts[$key] = $return;
 
                 // cleanup next return
-                if ($nextStmt instanceof Return_) {
-                    unset($node->stmts[$key + 1]);
-                }
+                unset($node->stmts[$key + 1]);
 
                 $hasChanged = true;
             }