Merge pull request #137 from devalv/bugfix/default-page-number

last page number fix

Author: uriyyo

fastapi_pagination/links/default.py

@@ -21,7 +21,7 @@ def __root_validator__(cls, value: Any) -> Any:
 
             value["links"] = create_links(
                 first={"page": 1},
-                last={"page": ceil(total / size) + 1},
+                last={"page": ceil(total / size) if total > 0 else 1},
                 next={"page": page + 1} if page * size < total else None,
                 prev={"page": page - 1} if 1 <= page - 1 else None,
             )

tests/test_links.py

@@ -31,21 +31,21 @@ async def route():
             None,
             "/default?page=2",
             "/default?page=1",
-            "/default?page=5",
+            "/default?page=4",
         ),
         (
             "/default?page=2",
             "/default?page=1",
             "/default?page=3",
             "/default?page=1",
-            "/default?page=5",
+            "/default?page=4",
         ),
         (
-            "/default?page=5",
             "/default?page=4",
+            "/default?page=3",
             None,
             "/default?page=1",
-            "/default?page=5",
+            "/default?page=4",
         ),
         (
             "/default-empty",