Arragon/SpaceNinjaServer
1
0
Fork 0
forked from OpenWF/SpaceNinjaServer
Code Pull Requests Activity

fix: use deleteMany for models where accountId is not unique when deleting account (#1290)

Browse Source 
Reviewed-on: OpenWF/SpaceNinjaServer#1290
This commit is contained in:
Sainan 2025-03-22 18:12:59 -07:00
parent 5a56c2e9d3
commit 5817b48db9
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/controllers/custom/deleteAccountController.ts
View File

@ -14,12 +14,12 @@ export const deleteAccountController: RequestHandler = async (req, res) => {
// TODO: Handle the account being the creator of a guild // TODO: Handle the account being the creator of a guild
await Promise.all([ await Promise.all([
Account.deleteOne({ _id: accountId }), Account.deleteOne({ _id: accountId }),
GuildMember.deleteOne({ accountId: accountId }), GuildMember.deleteMany({ accountId: accountId }),
Inbox.deleteMany({ ownerId: accountId }), Inbox.deleteMany({ ownerId: accountId }),
Inventory.deleteOne({ accountOwnerId: accountId }), Inventory.deleteOne({ accountOwnerId: accountId }),
Loadout.deleteOne({ loadoutOwnerId: accountId }), Loadout.deleteOne({ loadoutOwnerId: accountId }),
PersonalRooms.deleteOne({ personalRoomsOwnerId: accountId }), PersonalRooms.deleteOne({ personalRoomsOwnerId: accountId }),
Ship.deleteOne({ ShipOwnerId: accountId }), Ship.deleteMany({ ShipOwnerId: accountId }),
Stats.deleteOne({ accountOwnerId: accountId }) Stats.deleteOne({ accountOwnerId: accountId })
]); ]);
res.end(); res.end();