Fix thread duplication bug

Author: kyb3r

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+
+# v2.8.1
+### Fixed
+- Fixed bug where thread logs were getting duplicated when using the contact command.
+
 # v2.8.0
 ### Changed
 - Major improvement in thread log viewing. 

bot.py

@@ -22,7 +22,7 @@
 SOFTWARE.
 """
 
-__version__ = '2.8.0'
+__version__ = '2.8.1'
 
 import asyncio
 import textwrap

core/thread.py

@@ -189,6 +189,10 @@ async def reply(self, message):
             # to user
             self.send(message, self.recipient, from_mod=True)
             ]
+        
+        self.bot.loop.create_task(
+            self.bot.modmail_api.append_log(message, self.channel.id)
+            )
 
         if self.close_task is not None:
             # cancel closing if a thread message is sent.
@@ -207,17 +211,16 @@ async def send(self, message, destination=None, from_mod=False):
             await self.channel.send(embed=discord.Embed(
                 color=discord.Color.red(),
                 description='Scheduled close has been cancelled.'))
+        
+        if not from_mod:
+            self.bot.loop.create_task(
+                self.bot.modmail_api.append_log(message, self.channel.id)
+                )
 
         if not self.ready:
             await self.wait_until_ready()
 
         destination = destination or self.channel
-        if from_mod and not isinstance(destination, discord.User):
-            self.bot.loop.create_task(
-                self.bot.modmail_api.append_log(message))
-        elif not from_mod:
-            self.bot.loop.create_task(
-                self.bot.modmail_api.append_log(message, destination.id))
 
         author = message.author
 
@@ -431,8 +434,7 @@ async def create(self, recipient, *, creator=None, category=None):
         thread.channel = channel
 
         log_url, log_data = await asyncio.gather(
-            self.bot.modmail_api.get_log_url(recipient, channel,
-                                             creator or recipient),
+            self.bot.modmail_api.get_log_url(recipient, channel, creator or recipient),
             self.bot.modmail_api.get_user_logs(recipient.id)
             # self.get_dominant_color(recipient.avatar_url)
         )