Protects against null root node, and ensures instance of Array_

Stillat · Apr 10, 2021 · 299938a · 299938a
1 parent df0a140
commit 299938a
Showing 1 changed file with 13 additions and 12 deletions.
diff --git a/src/Analyzers/ConfigAnalyzer.php b/src/Analyzers/ConfigAnalyzer.php
@@ -472,7 +472,6 @@ public function removeNode($key)
         if ($currentNode instanceof ArrayItem) {
             $parent = $this->getParentKey($key);
 
-
             if (mb_strlen($parent) > 0 && $this->hasNode($parent)) {
                 $parentNode = $this->nodeMapping[$parent];
                 $relativeKey = $this->getLastKeySegment($key);
@@ -495,21 +494,23 @@ public function removeNode($key)
                     $parentNode->value->items = $newItems;
                 }
             } else {
-                $newItems = [];
+                if ($this->rootNode != null && $this->rootNode instanceof Array_) {
+                    $newItems = [];
 
-                /** @var ArrayItem $valueItem */
-                foreach ($this->rootNode->items as $valueItem) {
-                    $valueItemKey = $valueItem->key->value;
+                    /** @var ArrayItem $valueItem */
+                    foreach ($this->rootNode->items as $valueItem) {
+                        $valueItemKey = $valueItem->key->value;
 
-                    if ($valueItemKey === null || $valueItemKey !== $key) {
-                        $newItems[] = $valueItem;
-                    } else {
-                        $foundNode = true;
+                        if ($valueItemKey === null || $valueItemKey !== $key) {
+                            $newItems[] = $valueItem;
+                        } else {
+                            $foundNode = true;
+                        }
                     }
-                }
 
-                // Reassign the value items, without the node to remove.
-                $this->rootNode->items = $newItems;
+                    // Reassign the value items, without the node to remove.
+                    $this->rootNode->items = $newItems;
+                }
             }
         }