Skip to content

Commit 374ddc7

Browse files
committed
Use join function for simplicity
1 parent 738f863 commit 374ddc7

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/routes/postClearCache.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ export async function postClearCache(req: Request, res: Response): Promise<Respo
2323

2424
if (invalidFields.length !== 0) {
2525
// invalid request
26-
const fields = invalidFields.reduce((p, c, i) => p + (i !== 0 ? ", " : "") + c, "");
26+
const fields = invalidFields.join(", ");
2727
return res.status(400).send(`No valid ${fields} field(s) provided`);
2828
}
2929

src/routes/postSkipSegments.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -242,11 +242,11 @@ async function checkInvalidFields(videoID: VideoID, userID: UserID, hashedUserID
242242

243243
if (invalidFields.length !== 0) {
244244
// invalid request
245-
const formattedFields = invalidFields.reduce((p, c, i) => p + (i !== 0 ? ", " : "") + c, "");
246-
const formattedErrors = errors.reduce((p, c, i) => p + (i !== 0 ? ". " : " ") + c, "");
245+
const formattedFields = invalidFields.join(", ");
246+
const formattedErrors = errors.join(". ");
247247
return {
248248
pass: false,
249-
errorMessage: `No valid ${formattedFields}.${formattedErrors}`,
249+
errorMessage: `No valid ${formattedFields}. ${formattedErrors}`,
250250
errorCode: 400
251251
};
252252
}

0 commit comments

Comments
 (0)