From d6fce3c14504becbb960ee71c123d12a076ecf97 Mon Sep 17 00:00:00 2001 From: palmtree5 <3577255+palmtree5@users.noreply.github.com> Date: Sun, 5 Nov 2017 16:02:45 -0900 Subject: [PATCH] [Mod] fix duplicate cases on ban/unban (fixes Cog-Creators/Red-DiscordBot/issues/1052) (#1064) --- redbot/cogs/mod/mod.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/redbot/cogs/mod/mod.py b/redbot/cogs/mod/mod.py index 9c3df7799..ba9e56409 100644 --- a/redbot/cogs/mod/mod.py +++ b/redbot/cogs/mod/mod.py @@ -1167,8 +1167,9 @@ class Mod: mod = audit_case.user reason = audit_case.reason for case in sorted(modlog_cases, key=lambda x: x.case_number, reverse=True): - if case.moderator == mod and case.user == member\ + if mod == guild.me and case.user == member\ and case.action_type in ["ban", "hackban"]: + log.info("Case already exists for ban of {}".format(member.name)) break else: # no ban, softban, or hackban case with the mod and user combo try: @@ -1231,8 +1232,9 @@ class Mod: cases = await modlog.get_all_cases(guild, self.bot) for case in sorted(cases, key=lambda x: x.case_number, reverse=True): - if case.moderator == mod and case.user == user\ - and case.action_type == "unban": + if mod == guild.me and case.user == user\ + and case.action_type == "unban": + log.info("Case already exists for unban of {}".format(user.name)) break else: try: