fix(PagerFactory): pager should not be lazy, so that if the page do…

…es not exist, it will throw an exception immediately, not inside template.
rekalogika · Jun 8, 2024 · 53ea299 · 53ea299
1 parent 9e49188
commit 53ea299
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+# 0.9.2
+
+* fix(`PagerFactory`): pager should not be lazy, so that if the page does not
+  exist, it will throw an exception immediately, not inside template.
+
 # 0.9.1
 
 * fix(`QueryBuilderAdapter`): use generated field names for our boundary fields

diff --git a/packages/rekapager-bundle/src/PagerFactory.php b/packages/rekapager-bundle/src/PagerFactory.php
@@ -96,7 +96,6 @@ public function createPager(
         $queryParams = $request->query->all();
         $routeParams = array_merge($queryParams, $routeParams);
 
-
         /** @var array<string,string|int> $routeParams */
 
         $pageUrlGenerator = $this->pageUrlGeneratorFactory->createPageUrlGenerator(
@@ -106,13 +105,17 @@ public function createPager(
             routeParams: $routeParams
         );
 
-        return new Pager(
+        $pager = new Pager(
             page: $page,
             proximity: $proximity,
             pageLimit: $pageLimit,
             pageUrlGenerator: $pageUrlGenerator,
             pageIdentifierEncoderLocator: $this->pageIdentifierEncoderLocator,
         );
+
+        foreach ($pager->getCurrentPage() as $i);
+
+        return $pager;
     }
 
     /**