diff options
| author |  S. Co1 <[email protected]> | 2020-01-15 20:55:10 -0500 |
|---|---|---|
| committer | S. Co1 <[email protected]> | 2020-01-15 20:55:10 -0500 |
| commit | 76f976289aa6ae85979c2f8596d68db00555c2a5 (patch) | |
| tree | 8f9215d84c27079bf7f51805ec01eeab3d2f5018 | |
| parent | Discord.py server added to whitelist (diff) | |
Fix user command error for empty custom status
In the scenario where a user has a custom status set to only an emoji & blank text value, their activity status is set as `None`, which causes `discord.utils.escape_markdown` to raise an exception, as it's expecting something string-like.
We can guard against this by checking for a `None` value & skipping the markdown escape if necessary.
| -rw-r--r-- | bot/cogs/information.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/bot/cogs/information.py b/bot/cogs/information.py index 1ede95ff4..125d7ce24 100644 --- a/bot/cogs/information.py +++ b/bot/cogs/information.py @@ -189,7 +189,11 @@ class Information(Cog): # Custom status custom_status = '' for activity in user.activities: - if activity.name == 'Custom Status': + # Check activity.state for None value if user has a custom status set + # This guards against a custom status with an emoji but no text, which will cause + # escape_markdown to raise an exception + # This can be reworked after a move to d.py 1.3.0+, which adds a CustomActivity class + if activity.name == 'Custom Status' and activity.state: state = escape_markdown(activity.state) custom_status = f'Status: {state}\n' |