Improve logs format

Author: kyb3r

bot.py

@@ -259,7 +259,8 @@ async def get_context(self, message, *, cls=commands.Context):
         ctx.command = self.all_commands.get(invoker)
 
         if ctx.command is self.get_command('eval') and hasattr(ctx, '_alias_invoked'):
-            ctx.command.checks = None # Let anyone use the command.
+            # ctx.command.checks = None # Let anyone use the command.
+            pass
 
         return ctx
 

cogs/modmail.py

@@ -13,6 +13,9 @@ class Modmail:
 
     def __init__(self, bot):
         self.bot = bot
+    
+    def obj(arg):
+        return discord.Object(int(arg))
 
     @commands.command()
     @trigger_typing
@@ -171,7 +174,7 @@ async def nsfw(self, ctx):
     @commands.command()
     @commands.has_permissions(manage_messages=True)
     @trigger_typing
-    async def logs(self, ctx, *, member: Union[discord.Member, discord.User]=None):
+    async def logs(self, ctx, *, member: Union[discord.Member, discord.User, obj]=None):
         """Shows a list of previous modmail thread logs of a member."""
 
         if not member:
@@ -181,13 +184,15 @@ async def logs(self, ctx, *, member: Union[discord.Member, discord.User]=None):
 
         user = member or thread.recipient
 
+        icon_url = getattr(user, 'avatar_url', 'https://cdn.discordapp.com/embed/avatars/0.png')
+
         logs = await self.bot.modmail_api.get_user_logs(user.id)
 
         if not any(not e['open'] for e in logs):
             return await ctx.send('This user has no previous logs.')
 
         em = discord.Embed(color=discord.Color.green())
-        em.set_author(name='Previous Logs', icon_url=user.avatar_url)
+        em.set_author(name='Previous Logs', icon_url=icon_url)
 
         embeds = [em]
 
@@ -200,20 +205,31 @@ async def logs(self, ctx, *, member: Union[discord.Member, discord.User]=None):
         for index, entry in enumerate(closed_logs):
             if len(embeds[-1].fields) == 3:
                 em = discord.Embed(color=discord.Color.green())
-                em.set_author(name='Previous Logs', icon_url=user.avatar_url)
+                em.set_author(name='Previous Logs', icon_url=icon_url)
                 embeds.append(em)
 
             date = dateutil.parser.parse(entry['created_at'])
+
             new_day = date.strftime(r'%d %b %Y')
+            time = date.strftime(r'%H:%M')
 
             key = entry['key']
             user_id = entry['user_id']
             log_url = f"https://logs.modmail.tk/{user_id}/{key}"
 
-            fmt += f"[`{key}`]({log_url})\n"
+            truncate = lambda c: c[:47] + '...' if len(c) > 50 else c
+
+            if entry['messages']:
+                short_desc = truncate(entry['messages'][0]['content']) or 'No content'
+            else:
+                short_desc = 'No content'
+
+            fmt += f"[`[{time}]{key}`]({log_url}) - {short_desc}\n"
+
+            print(fmt)
 
             if current_day != new_day or index == len(closed_logs) - 1:
-                embeds[-1].add_field(name=current_day, value=fmt)
+                embeds[-1].add_field(name=current_day, value=fmt, inline=False)
                 current_day = new_day
                 fmt = ''
 
@@ -286,9 +302,6 @@ async def contact(self, ctx, *, user: Union[discord.Member, discord.User]):
 
         await ctx.send(embed=em)
 
-    def obj(arg):
-        return discord.Object(int(arg))
-
     @commands.command()
     @trigger_typing
     @commands.has_permissions(manage_channels=True)