roleypoly/v1
1
0
Fork 0
mirror of https://github.com/roleypoly/roleypoly-v1.git synced 2025-04-24 19:59:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Revert "fix(Servers API): sanitize for edge cases with roles that don't exist anymore"

Browse source 
This reverts commit 18e7e7ceb2.
This commit is contained in:
41666 2020-02-08 01:10:56 -05:00
parent 18e7e7ceb2
commit 951d6c41bf
No known key found for this signature in database
GPG key ID: BC51D07640DC10AF
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Server/api/servers.js
View file

@ -122,13 +122,12 @@ module.exports = (R, $) => {
}
const currentRoles = gm.rolesList
const allRoles = await $.discord.getRoles(server)
const allowedRoles = await $.server.getAllowedRoles(server)
// current roles and allowed roles are an inclusive set.
// first, filter added and removed.
const sanitizedAdded = added.filter(role => allRoles.includes(role) && allowedRoles.includes(role))
const sanitizedRemoved = removed.filter(role => allRoles.includes(role) && allowedRoles.includes(role))
const sanitizedAdded = added.filter(role => allowedRoles.includes(role))
const sanitizedRemoved = removed.filter(role => allowedRoles.includes(role))
// filter currentRoles by what's been removed (down is faster than up)
let newRoles = currentRoles.filter(role => !sanitizedRemoved.includes(role))