add checks for presence of user/role perms before checking
parent
0633564e6a
commit
9e5c9da6e2
|
@ -16,12 +16,21 @@ export default class {
|
||||||
const resolved: Command = this.client.util.resolveCommand(command);
|
const resolved: Command = this.client.util.resolveCommand(command);
|
||||||
if (!resolved) return;
|
if (!resolved) return;
|
||||||
if (resolved.guildOnly && !(message.channel instanceof TextChannel)) return;
|
if (resolved.guildOnly && !(message.channel instanceof TextChannel)) return;
|
||||||
const hasUserPerms: boolean = resolved.permissions.users.includes(message.author.id);
|
let hasUserPerms: boolean;
|
||||||
|
if (resolved.permissions.users) {
|
||||||
|
hasUserPerms = resolved.permissions.users.includes(message.author.id);
|
||||||
|
} else {
|
||||||
|
hasUserPerms = true;
|
||||||
|
}
|
||||||
let hasRolePerms: boolean = false;
|
let hasRolePerms: boolean = false;
|
||||||
for (const role of resolved.permissions.roles) {
|
if (resolved.permissions.roles) {
|
||||||
if (message.member && message.member.roles.includes(role)) {
|
for (const role of resolved.permissions.roles) {
|
||||||
hasRolePerms = true; break;
|
if (message.member && message.member.roles.includes(role)) {
|
||||||
|
hasRolePerms = true; break;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
} else {
|
||||||
|
hasRolePerms = true;
|
||||||
}
|
}
|
||||||
if (!hasRolePerms && !hasUserPerms) return;
|
if (!hasRolePerms && !hasUserPerms) return;
|
||||||
if (!resolved.enabled) { message.channel.createMessage(`***${this.client.stores.emojis.error} This command has been disabled***`); return; }
|
if (!resolved.enabled) { message.channel.createMessage(`***${this.client.stores.emojis.error} This command has been disabled***`); return; }
|
||||||
|
|
Loading…
Reference in New Issue