chore: apply to other list states

flarum · Nov 3, 2023 · 72eded7 · 72eded7
1 parent 61d1dc3
commit 72eded7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/framework/core/js/src/forum/states/NotificationListState.ts b/framework/core/js/src/forum/states/NotificationListState.ts
@@ -4,7 +4,7 @@ import Notification from '../../common/models/Notification';
 
 export default class NotificationListState extends PaginatedListState<Notification> {
   constructor() {
-    super({}, 1, 20);
+    super({}, 1, null);
   }
 
   get type(): string {

diff --git a/framework/core/src/Api/Controller/AbstractSerializeController.php b/framework/core/src/Api/Controller/AbstractSerializeController.php
@@ -55,7 +55,7 @@ abstract class AbstractSerializeController implements RequestHandlerInterface
     /**
      * The number of records included by default.
      */
-    public int $limit = 20;
+    public int $limit = 40;
 
     /**
      * The fields that are available to be sorted by.

diff --git a/framework/core/src/Api/Controller/ListNotificationsController.php b/framework/core/src/Api/Controller/ListNotificationsController.php
@@ -62,11 +62,10 @@ protected function data(ServerRequestInterface $request, Document $document): it
             $areMoreResults = true;
         }
 
-        $document->addPaginationLinks(
+        $this->addPaginationData(
+            $document,
+            $request,
             $this->url->to('api')->route('notifications.index'),
-            $request->getQueryParams(),
-            $offset,
-            $limit,
             $areMoreResults ? null : 0
         );