Fix issue during the parsing of "badbins" (#66)

* fix unpacking of what is actually a string

* update and run ruff

core/utils/paginator.py

@@ -237,10 +237,7 @@ async def stop_pages(self, interaction: discord.Interaction | None = None) -> No
     stop = stop_pages  # type: ignore
 
     def _check(self, interaction: discord.Interaction) -> bool:
-        if interaction.user.id != self.author.id:
-            return False
-
-        return True
+        return interaction.user.id == self.author.id
 
     async def interaction_check(self, interaction: discord.Interaction) -> bool:
         resp = self._check(interaction)

launcher.py

@@ -70,6 +70,7 @@ async def main() -> None:
         tasks.add(asyncio.create_task(bot.start(core.CONFIG["TOKENS"]["bot"])))
         await server.serve()
 
+
 try:
     asyncio.run(main())
 except KeyboardInterrupt:

modules/help.py

@@ -73,10 +73,7 @@ def predicate(ctx: Context) -> bool:
         if not isinstance(channel, discord.Thread):
             return False
 
-        if channel.parent_id != Channels.HELP_FORUM:
-            return False
-
-        return True
+        return channel.parent_id == Channels.HELP_FORUM
 
     return commands.check(predicate)
 

modules/moderation.py

@@ -243,25 +243,26 @@ async def post_mystbin_content(self, contents: list[tuple[str, str]]) -> str:
     async def find_badbins(self, message: discord.Message) -> None:
         matches = self.BADBIN_RE.findall(message.content)
 
-        if matches:
-            contents: list[tuple[str, str]] = []
+        if not matches:
+            return
+
+        contents: list[tuple[str, str]] = []
 
-            for match in matches:
-                site, slug, ext = match
+        for match in matches:
+            site, slug, ext = match
 
-                if site is None or slug is None:
-                    continue
+            if site is None or slug is None:
+                continue
 
-                contents.append((await self.pull_badbin_content(site, slug), f"migrated.{ext or 'txt'}"))
+            contents.append((await self.pull_badbin_content(site, slug), f"migrated.{ext or 'txt'}"))
 
-            if contents:
-                key, notice = await self.post_mystbin_content(contents)
-                msg = f"I've detected a badbin and have uploaded your pastes here: https://mystb.in/{key}"
+        if not contents:
+            return
 
-                if notice:
-                    msg += "\nnotice: " + notice
+        key = await self.post_mystbin_content(contents)
+        msg = f"I've detected a badbin and have uploaded your pastes here: https://mystb.in/{key}"
 
-                await message.reply(msg, mention_author=False)
+        await message.reply(msg, mention_author=False)
 
     @commands.Cog.listener()
     async def on_papi_dpy_modlog(self, payload: ModLogPayload, /) -> None: