fix: Role#setPermissions resolving & docs

This commit is contained in:
Lewdcario
2018-04-18 19:57:16 -05:00
parent 92b421607e
commit 7c0b6173dd
3 changed files with 3 additions and 3 deletions

View File

@@ -632,7 +632,7 @@ class RESTMethods {
data.hoist = typeof _data.hoist !== 'undefined' ? _data.hoist : role.hoist;
data.mentionable = typeof _data.mentionable !== 'undefined' ? _data.mentionable : role.mentionable;
if (_data.permissions) data.permissions = Permissions.resolve(_data.permissions);
if (typeof _data.permissions !== 'undefined') data.permissions = Permissions.resolve(_data.permissions);
else data.permissions = role.permissions;
return this.rest.makeRequest('patch', Endpoints.Guild(role.guild).Role(role.id), true, data, undefined, reason)

View File

@@ -276,7 +276,7 @@ class Role {
/**
* Set the permissions of the role.
* @param {string[]} permissions The permissions of the role
* @param {PermissionResolvable|PermissionResolvable[]} permissions The permissions of the role
* @param {string} [reason] Reason for changing the role's permissions
* @returns {Promise<Role>}
* @example

View File

@@ -165,7 +165,7 @@ class Permissions {
static resolve(permission) {
if (permission instanceof Array) return permission.map(p => this.resolve(p)).reduce((prev, p) => prev | p, 0);
if (typeof permission === 'string') permission = this.FLAGS[permission];
if (typeof permission !== 'number' || permission < 1) throw new RangeError(Constants.Errors.NOT_A_PERMISSION);
if (typeof permission !== 'number' || permission < 0) throw new RangeError(Constants.Errors.NOT_A_PERMISSION);
return permission;
}
}