trying to fix duplicate lines in 'leveling_debug'

Author: modernNeo

wall_e/extensions/ban.py

@@ -33,7 +33,7 @@ def __init__(self):
         self.logger.info("[Ban __init__()] initializing Ban")
         self.mod_channel = None
         self.bot_management_channel = None
-        self.guild: discord.Guild = None
+        self.guild: discord.Guild | None = None
         self.purge_messages_task.start()
 
     @commands.Cog.listener(name="on_ready")

wall_e/utilities/log_channel.py

@@ -25,45 +25,47 @@ async def write_to_bot_log_channel(logger, config, bot, file_path, chan_id, chan
     )
     f = open(file_path, 'r')
     f.seek(0)
+    channels_with_rate_limit = channel_name in ['leveling_debug', 'role_commands_debug']
     incident_report_chanel_name = config.get_config_value('channel_names', 'INCIDENT_REPORT_CHANNEL')
     while not bot.is_closed():
         f.flush()
         line = f.readline()
-        while line:
-            if line.strip() != "":
-                # this was done so that no one gets accidentally pinged from the bot log channel
-                if channel.name != incident_report_chanel_name:
-                    line = line.replace("@", "[at]")
-                if line[0] == ' ':
-                    line = f".{line}"
-                output = line
-                # done because discord has a character limit of 2000 for each message
-                # so what basically happens is it first tries to send the full message, then if it cant, it
-                # breaks it down into 2000 sizes messages and send them individually
-                try:
-                    await channel.send(output)
-                except (aiohttp.ClientError, discord.errors.HTTPException):
-                    finished = False
-                    first_index, last_index = 0, 2000
-                    while not finished:
-                        await channel.send(output[first_index:last_index])
-                        first_index = last_index
-                        last_index += 2000
-                        if len(output[first_index:last_index]) == 0:
-                            finished = True
-                except RuntimeError:
-                    logger.debug(
-                        "[log_channel.py write_to_bot_log_channel()] encountered RuntimeError, "
-                        " will assume that the user is attempting to exit"
-                    )
-                    break
-                except Exception as exc:
-                    exc_str = f'{type(exc).__name__}: {exc}'
-                    raise Exception(
-                        f'[log_channel.py write_to_bot_log_channel()] write to channel failed\n{exc_str}'
-                    )
-            line = f.readline()
-            if channel_name in ['leveling_debug', 'role_commands_debug']:
+        if line and line.strip() != "":
+            # this was done so that no one gets accidentally pinged from the bot log channel
+            if channel.name != incident_report_chanel_name:
+                line = line.replace("@", "[at]")
+            if line[0] == ' ':
+                line = f".{line}"
+            output = line
+            # done because discord has a character limit of 2000 for each message
+            # so what basically happens is it first tries to send the full message, then if it cant, it
+            # breaks it down into 2000 sizes messages and send them individually
+            message_sent = False
+            try:
+                await channel.send(output)
+                message_sent = True
+            except (aiohttp.ClientError, discord.errors.HTTPException):
+                finished = False
+                first_index, last_index = 0, 2000
+                while not finished:
+                    await channel.send(output[first_index:last_index])
+                    message_sent = True
+                    first_index = last_index
+                    last_index += 2000
+                    if len(output[first_index:last_index]) == 0:
+                        finished = True
+            except RuntimeError:
+                logger.debug(
+                    "[log_channel.py write_to_bot_log_channel()] encountered RuntimeError, "
+                    " will assume that the user is attempting to exit"
+                )
+                break
+            except Exception as exc:
+                exc_str = f'{type(exc).__name__}: {exc}'
+                raise Exception(
+                    f'[log_channel.py write_to_bot_log_channel()] write to channel failed\n{exc_str}'
+                )
+            if message_sent and channels_with_rate_limit:
                 # adding a sleep cause the amount of debug logs that I print due to the wall_e_models module can
                 # trigger a Rate Limit exception if done too fast
                 await asyncio.sleep(1)