CopyBot/Red-DiscordBot
1
0
Fork 0
mirror of https://github.com/Cog-Creators/Red-DiscordBot.git synced 2025-11-07 03:38:53 -05:00
Code Issues Packages Projects Releases Wiki Activity

[V3 Mod] fix error with [p]names (#1188)

Browse Source 
* [V3 Mod] fix error with [p]names

* [V3 Mod] add null cleaner to names command
This commit is contained in:
palmtree5 2017-12-17 16:23:35 -09:00 committed by Will
parent ba5b18dfe8
commit 73d68e926d
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
redbot/cogs/mod/mod.py
View File

@ -1122,6 +1122,9 @@ class Mod:
@commands.command() @commands.command()
async def names(self, ctx: RedContext, user: discord.Member): async def names(self, ctx: RedContext, user: discord.Member):
"""Show previous names/nicknames of a user""" """Show previous names/nicknames of a user"""
async with self.settings.user(user).past_names() as name_list:
while None in name_list: # clean out null entries from a bug
name_list.remove(None)
names = await self.settings.user(user).past_names() names = await self.settings.user(user).past_names()
nicks = await self.settings.member(user).past_nicks() nicks = await self.settings.member(user).past_nicks()
msg = "" msg = ""
@ -1343,13 +1346,15 @@ class Mod:
if entry.target == target: if entry.target == target:
return entry return entry
async def on_member_update(self, before, after): async def on_member_update(self, before: discord.Member, after: discord.Member):
if before.name != after.name: if before.name != after.name:
async with self.settings.user(before).past_names() as name_list: async with self.settings.user(before).past_names() as name_list:
if after.nick in name_list: while None in name_list: # clean out null entries from a bug
name_list.remove(None)
if after.name in name_list:
# Ensure order is maintained without duplicates occuring # Ensure order is maintained without duplicates occuring
name_list.remove(after.nick) name_list.remove(after.name)
name_list.append(after.nick) name_list.append(after.name)
while len(name_list) > 20: while len(name_list) > 20:
name_list.pop(0) name_list.pop(0)