diff options
| author |  Manuel Ignacio Pérez Alcolea <[email protected]> | 2019-12-09 02:37:30 -0300 |
|---|---|---|
| committer | Manuel Ignacio Pérez Alcolea <[email protected]> | 2019-12-09 02:37:30 -0300 |
| commit | 34bac05ccc6c11ea370aa14431e4d6d6cd28f1d6 (patch) | |
| tree | 4cf3a079a387b5263c5850f15c1c59ee97f7f75a | |
| parent | Added optional channel parameter to !echo: (diff) | |
Ensure hidden_channels and bypass_roles use a list when not passed.
The in_channel decorator raised 'NoneType' is not iterable when it
wasn't passed, due to the default value being None but not checked
against before iterating over it. This edit ensures the arguments are
set to an empty list in cases where they have a value of None instead.
| -rw-r--r-- | bot/decorators.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/bot/decorators.py b/bot/decorators.py index 61587f406..2d18eaa6a 100644 --- a/bot/decorators.py +++ b/bot/decorators.py @@ -38,6 +38,9 @@ def in_channel( Hidden channels are channels which will not be displayed in the InChannelCheckFailure error message. """ + hidden_channels = hidden_channels or [] + bypass_roles = bypass_roles or [] + def predicate(ctx: Context) -> bool: """In-channel checker predicate.""" if ctx.channel.id in channels or ctx.channel.id in hidden_channels: |