Update CanEagerLoadTrait.php

apiato · Dec 3, 2024 · 22376a0 · 22376a0
1 parent e774882
commit 22376a0
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/Traits/CanEagerLoadTrait.php b/src/Traits/CanEagerLoadTrait.php
@@ -23,6 +23,8 @@ protected function eagerLoadRequestedRelations(): void
         $this->scopeQuery(function (Builder|Model $model) {
             if (request()?->has(config('fractal.auto_includes.request_key'))) {
                 $validIncludes = [];
+                // TODO: Do we need to do the same for the excludes?
+                // TODO: Or default includes! Are they eager loaded by default?
                 foreach (Response::getRequestedIncludesAsModelRelation() as $includeName) {
                     $relationParts = explode('.', $includeName);
                     $camelCasedIncludeName = $this->filterInvalidRelations($this->model, $relationParts);