fix: HasMany - hookResults param is ignored

SoftwarePunt · Jan 20, 2024 · 05f746a · 05f746a
1 parent 02eebb6
commit 05f746a
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/Relationships/HasManyRelationship.php b/lib/Relationships/HasManyRelationship.php
@@ -58,10 +58,12 @@ public function query(bool $hookResults = true): ModelQuery
         $query = $this->referenceModel::query()
             ->where("{$this->foreignKeyColumn} = ?", $pkVal);
 
-        $query->addResultHook(function (Model $model) {
-            $pkVal = $model->getPrimaryKeyValue();
-            $this->loadedModels[$pkVal] = $model;
-        });
+        if ($hookResults) {
+            $query->addResultHook(function (Model $model) {
+                $pkVal = $model->getPrimaryKeyValue();
+                $this->loadedModels[$pkVal] = $model;
+            });
+        }
 
         return $query;
     }