Rework logic of purge_user_messages

Author: FrozenChen

cogs/server_logs.py

@@ -180,7 +180,7 @@ async def logsdelete(self, interaction: discord.Interaction,
                          before: app_commands.Transform[Optional[datetime], DateTransformer] = None,
                          after: app_commands.Transform[Optional[datetime], DateTransformer] = None,
                          during: app_commands.Transform[Optional[datetime], DateTransformer] = None,
-                         limit: app_commands.Range[int, 1, 100] = 10,
+                         limit: app_commands.Range[int, 1, 50] = 10,
                          ):
         """Delete messages from user according to messages stored in the server logs, deletes last limit messages by default.
 

utils/serverlogs.py

@@ -86,7 +86,7 @@ async def purge_user_messages(self, user_id: int, limit: int, channel_id: int =
         if self.conn is None:
             return deleted, failures
 
-        stmt, bindings = self.build_query(member=user_id, after=after, limit=limit, channel=channel_id, before=before, during=during, sort_by_channel=False)
+        stmt, bindings = self.build_query(member=user_id, after=after, limit=50, channel=channel_id, before=before, during=during, sort_by_channel=False)
 
         async with self.conn.transaction():
             async for message_id, created_at, channel_id, channel_name, _, _ in self.conn.cursor(stmt, *bindings):
@@ -96,6 +96,8 @@ async def purge_user_messages(self, user_id: int, limit: int, channel_id: int =
                 try:
                     await channel.get_partial_message(message_id).delete()
                     deleted += 1
+                    if deleted == limit:
+                        break
                 except discord.NotFound:
                     pass
                 except (discord.Forbidden, discord.HTTPException) as e: