diff options
| author |  Izan <[email protected]> | 2021-08-13 10:18:31 +0100 |
|---|---|---|
| committer | Izan <[email protected]> | 2021-08-22 20:24:00 +0100 |
| commit | 39818408bcd9ca2a65d2459a216c81db39c80d18 (patch) | |
| tree | 4289022e5be4f5f2ddb4191031f01af2245cb1ef | |
| parent | Add missing call to `escape_markdown` (diff) | |
Remove redundant fetching of user code
| -rw-r--r-- | bot/exts/moderation/infraction/_scheduler.py | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/bot/exts/moderation/infraction/_scheduler.py b/bot/exts/moderation/infraction/_scheduler.py index 13f59bb76..3c5e5d3bf 100644 --- a/bot/exts/moderation/infraction/_scheduler.py +++ b/bot/exts/moderation/infraction/_scheduler.py @@ -165,17 +165,10 @@ class InfractionScheduler: dm_result = f"{constants.Emojis.failmail} " dm_log_text = "\nDM: **Failed**" - # Sometimes user is a discord.Object; make it a proper user. - try: - if not isinstance(user, (discord.Member, discord.User)): - user = await self.bot.fetch_user(user.id) - except discord.HTTPException as e: - log.error(f"Failed to DM {user.id}: could not fetch user (status {e.status})") - else: - # Accordingly display whether the user was successfully notified via DM. - if await _utils.notify_infraction(user, infr_type.replace("_", " ").title(), expiry, user_reason, icon): - dm_result = ":incoming_envelope: " - dm_log_text = "\nDM: Sent" + # Accordingly display whether the user was successfully notified via DM. + if await _utils.notify_infraction(user, infr_type.replace("_", " ").title(), expiry, user_reason, icon): + dm_result = ":incoming_envelope: " + dm_log_text = "\nDM: Sent" end_msg = "" if infraction["actor"] == self.bot.user.id: |