From 0aead491dcbb6ad5ece1bf5f8d58341569ef99c5 Mon Sep 17 00:00:00 2001
From: Madeline <46743919+MaddyUnderStars@users.noreply.github.com>
Date: Wed, 21 Dec 2022 15:17:06 +1100
Subject: Fix the roles bug

---
 src/api/routes/guilds/#guild_id/members/#member_id/index.ts | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/api/routes/guilds/#guild_id/members/#member_id/index.ts b/src/api/routes/guilds/#guild_id/members/#member_id/index.ts
index 8e6c3ce7..f1d343d9 100644
--- a/src/api/routes/guilds/#guild_id/members/#member_id/index.ts
+++ b/src/api/routes/guilds/#guild_id/members/#member_id/index.ts
@@ -36,17 +36,20 @@ router.patch("/", route({ body: "MemberChangeSchema" }), async (req: Request, re
 	const permission = await getPermission(req.user_id, guild_id);
 	const everyone = await Role.findOneOrFail({ where: { guild_id: guild_id, name: "@everyone", position: 0 } });
 
-	if (body.roles) {
+	if (body.avatar) body.avatar = await handleFile(`/guilds/${guild_id}/users/${member_id}/avatars`, body.avatar as string);
+
+	member.assign(body);
+
+	if ('roles' in body) {
 		permission.hasThrow("MANAGE_ROLES");
 
+		body.roles = body.roles || [];
+		body.roles.filter(x => !!x);
+
 		if (body.roles.indexOf(everyone.id) === -1) body.roles.push(everyone.id);
 		member.roles = body.roles.map((x) => Role.create({ id: x })); // foreign key constraint will fail if role doesn't exist
 	}
 
-	if (body.avatar) body.avatar = await handleFile(`/guilds/${guild_id}/users/${member_id}/avatars`, body.avatar as string);
-
-	member.assign(body);
-
 	await member.save();
 
 	member.roles = member.roles.filter((x) => x.id !== everyone.id);
-- 
cgit 1.4.1