dan/discord.js
1
0
Fork 0
mirror of https://github.com/danbulant/discord.js synced 2026-06-09 09:42:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix addRole/removeRole when the user has/doesn't have the role (#1288)

Browse source
This commit is contained in:
Programmix 2017-03-20 22:52:43 -07:00 committed by Schuyler Cebulskie
parent 2e54f6d90d
commit 633e3ca896
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/client/rest/RESTMethods.js
View file

@ -501,6 +501,11 @@ class RESTMethods {
addMemberRole(member, role) { addMemberRole(member, role) {
return new Promise(resolve => { return new Promise(resolve => {
if (member._roles.includes(role.id)) {
resolve(member);
return;
}
const listener = (oldMember, newMember) => { const listener = (oldMember, newMember) => {
if (!oldMember._roles.includes(role.id) && newMember._roles.includes(role.id)) { if (!oldMember._roles.includes(role.id) && newMember._roles.includes(role.id)) {
this.client.removeListener('guildMemberUpdate', listener); this.client.removeListener('guildMemberUpdate', listener);
@ -521,6 +526,11 @@ class RESTMethods {
removeMemberRole(member, role) { removeMemberRole(member, role) {
return new Promise(resolve => { return new Promise(resolve => {
if (!member._roles.includes(role.id)) {
resolve(member);
return;
}
const listener = (oldMember, newMember) => { const listener = (oldMember, newMember) => {
if (oldMember._roles.includes(role.id) && !newMember._roles.includes(role.id)) { if (oldMember._roles.includes(role.id) && !newMember._roles.includes(role.id)) {
this.client.removeListener('guildMemberUpdate', listener); this.client.removeListener('guildMemberUpdate', listener);