Use join function for simplicity

ajayyy · Jun 25, 2024 · 374ddc7 · 374ddc7
1 parent 738f863
commit 374ddc7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/routes/postClearCache.ts b/src/routes/postClearCache.ts
@@ -23,7 +23,7 @@ export async function postClearCache(req: Request, res: Response): Promise<Respo
 
     if (invalidFields.length !== 0) {
         // invalid request
-        const fields = invalidFields.reduce((p, c, i) => p + (i !== 0 ? ", " : "") + c, "");
+        const fields = invalidFields.join(", ");
         return res.status(400).send(`No valid ${fields} field(s) provided`);
     }
 

diff --git a/src/routes/postSkipSegments.ts b/src/routes/postSkipSegments.ts
@@ -242,11 +242,11 @@ async function checkInvalidFields(videoID: VideoID, userID: UserID, hashedUserID
 
     if (invalidFields.length !== 0) {
         // invalid request
-        const formattedFields = invalidFields.reduce((p, c, i) => p + (i !== 0 ? ", " : "") + c, "");
-        const formattedErrors = errors.reduce((p, c, i) => p + (i !== 0 ? ". " : " ") + c, "");
+        const formattedFields = invalidFields.join(", ");
+        const formattedErrors = errors.join(". ");
         return {
             pass: false,
-            errorMessage: `No valid ${formattedFields}.${formattedErrors}`,
+            errorMessage: `No valid ${formattedFields}. ${formattedErrors}`,
             errorCode: 400
         };
     }