performance patch for the warning commands.

README.md

@@ -22,6 +22,13 @@ A simple Discord Bot used by me personally, written for fun.
 ## Changelog
 
 #### v1.3_3
+###### Performance
+- Changed the warning listing command do that it isn't slow.
+###### Bug Fixes
+- Removing all warnings of a user now removes them from the dictionary. Making them not display in the list even though they don't have any warnings.
+
+
+#### v1.3.3
 ###### Hottest of Bug Fixes
 - Warn list now actually works for all warnings.
 - Slowmode now ignores mods and admins.

cogs/admin.py

@@ -205,14 +205,26 @@ class Admin():
 	@warn.command(pass_context=True)
 	async def list(self, ctx, *, user: discord.User = None):
 		await self.bot.send_typing(ctx.message.channel)
+
 		if user == None:
 			output = ""
-			for user in self.servers[ctx.message.server.id]["warnings"]:
-				user_obj = await self.bot.get_user_info(user)
-				output += "{}#{}: {} Warning(s)\n".format(user_obj.name, user_obj.discriminator, len(self.servers[ctx.message.server.id]["warnings"][user]))
+			for member in self.servers[ctx.message.server.id]["warnings"]:
+				# Remove users with no warning here instead of remove cause im lazy
+				if not self.servers[ctx.message.server.id]["warnings"][user]:
+					self.servers[ctx.message.server.id]["warnings"].pop(user)
+				else:
+					member_obj = discord.utils.get(ctx.message.guild.members, id=member)
+					if member_obj:
+						output += "{}#{}: {} Warning(s)\n".format(member_obj.name, member_obj.discriminator, len(
+							self.servers[ctx.message.server.id]["warnings"][user]))
+					else:
+						member_obj = await self.bot.get_user_info(user)
+						output += "{}#{}: {} Warning(s)\n".format(member_obj.name, member_obj.discriminator, len(
+							self.servers[ctx.message.server.id]["warnings"][user]))
 			return await self.bot.say(output)
 
-
+		if not self.servers[ctx.message.server.id]["warnings"][user]:
+			self.servers[ctx.message.server.id]["warnings"].pop(user)
 		if not user.id in self.servers[ctx.message.server.id]["warnings"]:
 			return await self.bot.say("This user doesn't have any warning on record.")
 		em = discord.Embed(title="Warnings for {}".format(user.name+"#"+user.discriminator), colour=0XDEADBF)

config/meta.py

@@ -3,5 +3,5 @@ __description__ = """RoxBot, A Discord Bot made by a filthy Mercy Main. Built wi
 [Github link](https://github.com/RainbowDinoaur/roxbot)
 [Changelog](https://github.com/RainbowDinoaur/roxbot#v100)"""
 __author__ = "Roxanne Gibson"
-__version__= "1.3.3"
+__version__= "1.3.4"
 embedcolour = 0xDEADBF