make sure the getCurrentPage does not return anything below 1

curious-leaf · Apr 2, 2024 · e6019e7 · e6019e7
1 parent 9ab3a37
commit e6019e7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@curiousleaf/utils",
   "description": "A lightweight set of utilities",
-  "version": "1.0.21",
+  "version": "1.0.22",
   "license": "MIT",
   "type": "module",
   "author": {

diff --git a/src/params/params.test.ts b/src/params/params.test.ts
@@ -20,6 +20,12 @@ describe("getCurrentPage", () => {
 
     expect(currentPage).toBe(1)
   })
+
+  it("returns 1 if the provided page is less than 1", () => {
+    const currentPage = getCurrentPage("0")
+
+    expect(currentPage).toBe(1)
+  })
 })
 
 describe("getPageLink", () => {

diff --git a/src/params/params.ts b/src/params/params.ts
@@ -26,7 +26,7 @@ export const getSearchParams = (request: Request) => {
  * @returns The current page number as a number.
  */
 export const getCurrentPage = (page?: string | null) => {
-  return page && !Number.isNaN(Number(page)) ? parseInt(page || "1", 10) : 1
+  return Math.max(page && !Number.isNaN(Number(page)) ? parseInt(page || "1", 10) : 1, 1)
 }
 
 /**