Fix

Author: samsonasik

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

@@ -20,7 +20,8 @@ class MultipleCoalesce
 {
     public function run($content)
     {
-        if (($contentId = $content->getInternalId() ?? $content->getId()) === null) {
+        $contentId = $content->getInternalId() ?? $content->getId();
+        if ($contentId === null) {
             throw new \LogicException('content id is required for ackVersion');
         }
     }

rules/DowngradePhp80/Rector/Expression/DowngradeThrowExprRector.php

@@ -202,7 +202,7 @@ private function refactorCoalesce(Coalesce $coalesce, ?Assign $assign): If_|null
     /**
      * @return If_|Stmt[]|null
      */
-    private function processCoalesce(Coalesce $coalesce, ?Assign $assign, bool $innerAssign = false): If_|null|array
+    private function processCoalesce(Coalesce $coalesce, ?Assign $assign, bool $assignEarly = false): If_|null|array
     {
         if (! $this->coalesceAnalyzer->hasIssetableLeft($coalesce)) {
             return null;
@@ -219,9 +219,11 @@ private function processCoalesce(Coalesce $coalesce, ?Assign $assign, bool $inne
 
         $assign->expr = $coalesce->left;
 
-        if ($innerAssign && $if->cond instanceof Identical) {
-            $if->cond->left = new Assign($assign->var, $if->cond->left);
-            return $if;
+        if ($assignEarly && $if->cond instanceof Identical) {
+            $expression = new Expression(new Assign($assign->var, $if->cond->left));
+            $if->cond->left = $assign->var;
+
+            return [$expression, $if];
         }
 
         return [$if, new Expression($assign)];