fix(laravel): graphql fix relationship loading (#6857)

fix loading relationships by only loading those related to the model

Closes: #6791

Signed-off-by: Tobias Oitzinger <[email protected]>

Author: toitzi

Eloquent/Paginator.php

@@ -13,6 +13,7 @@
 
 namespace ApiPlatform\Laravel\Eloquent;
 
+use ApiPlatform\State\Pagination\HasNextPagePaginatorInterface;
 use ApiPlatform\State\Pagination\PaginatorInterface;
 use Illuminate\Pagination\LengthAwarePaginator;
 use IteratorAggregate;
@@ -21,7 +22,7 @@
  * @implements IteratorAggregate<mixed,object>
  * @implements PaginatorInterface<object>
  */
-final class Paginator implements PaginatorInterface, \IteratorAggregate
+final class Paginator implements PaginatorInterface, HasNextPagePaginatorInterface, \IteratorAggregate
 {
     /**
      * @param LengthAwarePaginator<object> $paginator
@@ -60,4 +61,9 @@ public function getIterator(): \Traversable
     {
         return $this->paginator->getIterator();
     }
+
+    public function hasNextPage(): bool
+    {
+        return $this->paginator->hasMorePages();
+    }
 }

Tests/GraphQlTest.php

@@ -48,6 +48,35 @@ public function testGetBooks(): void
         $this->assertArrayNotHasKey('errors', $data);
     }
 
+    public function testGetBooksWithSimplePagination(): void
+    {
+        BookFactory::new()->has(AuthorFactory::new())->count(9)->create();
+        $response = $this->postJson('/api/graphql', ['query' => '{
+  simplePaginationBooks(page: 1) {
+    collection {
+        id
+     },
+    paginationInfo {
+        itemsPerPage,
+        currentPage,
+        lastPage,
+        totalCount,
+        hasNextPage
+     }
+  }
+}'], ['accept' => ['application/json']]);
+        $response->assertStatus(200);
+        $data = $response->json();
+        $this->assertArrayHasKey('data', $data);
+        $this->assertCount(3, $data['data']['simplePaginationBooks']['collection']);
+        $this->assertEquals(3, $data['data']['simplePaginationBooks']['paginationInfo']['itemsPerPage']);
+        $this->assertEquals(1, $data['data']['simplePaginationBooks']['paginationInfo']['currentPage']);
+        $this->assertEquals(3, $data['data']['simplePaginationBooks']['paginationInfo']['lastPage']);
+        $this->assertEquals(9, $data['data']['simplePaginationBooks']['paginationInfo']['totalCount']);
+        $this->assertTrue($data['data']['simplePaginationBooks']['paginationInfo']['hasNextPage']);
+        $this->assertArrayNotHasKey('errors', $data);
+    }
+
     public function testGetBooksWithPaginationAndOrder(): void
     {
         BookFactory::new()->has(AuthorFactory::new())->count(10)->create();

workbench/app/Models/Book.php

@@ -56,6 +56,11 @@
                 new QueryParameter(key: 'order[:property]', filter: OrderFilter::class),
             ],
         ),
+        new QueryCollection(
+            paginationItemsPerPage: 3,
+            name: 'simplePagination',
+            paginationType: 'page',
+        ),
         new Mutation(name: 'create'),
     ]
 )]