added optional user_id field to edit bag route to validate that user does not already have a bag of the given name

Author: cdleveille

src/models/bag.ts

@@ -69,8 +69,12 @@ const removeDiscFromBag = async (id: string, disc_id: string) => {
 	return bag;
 };
 
-const updateBagName = async (id: string, name: string) => {
+const updateBagName = async (id: string, name: string, user_id?: string) => {
 	const bag = await assertBagExists(id);
+	if (user_id) {
+		const existingBag = await BagModel.findOne({ user_id, name });
+		if (existingBag) throw new CustomError(`You already have a bag named '${name}'`, 400);
+	}
 	bag.name = name;
 	await BagModel.updateOne({ id }, bag);
 	return bag;

src/routes/bag.ts

@@ -225,7 +225,11 @@ export const initBagRoutes = (app: OpenAPIHono) => {
 				body: {
 					content: {
 						"application/json": {
-							schema: z.object({ id: z.string(), name: z.string() })
+							schema: z.object({
+								id: z.string(),
+								name: z.string(),
+								user_id: z.optional(z.string())
+							})
 						}
 					},
 					required: true
@@ -257,8 +261,8 @@ export const initBagRoutes = (app: OpenAPIHono) => {
 		async c => {
 			const { authorization } = c.req.valid("header");
 			assertIsRequestAuthorized(authorization);
-			const { id, name } = c.req.valid("json");
-			const bag = await Bag.updateBagName(id, name);
+			const { id, name, user_id } = c.req.valid("json");
+			const bag = await Bag.updateBagName(id, name, user_id);
 			return c.json(bag, 200);
 		}
 	);