Change CMD_SIGN into Config.CRSM.CommandSign

2 files changed, 14 insertions, 22 deletions

diff --git a/src/CRSMConfig.hpp b/src/CRSMConfig.hpp
index 1c265ec..c8750dd 100644
--- a/src/CRSMConfig.hpp
+++ b/src/CRSMConfig.hpp
@@ -244,6 +244,7 @@ public:
     struct {
         Port ManagementPort = 9372;
         String ListFolder = "ScenarioLists";
+        String CommandSign = "!";
     } CRSM;
 
     struct {
@@ -319,6 +320,7 @@ public:
     explicit CRSMConfig() : CRSMConfigBase({
         ConfigValue(CRSM.ManagementPort),
         ConfigValue(CRSM.ListFolder),
+        ConfigValue(CRSM.CommandSign),
 
 
 
diff --git a/src/crsm.cpp b/src/crsm.cpp
index 129b10d..298c1e6 100644
--- a/src/crsm.cpp
+++ b/src/crsm.cpp
@@ -8,8 +8,6 @@
 #include <QTimer>
 #include <QRegularExpression>
 
-#define CMD_SIGN "!"
-
 #define MGMT_BUFFER_FILENAME "CRSM-MGMT-Buffer"
 
 CRSM::CRSM(QObject *parent) :
@@ -158,7 +156,7 @@ void CRSM::readServerOutput()
             else if(!isMeMessage)
             {
                 QString msg = userexp.cap(5).trimmed();
-                QRegExp commandExp("^\\" CMD_SIGN "([^ ]+\\s+(.*)\\s*)?$");
+                QRegExp commandExp("^\\" + Config.CRSM.CommandSign + "([^ ]+\\s+(.*)\\s*)?$");
                 if(commandExp.exactMatch(msg))
                 {
                     QString command = commandExp.cap(1).trimmed();
@@ -456,17 +454,9 @@ void CRSM::ircMessageReceived(IrcMessage *message)
         {
             writeToServer("[IRC]<" + message->nick() + "> " + mess + "\n");
         }
-        else if(mess.length() > QString(CMD_SIGN).length() && mess.left(QString(CMD_SIGN).length()) == CMD_SIGN)
+        else if(mess.length() > Config.CRSM.CommandSign.length() && mess.left(Config.CRSM.CommandSign.length()) == Config.CRSM.CommandSign)
         {
-<<<<<<< HEAD
-<<<<<<< HEAD
-            QString command = mess.right(mess.length() - QString(CMD_SIGN).length());
-=======
             QString command = mess.mid(Config.CRSM.CommandSign.length()).trimmed();
->>>>>>> 49e362f... fixup! Add command groups
-=======
-            QString command = mess.mid(QString(CMD_SIGN).length());
->>>>>>> 3791e42... Use QString::mid instead of QString::right where appropriate
             const ClientInfo& client = ClientInfo::ircClient(message->nick(), target);
             if(!cmd(command, client))
             {
@@ -1414,7 +1404,7 @@ void CRSM::setupCmds()
 
     addCommand("skip", &CRSM::skip, Clonk | IRC | Management, Moderator, "Entfernt das nächste Szenario aus der Wunschliste.");
     addCommand("next", &CRSM::next, Clonk | IRC | Management, Moderator, "Versucht das aktuelle Szenario zu überspringen.");
-    addCommand("kill", &CRSM::kill, IRC | Management, Moderator, "Tötet den Clonk-Server, vorausgesetzt es läuft einer.", "", "Tötet den Clonk-Server, vorausgesetzt es läuft einer. Bitte nur verwenden wenn " CMD_SIGN "next nicht funktioniert.");
+    addCommand("kill", &CRSM::kill, IRC | Management, Moderator, "Tötet den Clonk-Server, vorausgesetzt es läuft einer.", "", "Tötet den Clonk-Server, vorausgesetzt es läuft einer. Bitte nur verwenden wenn " + Config.CRSM.CommandSign + "next nicht funktioniert.");
     addCommand("clear", &CRSM::clear, Clonk | IRC | Management, Moderator, "Löscht die Wunschliste.");
     addCommand("aliaswishes", &CRSM::aliaswishes, IRC | Management, Moderator, "Zum geführten Bearbeiten der Aliaswünsche.");
     addCommand("newalias", &CRSM::newalias, Clonk | IRC | Management, Moderator, "Trägt <Alias> als Alias für <Szenario> ein.", "<Alias> = <Szenario>");
@@ -1440,7 +1430,7 @@ void CRSM::setupCmds()
 
     addCommandGroup("exit", Management, UserType::Max, "Beendet den Server Mananger.", "Beendet den Server Mananger und falls ein Clonk-Server läuft auch diesen. Die Unterbefehle ermöglichen außerdem diverse Zusatzvarienten des Beendens.", &CRSM::exit);
     addCommand("exit detach", &CRSM::exitDetach, Management, UserType::Max, "Beendet den Server Mananger, lässt den Clonk-Server weiterlaufen.", "",  "Beendet den Server Mananger und falls ein Clonk-Server läuft, läuft dieser weiter.");
-    addCommand("exit update", &CRSM::exitUpdate, Management, UserType::Max, "Zusätzlich zu " CMD_SIGN "exit detach wird im laufenden Spiel angekündigt, dass der Server-Manager wegen eines Updates beendet wird.");
+    addCommand("exit update", &CRSM::exitUpdate, Management, UserType::Max, "Zusätzlich zu " + Config.CRSM.CommandSign + "exit detach wird im laufenden Spiel angekündigt, dass der Server-Manager wegen eines Updates beendet wird.");
     addCommand("exit after", &CRSM::exitAfter, Management, UserType::Max, "Beendet den Server-Manager, sobald das laufende Spiel vorbei ist.");
 
     addCommandGroup("config", Management, UserType::Max, "Die config-Befehlsgruppe bietet diverse Befehle zum Verwalten der Konfiguration.");
@@ -1663,7 +1653,7 @@ CMD_FUNCTION_IMPL(help)
             {
                 if(longHelp)
                 {
-                    response += "\n" CMD_SIGN + cmd.name + QString(" ") + (!cmd.argList.isEmpty() ? cmd.argList + QString(" ") : QString("")) + (!cmd.shortDescription.isEmpty() ? QString("- ") + cmd.shortDescription : QString(""));
+                    response += "\n" + Config.CRSM.CommandSign + cmd.name + QString(" ") + (!cmd.argList.isEmpty() ? cmd.argList + QString(" ") : QString("")) + (!cmd.shortDescription.isEmpty() ? QString("- ") + cmd.shortDescription : QString(""));
                 }
                 else
                     response += cmd.name + ", ";
@@ -1672,7 +1662,7 @@ CMD_FUNCTION_IMPL(help)
         if(!longHelp)
             response = response.left(response.length() - 2);
         respond(client, response + "\n");
-        respond(client, "Genauere Informationen zu einem Befehl mit " CMD_SIGN "help <Befehlsname>\n"
+        respond(client, "Genauere Informationen zu einem Befehl mit " + Config.CRSM.CommandSign + "help <Befehlsname>\n"
                         "[Sachen in eckigen Klammern sind optional]\n"
                         "<Sachen in spitzen Klammern sind benoetigte Argumente>\n"
                         "Beim Einsetzen der Argumente werden <> bzw. [] weggelassen.\n");
@@ -1682,7 +1672,7 @@ CMD_FUNCTION_IMPL(help)
         if(cmdExists(args, client.interface))
         {
             const CmdFunctionRef& cmd = cmds.value(args);
-            respond(client, CMD_SIGN + cmd.name + " " + (!cmd.argList.isEmpty() ? cmd.argList + " " : "") + (!cmd.longDescription.isEmpty() ? "- " + cmd.longDescription : !cmd.shortDescription.isEmpty() ? "- " + cmd.shortDescription : "") + "\n");
+            respond(client, Config.CRSM.CommandSign + cmd.name + " " + (!cmd.argList.isEmpty() ? cmd.argList + " " : "") + (!cmd.longDescription.isEmpty() ? "- " + cmd.longDescription : !cmd.shortDescription.isEmpty() ? "- " + cmd.shortDescription : "") + "\n");
             if(cmdGroups.contains(args))
             {
                 bool first = true;
@@ -1696,7 +1686,7 @@ CMD_FUNCTION_IMPL(help)
                         if(cmdRef.userType > userType || !(cmdRef.interfaces & client.interface)) continue;
                         if(first)
                         {
-                            response = "Verfügbare Unterbefehle für " + args + " (nähere Infos zu einem Befehl mit " CMD_SIGN "help " + args + " <Unterbefehlsname>):\n ";
+                            response = "Verfügbare Unterbefehle für " + args + " (nähere Infos zu einem Befehl mit " + Config.CRSM.CommandSign + "help " + args + " <Unterbefehlsname>):\n ";
                             first = false;
                         }
                         else
@@ -1958,7 +1948,7 @@ CMD_FUNCTION_IMPL(io)
 CMD_FUNCTION_IMPL(passToClonkPcName)
     if(args == Config.Auto.Volatile.Clonk.ServerNick || args == Config.Auto.Volatile.Clonk.ServerPCName)
     {
-        respond(client, CMD_SIGN + cmd + " kann nicht auf den Server angewendet werden!\n");
+        respond(client, Config.CRSM.CommandSign + cmd + " kann nicht auf den Server angewendet werden!\n");
         return;
     }
     ClientInfo info;
@@ -2048,7 +2038,7 @@ CMD_FUNCTION_IMPL(passToClonkNumeric)
     int number = args.toInt(&ok);
     if(!ok)
     {
-        respond(client, CMD_SIGN + cmd + " kann nur mit einer Zahl verwendet werden!\n");
+        respond(client, Config.CRSM.CommandSign + cmd + " kann nur mit einer Zahl verwendet werden!\n");
     }
     else
     {
@@ -2059,7 +2049,7 @@ CMD_FUNCTION_IMPL(passToClonkNumeric)
 CMD_FUNCTION_IMPL(passToClonkOnOff)
     if(args != "on" && args != "off")
     {
-        respond(client, CMD_SIGN + cmd + " kann nur mit \"on\" oder \"off\" verwendet werden.\n");
+        respond(client, Config.CRSM.CommandSign + cmd + " kann nur mit \"on\" oder \"off\" verwendet werden.\n");
     }
     else
     {
@@ -2221,7 +2211,7 @@ IRC_CHECK_CALLBACK_IMPL(ircModCmd)
         if(!Config.Auto.Hosting.AliasWishes.isEmpty())
         {
             respond(subject, QString::number(Config.Auto.Hosting.AliasWishes.size()) + " neue" + (Config.Auto.Hosting.AliasWishes.size() > 1 ? QString("") : QString("r")) + " Aliasw" + (Config.Auto.Hosting.AliasWishes.size() > 1 ? QString("ü") : QString("u")) + "nsch" + (Config.Auto.Hosting.AliasWishes.size() > 1 ? QString("e") : QString("")) + " " + (Config.Auto.Hosting.AliasWishes.size() > 1 ? QString("sind") : QString("ist")) + " verfügbar.", RespondType::PrivateNotice);
-            respond(subject, "Bitte bei nächster Gelegenheit mit " CMD_SIGN "aliaswishes bearbeiten.", RespondType::PrivateNotice);
+            respond(subject, "Bitte bei nächster Gelegenheit mit " + Config.CRSM.CommandSign + "aliaswishes bearbeiten.", RespondType::PrivateNotice);
         }
     }
     else