Merge pull request #25 from CodeWithDennis/23-nullvalue

Fixed an issue where tree would go into inf loop
CodeWithDennis · Sep 26, 2023 · a7c6223 · a7c6223
2 parents eb9424b + c85ad1b
commit a7c6223
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/SelectTree.php b/src/SelectTree.php
@@ -84,7 +84,9 @@ protected function setUp(): void
     private function buildTree($parent = null): array|Collection
     {
         // Assign the parent's null value to the $parent variable if it's not null
-        $parent = $this->getParentNullValue() ?? $parent;
+        if ($parent == null || $parent == $this->getParentNullValue()) {
+            $parent = $this->getParentNullValue() ?? $parent;
+        }
 
         // Create a default query to retrieve related items.
         $defaultQuery = $this->getRelationship()