Nomchy/discord.js
1
0
Fork 0
mirror of https://github.com/discordjs/discord.js.git synced 2026-03-16 19:43:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fixed manageRoles bug in lib. Fixes #507 (#570)

Browse Source 
* ChannelPermissions changed manageRoles to administrator. fixes #507

* fixed manageRoles bug inside lib/ChannelPermissions. fixes #507
fixed related comments inside Role.
This commit is contained in:
Tain101
2016-08-30 19:40:29 -04:00
committed by abal
parent ee64253f0b
commit 0adfc0fe95
3 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/Structures/ChannelPermissions.js
View File

@@ -45,8 +45,8 @@ var ChannelPermissions = (function () {
} }
if (!explicit) { if (!explicit) {
// implicit permissions allowed // implicit permissions allowed
if (!!(this.permissions & _Constants.Permissions.manageRoles)) { if (!!(this.permissions & _Constants.Permissions.administrator)) {
// manageRoles allowed, they have all permissions // administrator allowed, they have all permissions
return true; return true;
} }
} }

2
lib/Structures/Role.js
View File

@@ -118,7 +118,7 @@ var Role = (function () {
if (!explicit) { if (!explicit) {
// implicit permissions allowed // implicit permissions allowed
if (!!(this.permissions & _Constants.Permissions.administrator)) { if (!!(this.permissions & _Constants.Permissions.administrator)) {
// manageRoles allowed, they have all permissions // administrator allowed, they have all permissions
return true; return true;
} }
} }

4
src/Structures/Role.js
View File

@@ -54,7 +54,7 @@ export default class Role {
for (let k of keys) { for (let k of keys) {
obj[k] = this[k]; obj[k] = this[k];
} }
return obj; return obj;
} }
@@ -104,7 +104,7 @@ export default class Role {
} }
if(!explicit){ // implicit permissions allowed if(!explicit){ // implicit permissions allowed
if( !!(this.permissions & Permissions.administrator) ){ if( !!(this.permissions & Permissions.administrator) ){
// manageRoles allowed, they have all permissions // administrator allowed, they have all permissions
return true; return true;
} }
} }