[DowngradePhp80] Handle multiple coalesce before throw on DowngradeThrowExprRector

Author: samsonasik

rules-tests/DowngradePhp80/Rector/Expression/DowngradeThrowExprRector/Fixture/multiple_coalesce.php.inc

@@ -0,0 +1,30 @@
+<?php
+
+namespace Rector\Tests\DowngradePhp80\Rector\Expression\DowngradeThrowExprRector\Fixture;
+
+class MultipleCoalesce
+{
+    public function run($content)
+    {
+        $contentId = $content->getInternalId() ?? $content->getId() ?? throw new \LogicException('content id is required for ackVersion');
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\DowngradePhp80\Rector\Expression\DowngradeThrowExprRector\Fixture;
+
+class MultipleCoalesce
+{
+    public function run($content)
+    {
+        if (($content->getInternalId() ?? $content->getId()) === null) {
+            throw new \LogicException('content id is required for ackVersion');
+        }
+        $contentId = $content->getInternalId() ?? $content->getId();
+    }
+}
+
+?>

rules/DowngradePhp80/Rector/Expression/DowngradeThrowExprRector.php

@@ -180,9 +180,30 @@ private function refactorTernary(Ternary $ternary, ?Assign $assign): If_|null|ar
     private function refactorCoalesce(Coalesce $coalesce, ?Assign $assign): If_|null|array
     {
         if (! $coalesce->right instanceof Throw_) {
+            $rightCoalesce = $coalesce->right;
+            $leftCoalesce = $coalesce->left;
+
+            while ($rightCoalesce instanceof Coalesce) {
+                $leftCoalesce = new Coalesce($leftCoalesce, $rightCoalesce->left);
+                $rightCoalesce = $rightCoalesce->right;
+            }
+
+            if ($rightCoalesce instanceof Throw_) {
+                $coalesce = new Coalesce($leftCoalesce, $rightCoalesce);
+                return $this->processCoalesce($coalesce, $assign);
+            }
+
             return null;
         }
 
+        return $this->processCoalesce($coalesce, $assign);
+    }
+
+    /**
+     * @return If_|Stmt[]|null
+     */
+    private function processCoalesce(Coalesce $coalesce, ?Assign $assign): If_|null|array
+    {
         if (! $this->coalesceAnalyzer->hasIssetableLeft($coalesce)) {
             return null;
         }

src/NodeAnalyzer/CoalesceAnalyzer.php

@@ -25,6 +25,10 @@ final class CoalesceAnalyzer
 
     public function hasIssetableLeft(Coalesce $coalesce): bool
     {
+        if ($coalesce->left instanceof Coalesce) {
+            return true;
+        }
+
         $leftClass = $coalesce->left::class;
         return in_array($leftClass, self::ISSETABLE_EXPR, true);
     }