Fix casting params for page and maxUsersOnPage

Karol-2 · Mar 25, 2024 · 10fcad0 · 10fcad0
1 parent c8642bf
commit 10fcad0
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/backend/src/routes/usersFriendsRoute.ts b/backend/src/routes/usersFriendsRoute.ts
@@ -39,8 +39,10 @@ friendshipRouter.get(
     try {
       const session = driver.session();
       const userId = req.params.userId;
-      const page: number = parseInt(req.query.page as string);
-      const maxUsersOnPage: number = parseInt(req.query.maxUsers as string);
+      const page: number = parseInt((req.query.page as string) || "");
+      const maxUsersOnPage: number = parseInt(
+        (req.query.maxUsers as string) || "",
+      );
 
       const user = await userExists(session, res, userId);
       if ("json" in user) {
@@ -142,9 +144,10 @@ friendshipRouter.get(
     try {
       const session: Session = driver.session();
       const userId: string = req.params.userId;
-      const page: number = parseInt(req.query.page as string);
-      const maxUsersOnPage: number = parseInt(req.query.maxUsers as string);
-
+      const page: number = parseInt((req.query.page as string) || "");
+      const maxUsersOnPage: number = parseInt(
+        (req.query.maxUsers as string) || "",
+      );
       const user = await userExists(session, res, userId);
       if ("json" in user) {
         await session.close();

diff --git a/backend/src/routes/usersRoute.ts b/backend/src/routes/usersRoute.ts
@@ -90,8 +90,10 @@ usersRouter.get(
     }
 
     try {
-      const page: number = parseInt(req.query.page as string);
-      const maxUsersOnPage: number = parseInt(req.query.maxUsers as string);
+      const page: number = parseInt((req.query.page as string) || "");
+      const maxUsersOnPage: number = parseInt(
+        (req.query.maxUsers as string) || "",
+      );
       const session = driver.session();
       let allUsers: User[] = [];