aboutsummaryrefslogtreecommitdiffstats
path: root/bot/decorators.py
diff options
context:
space:
mode:
authorGravatar Numerlor <[email protected]>2019-10-02 18:24:45 +0200
committerGravatar Numerlor <[email protected]>2019-10-02 18:24:45 +0200
commitb579d19beae6f92c5a53f110a3031a4e172d3aec (patch)
treee184ef9c7b7f2c7d0bf336e6f42db8077bc62e57 /bot/decorators.py
parentMerge pull request #285 from Numerlor/hacktober-date-channel-fixes (diff)
put role bypass check above channel override to make sure bypass roles can activate commands everywhere even with override channels specified
Diffstat (limited to 'bot/decorators.py')
-rw-r--r--bot/decorators.py14
1 files changed, 7 insertions, 7 deletions
diff --git a/bot/decorators.py b/bot/decorators.py
index 2c042b56..58f67a15 100644
--- a/bot/decorators.py
+++ b/bot/decorators.py
@@ -81,6 +81,13 @@ def in_channel_check(*channels: int, bypass_roles: typing.Container[int] = None)
)
return True
+ if bypass_roles and any(r.id in bypass_roles for r in ctx.author.roles):
+ log.debug(
+ f"{ctx.author} called the '{ctx.command.name}' command and "
+ f"had a role to bypass the in_channel check."
+ )
+ return True
+
if hasattr(ctx.command.callback, "in_channel_override"):
override = ctx.command.callback.in_channel_override
if override is None:
@@ -106,13 +113,6 @@ def in_channel_check(*channels: int, bypass_roles: typing.Container[int] = None)
f"Sorry, but you may only use this command within {channels_str}."
)
- if bypass_roles and any(r.id in bypass_roles for r in ctx.author.roles):
- log.debug(
- f"{ctx.author} called the '{ctx.command.name}' command and "
- f"had a role to bypass the in_channel check."
- )
- return True
-
log.debug(
f"{ctx.author} tried to call the '{ctx.command.name}' command. "
f"The in_channel check failed."