[Php80] Handle crash leaveNode() returned invalid value of type integ…

…er on TokenGetAllToObjectRector (#3291)

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

rules-tests/Php80/Rector/FuncCall/TokenGetAllToObjectRector/Fixture/ternary_if.php.inc

@@ -0,0 +1,35 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\FuncCall\TokenGetAllToObjectRector\Fixture;
+
+final class TernaryIf
+{
+    public function run()
+    {
+        $tokens = token_get_all('<?php ' . implode('', array_slice($fileContent, $start, $end - $start)));
+
+        foreach ($tokens as $i => $token) {
+            $readableToken = is_array($token) ? $token[1] : $token;
+        }
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\FuncCall\TokenGetAllToObjectRector\Fixture;
+
+final class TernaryIf
+{
+    public function run()
+    {
+        $tokens = \PhpToken::tokenize('<?php ' . implode('', array_slice($fileContent, $start, $end - $start)));
+
+        foreach ($tokens as $i => $token) {
+            $readableToken = $token->text;
+        }
+    }
+}
+
+?>

rules/Php80/NodeManipulator/TokenManipulator.php

@@ -15,10 +15,12 @@
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Expr\PropertyFetch;
+use PhpParser\Node\Expr\Ternary;
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Stmt\If_;
 use Rector\Core\NodeAnalyzer\ArgsAnalyzer;
 use Rector\Core\PhpParser\Comparing\NodeComparator;
+use Rector\Core\PhpParser\Node\BetterNodeFinder;
 use Rector\Core\PhpParser\Node\Value\ValueResolver;
 use Rector\Core\Util\StringUtils;
 use Rector\NodeNameResolver\NodeNameResolver;
@@ -39,7 +41,8 @@ public function __construct(
         private readonly NodesToRemoveCollector $nodesToRemoveCollector,
         private readonly ValueResolver $valueResolver,
         private readonly NodeComparator $nodeComparator,
-        private readonly ArgsAnalyzer $argsAnalyzer
+        private readonly ArgsAnalyzer $argsAnalyzer,
+        private readonly BetterNodeFinder $betterNodeFinder
     ) {
     }
 
@@ -176,7 +179,7 @@ public function removeIsArray(array $nodes, Variable $singleTokenVariable): void
     {
         $this->simpleCallableNodeTraverser->traverseNodesWithCallable($nodes, function (Node $node) use (
             $singleTokenVariable
-        ) {
+        ): ?FuncCall {
             if (! $node instanceof FuncCall) {
                 return null;
             }
@@ -202,10 +205,34 @@ public function removeIsArray(array $nodes, Variable $singleTokenVariable): void
             // remove correct node
             $nodeToRemove = $this->matchParentNodeInCaseOfIdenticalTrue($node);
 
+            $parentNode = $nodeToRemove->getAttribute(AttributeKey::PARENT_NODE);
+
+            if ($parentNode instanceof Ternary) {
+                $this->replaceTernary($parentNode);
+                return $node;
+            }
+
             $this->nodesToRemoveCollector->addNodeToRemove($nodeToRemove);
+            return $node;
         });
     }
 
+    private function replaceTernary(Ternary $ternary): void
+    {
+        $currentStmt = $this->betterNodeFinder->resolveCurrentStatement($ternary);
+
+        $this->simpleCallableNodeTraverser->traverseNodesWithCallable(
+            $currentStmt,
+            static function (Node $subNode) use ($ternary): ?Expr {
+                if ($subNode === $ternary) {
+                    return $ternary->if;
+                }
+
+                return null;
+            }
+        );
+    }
+
     /**
      * Replace $token[0] with $token->getTokenName() call
      *