From 36ae732ea72ccda814dd12c076b8a5047d18e37c Mon Sep 17 00:00:00 2001
From: Stefano Pigozzi <ste.pigozzi@gmail.com>
Date: Fri, 4 Jan 2019 18:44:03 +0100
Subject: [PATCH] Get the command prefix from the config

---
 discordbot.py       | 72 ++++++++++++++++++++++++---------------------
 template_config.ini |  2 ++
 2 files changed, 41 insertions(+), 33 deletions(-)

diff --git a/discordbot.py b/discordbot.py
index 9731bf9f..6e1f6048 100644
--- a/discordbot.py
+++ b/discordbot.py
@@ -382,44 +382,44 @@ def requires_rygdb(func, optional=False):
 
 
 class RoyalDiscordBot(discord.Client):
-    def __init__(self, command_prefix="!", *args, **kwargs):
+    def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.main_channel: typing.Optional[discord.TextChannel] = None
         self.main_guild: typing.Optional[discord.Guild] = None
-        self.commands = {
-            f"{command_prefix}ping": self.cmd_ping,
-            f"{command_prefix}cv": self.cmd_cv,
-            f"{command_prefix}summon": self.cmd_cv,
-            f"{command_prefix}play": self.cmd_play,
-            f"{command_prefix}alexaplay": self.cmd_play,
-            f"{command_prefix}okgoogleplay": self.cmd_play,
-            f"{command_prefix}heysiriplay": self.cmd_play,
-            f"{command_prefix}p": self.cmd_play,
-            f"{command_prefix}search": self.cmd_play,
-            f"{command_prefix}file": self.cmd_play,
-            f"{command_prefix}skip": self.cmd_skip,
-            f"{command_prefix}s": self.cmd_skip,
-            f"{command_prefix}next": self.cmd_skip,
-            f"{command_prefix}remove": self.cmd_remove,
-            f"{command_prefix}r": self.cmd_remove,
-            f"{command_prefix}cancel": self.cmd_remove,
-            f"{command_prefix}queue": self.cmd_queue,
-            f"{command_prefix}q": self.cmd_queue,
-            f"{command_prefix}shuffle": self.cmd_shuffle,
-            f"{command_prefix}clear": self.cmd_clear,
-            f"{command_prefix}register": self.cmd_register,
-            f"{command_prefix}radiomessages": self.cmd_radiomessages,
-            f"{command_prefix}yes": self.null,
-            f"{command_prefix}no": self.null,
-            f"{command_prefix}pause": self.cmd_pause,
-            f"{command_prefix}resume": self.cmd_resume,
-            f"{command_prefix}loop": self.cmd_mode,
-            f"{command_prefix}l": self.cmd_mode,
-            f"{command_prefix}mode": self.cmd_mode,
-            f"{command_prefix}m": self.cmd_mode
-        }
         self.video_queue: VideoQueue = VideoQueue()
         self.load_config("config.ini")
+        self.commands = {
+            f"{self.command_prefix}ping": self.cmd_ping,
+            f"{self.command_prefix}cv": self.cmd_cv,
+            f"{self.command_prefix}summon": self.cmd_cv,
+            f"{self.command_prefix}play": self.cmd_play,
+            f"{self.command_prefix}alexaplay": self.cmd_play,
+            f"{self.command_prefix}okgoogleplay": self.cmd_play,
+            f"{self.command_prefix}heysiriplay": self.cmd_play,
+            f"{self.command_prefix}p": self.cmd_play,
+            f"{self.command_prefix}search": self.cmd_play,
+            f"{self.command_prefix}file": self.cmd_play,
+            f"{self.command_prefix}skip": self.cmd_skip,
+            f"{self.command_prefix}s": self.cmd_skip,
+            f"{self.command_prefix}next": self.cmd_skip,
+            f"{self.command_prefix}remove": self.cmd_remove,
+            f"{self.command_prefix}r": self.cmd_remove,
+            f"{self.command_prefix}cancel": self.cmd_remove,
+            f"{self.command_prefix}queue": self.cmd_queue,
+            f"{self.command_prefix}q": self.cmd_queue,
+            f"{self.command_prefix}shuffle": self.cmd_shuffle,
+            f"{self.command_prefix}clear": self.cmd_clear,
+            f"{self.command_prefix}register": self.cmd_register,
+            f"{self.command_prefix}radiomessages": self.cmd_radiomessages,
+            f"{self.command_prefix}yes": self.null,
+            f"{self.command_prefix}no": self.null,
+            f"{self.command_prefix}pause": self.cmd_pause,
+            f"{self.command_prefix}resume": self.cmd_resume,
+            f"{self.command_prefix}loop": self.cmd_mode,
+            f"{self.command_prefix}l": self.cmd_mode,
+            f"{self.command_prefix}mode": self.cmd_mode,
+            f"{self.command_prefix}m": self.cmd_mode
+        }
         if self.sentry_token:
             self.sentry = raven.Client(self.sentry_token,
                                        release=raven.fetch_git_sha(os.path.dirname(__file__)),
@@ -552,6 +552,12 @@ class RoyalDiscordBot(discord.Client):
         except (KeyError, ValueError):
             logger.warning("Song text easter egg information not found, defaulting to disabled.")
             self.song_text_easter_eggs = {}
+        # Discord command prefix
+        try:
+            self.command_prefix = config["Discord"]["command_prefix"]
+        except (KeyError, ValueError):
+            logger.warning("Command prefix not set, defaulting to '!'.")
+            self.command_prefix = "!"
 
     # noinspection PyAsyncCall
     async def on_ready(self):
diff --git a/template_config.ini b/template_config.ini
index 7a5099b3..8ffde7c8 100644
--- a/template_config.ini
+++ b/template_config.ini
@@ -23,6 +23,8 @@ max_ready_time = 30
 radio_messages_enabled = False
 # Number of videos between two radio messages
 radio_messages_every = 5
+# Command prefix
+command_prefix = !
 
 [Telegram]
 # Telegram bot token, obtainable at https://t.me/BotFather