From e0c288289d9308cdf39ce912767675bdffd2946e Mon Sep 17 00:00:00 2001
From: ddorn <diego.dorn@free.fr>
Date: Sat, 23 May 2020 20:18:21 +0200
Subject: [PATCH] :sparkles: tourist for final

---
 src/cogs/misc.py    |  1 -
 src/cogs/teams.py   | 15 ++++++++++++---
 src/cogs/tirages.py |  2 +-
 3 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/src/cogs/misc.py b/src/cogs/misc.py
index 84acffa..faa7941 100644
--- a/src/cogs/misc.py
+++ b/src/cogs/misc.py
@@ -34,7 +34,6 @@ from discord.ext.commands import (
 from discord.utils import get
 
 from src.constants import *
-from src.constants import Emoji
 from src.core import CustomBot
 from src.errors import TfjmError
 from src.utils import has_role, start_time, send_and_bin
diff --git a/src/cogs/teams.py b/src/cogs/teams.py
index 1c4b0b7..2639020 100644
--- a/src/cogs/teams.py
+++ b/src/cogs/teams.py
@@ -91,7 +91,7 @@ class TeamsCog(Cog, name="Teams"):
 
     @commands.command(name="tourist")
     @send_and_bin
-    async def touriste_cmd(self, ctx: Context, tournoi):
+    async def touriste_cmd(self, ctx: Context, tournoi="Finale"):
         """
         Permet de voir et ecouter dans les salons d'un certain tournoi.
 
@@ -99,9 +99,13 @@ class TeamsCog(Cog, name="Teams"):
             `!tourist Bordeaux-Nancy` - Donne les droits de spectateurs pour Bordeaux-Nancy
         """
 
-        if ctx.author.top_role != ctx.guild.default_role:
+        if ctx.author.top_role not in (
+            ctx.guild.default_role,
+            get(ctx.guild.roles, name=Role.TOURIST),
+        ):
             return f"{ctx.author.mention} tu as déjà un role, devenir spéctateur t'enlèverait des droits."
 
+        tournoi = tournoi.title()
         if tournoi not in TOURNOIS:
             return f"{tournoi} n'est pas un nom de tournoi. Possibilités: {french_join(TOURNOIS)}"
 
@@ -109,11 +113,16 @@ class TeamsCog(Cog, name="Teams"):
         tournoi_role = get(guild.roles, name=tournoi)
         touriste_role = get(guild.roles, name=Role.TOURIST)
 
+        if tournoi == "Finale":
+            # we don't give the "Finaliste" role to the tourists, it would give them rights.
+            # to simplify branching we assign twice "Touriste"
+            touriste_role = touriste_role
+
         await ctx.author.add_roles(
             touriste_role, tournoi_role, reason="Demande via le bot."
         )
 
-        return f"{ctx.author.mention} à été ajouté comme spectateur dans à {tournoi}."
+        return f"{ctx.author.mention} à été ajouté comme spectateur pour {tournoi}."
 
     @group(name="team", invoke_without_command=True, case_insensitive=True, hidden=True)
     async def team(self, ctx):
diff --git a/src/cogs/tirages.py b/src/cogs/tirages.py
index 8e88877..31f4a4a 100644
--- a/src/cogs/tirages.py
+++ b/src/cogs/tirages.py
@@ -280,7 +280,7 @@ class DiscordTirage(BaseTirage):
             table = """```
 ╔═════╦═════════╦═════════╦═════════╗
 ║ T F ║ Phase 1 ║ Phase 2 ║ Phase 3 ║
-║ J M ║   Pb {0.pb}  ║   Pb {1.pb}  ║   Pb {2.pb}  ║
+║ J M ║  Pb. {0.pb}  ║  Pb. {1.pb}  ║  Pb. {2.pb}  ║
 ╠═════╬═════════╬═════════╬═════════╣
 ║ {0.name} ║   Def   ║   Rap   ║   Opp   ║
 ╠═════╬═════════╬═════════╬═════════╣