merge few loop into a single one

Author: TitouanDor

src/commands/club.py

@@ -174,6 +174,16 @@ async def stop_club(
         self, interaction: Interaction, club: Transform[ClubSchema, ClubTransformer]
     ):
         await interaction.response.defer(thinking=True)
+        guild = interaction.guild
         discord_club = DiscordClub.load(club.id)
         await self.club_service.stop_club(discord_club, interaction.guild)
+        annonce = await self.club_service.get_channel(
+            guild, discord_club.category_id, f"annonces {club.name}".lower()
+        )
+
+        if annonce:
+            await annonce.send(
+                "Le club, n'ayant pas été repris, est "
+                "temporairement fermer jusqu'à reprise du club"
+            )
         await interaction.followup.send(f"Le club : {club.name} à été arrété")

src/services/club.py

@@ -185,15 +185,13 @@ async def stop_club(self, club: DiscordClub, guild: Guild):
         role_treso = utils.get(guild.roles, id=club.treasurer_role_id)
         role_member = utils.get(guild.roles, id=club.member_role_id)
         role_former = utils.get(guild.roles, id=club.former_member_role_id)
-
-        for e in role_pres.members:
-            await e.remove_roles(role_pres, reason=f"Arrêt du club : {club.name}")
-            await e.add_roles(role_former, reason=f"Arrêt du club : {club.name}")
-
-        for e in role_treso.members:
-            await e.remove_roles(role_pres, reason=f"Arrêt du club : {club.name}")
-            await e.add_roles(role_former, reason=f"Arrêt du club : {club.name}")
-
-        for e in role_member.members:
-            await e.remove_roles(role_member, reason=f"Arrêt du club : {club.name}")
+        old_member = {*role_pres.members, *role_treso.members, *role_member.members}
+
+        for e in old_member:
+            await e.remove_roles(
+                role_pres,
+                role_treso,
+                role_member,
+                reason=f"Arrêt du club : {club.name}",
+            )
             await e.add_roles(role_former, reason=f"Arrêt du club : {club.name}")