wip

Author: tobyzerner

src/Adapter/AdapterInterface.php

@@ -46,5 +46,5 @@ public function sortByAttribute($query, Attribute $attribute, string $direction)
 
     public function paginate($query, int $limit, int $offset);
 
-    public function load(array $models, array $relationships);
+    public function load(array $models, array $relationships, \Closure $scope);
 }

src/Adapter/EloquentAdapter.php

@@ -5,6 +5,7 @@
 use Illuminate\Database\Eloquent\Collection;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
+use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use Tobscure\JsonApiServer\Schema\Attribute;
 use Tobscure\JsonApiServer\Schema\HasMany;
@@ -185,9 +186,13 @@ public function paginate($query, int $limit, int $offset)
         $query->take($limit)->skip($offset);
     }
 
-    public function load(array $models, array $trail)
+    public function load(array $models, array $trail, \Closure $scope)
     {
-        (new Collection($models))->load($this->relationshipTrailToPath($trail));
+        (new Collection($models))->load([
+            $this->relationshipTrailToPath($trail) => function ($relation) use ($scope) {
+                $scope($relation->getQuery());
+            }
+        ]);
     }
 
     public function loadIds(array $models, Relationship $relationship)
@@ -199,7 +204,7 @@ public function loadIds(array $models, Relationship $relationship)
         $property = $this->getRelationshipProperty($relationship);
         $relation = $models[0]->$property();
 
-        if ($relation instanceof BelongsTo) {
+        if ($relation instanceof BelongsTo || $relation instanceof BelongsToMany) {
             return;
         }
 

src/Handler/Concerns/IncludesData.php

@@ -119,7 +119,13 @@ private function loadRelationships(array $models, array $include, Request $reque
                 // TODO: probably need to loop through relationships here
                 ($loader)($models, false);
             } else {
-                $adapter->load($models, $relationships);
+                $scope = function ($query) use ($relationships, $request) {
+                    foreach ($this->api->getResource(end($relationships)->resource)->getSchema()->scopes as $scope) {
+                        $scope($request, $query);
+                    }
+                };
+
+                $adapter->load($models, $relationships, $scope);
             }
         }
     }

src/Schema/HasMany.php

@@ -12,18 +12,4 @@ public function __construct(string $name)
 
         $this->resource = $name;
     }
-
-    public function includable()
-    {
-        $this->includable = true;
-
-        return $this;
-    }
-
-    public function included()
-    {
-        $this->includable();
-
-        return parent::included();
-    }
 }

src/Schema/Relationship.php

@@ -13,6 +13,7 @@ abstract class Relationship extends Field
     public $loadable = true;
     public $loader;
     public $included = false;
+    public $includable = true;
     public $resource;
 
     public function __construct(string $name)
@@ -78,8 +79,24 @@ public function load(Closure $callback)
         return $this;
     }
 
+    public function includable()
+    {
+        $this->includable = true;
+
+        return $this;
+    }
+
+    public function notIncludable()
+    {
+        $this->includable = false;
+
+        return $this;
+    }
+
     public function included()
     {
+        $this->includable();
+
         $this->included = true;
 
         return $this;