fix: correctly parse pageNumber in pagination logic

Author: pankajtyagi1601

controllers/discordactions.js

@@ -128,7 +128,8 @@ const getPaginatedAllGroupRoles = async (req, res) => {
   try {
     const { page = 0, size = 10, dev } = req.query;
     const limit = Number.parseInt(size, 10) || 10;
-    const offset = Number.parseInt(page, 10) * limit;
+    const pageNumber = Number.isNaN(Number.parseInt(page, 10)) ? 0 : Number.parseInt(page, 10);
+    const offset = pageNumber * limit;
 
     if (limit < 1 || limit > 100) {
       return res.boom.badRequest("Invalid size. Must be between 1 and 100.");
@@ -139,8 +140,8 @@ const getPaginatedAllGroupRoles = async (req, res) => {
       const { roles, total } = await discordRolesModel.getPaginatedGroupRolesByPage({ offset, limit });
       const groupsWithMembershipInfo = await discordRolesModel.enrichGroupDataWithMembershipInfo(discordId, roles);
 
-      const nextPage = offset + limit < total ? Number.parseInt(page, 10) + 1 : null;
-      const prevPage = page > 0 ? Number.parseInt(page, 10) - 1 : null;
+      const nextPage = offset + limit < total ? pageNumber + 1 : null;
+      const prevPage = pageNumber > 0 ? pageNumber - 1 : null;
 
       const baseUrl = `${req.baseUrl}${req.path}`;
       const next = nextPage !== null ? `${baseUrl}?page=${nextPage}&size=${limit}&dev=true` : null;

controllers/progresses.js

@@ -116,10 +116,11 @@ const getProgress = async (req, res) => {
   try {
     if (dev === "true") {
       const { progressDocs, totalProgressCount } = await progressesModel.getPaginatedProgressDocument(req.query);
-      const limit = Number.parseInt(size, 10);
-      const offset = Number.parseInt(page, 10) * limit;
-      const nextPage = offset + limit < totalProgressCount ? Number.parseInt(page, 10) + 1 : null;
-      const prevPage = page > 0 ? Number.parseInt(page, 10) - 1 : null;
+      const limit = Number.parseInt(size, 10) || 100;
+      const pageNumber = Number.isNaN(Number.parseInt(page, 10)) ? 0 : Number.parseInt(page, 10);
+      const offset = pageNumber * limit;
+      const nextPage = offset + limit < totalProgressCount ? pageNumber + 1 : null;
+      const prevPage = pageNumber > 0 ? pageNumber - 1 : null;
       let baseUrl = `${req.baseUrl}`;
       if (type) {
         baseUrl += `?type=${type}`;