Merge pull request #176 from whiteoctober/fix/158-first-page

Fix: 'Problem with "omitFirstPage: true" - can't navigate to first page.'
sampart · Nov 30, 2017 · 9b1f3d4 · 9b1f3d4
2 parents 5fd47a5 + a252e1d
commit 9b1f3d4
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/Twig/PagerfantaExtension.php b/Twig/PagerfantaExtension.php
@@ -140,8 +140,9 @@ private function createRouteGenerator($options = array())
 
         return function($page) use($router, $routeName, $routeParams, $pagePropertyPath, $omitFirstPage) {
             $propertyAccessor = PropertyAccess::createPropertyAccessor();
-            // don't set the page parameter if it is the first page and the omitFirstPage flag is true
-            if(!$omitFirstPage || $page > 1 ){
+            if($omitFirstPage){
+                $propertyAccessor->setValue($routeParams, $pagePropertyPath, $page > 1 ? $page : null);
+            } else {
                 $propertyAccessor->setValue($routeParams, $pagePropertyPath, $page);
             }