diff options
| author | 2019-10-02 18:24:45 +0200 | |
|---|---|---|
| committer | 2019-10-02 18:24:45 +0200 | |
| commit | b579d19beae6f92c5a53f110a3031a4e172d3aec (patch) | |
| tree | e184ef9c7b7f2c7d0bf336e6f42db8077bc62e57 /bot | |
| parent | Merge 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')
| -rw-r--r-- | bot/decorators.py | 14 | 
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." | 
