[DeadCode] Skip multi no stmts on RemoveDuplicatedCaseInSwitchRector (#…

…5258)

* [DeadCode] Handle multi no stmts on RemoveDuplicatedCaseInSwitchRector

* Fixed 🎉

rules-tests/DeadCode/Rector/Switch_/RemoveDuplicatedCaseInSwitchRector/Fixture/skip_multi_no_stmts.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\Switch_\RemoveDuplicatedCaseInSwitchRector\Fixture;
+
+class SkipMultiNoStmts
+{
+    public function run($foo)
+    {
+        switch ($foo) {
+            default:
+            case 'A':
+                // Nothing to do
+                break;
+            case 'B':
+            case 'C':
+                // Nothing to do 2
+                break;
+            case 'D':
+                $type = 'BAR';
+                break;
+            case 'E':
+            case 'F':
+                // Nothing to do 3
+                break;
+        }
+    }
+}

rules/DeadCode/Rector/Switch_/RemoveDuplicatedCaseInSwitchRector.php

@@ -98,6 +98,10 @@ private function removeDuplicatedCases(Switch_ $switch): void
     {
         $totalKeys = count($switch->cases);
         foreach (array_keys($switch->cases) as $key) {
+            if (isset($switch->cases[$key - 1]) && $switch->cases[$key-1]->stmts === []) {
+                continue;
+            }
+
             $nextCases = [];
             for ($jumpToKey = $key + 1; $jumpToKey < $totalKeys; ++$jumpToKey) {
                 if (! isset($switch->cases[$jumpToKey])) {