From d3b8ede229c0789d13c24951fa57af312d10962c Mon Sep 17 00:00:00 2001
From: TheUbMunster <66451362+TheUbMunster@users.noreply.github.com>
Date: Thu, 28 Jul 2022 19:16:24 -0600
Subject: [PATCH] Messages are now split up into lengths small enough to not
 cause the discord bot to throw an exception with too long a message.

---
 Server/DiscordBot.cs | 33 +++++++++++++++++++++++++--------
 Server/Program.cs    |  6 ++++++
 2 files changed, 31 insertions(+), 8 deletions(-)

diff --git a/Server/DiscordBot.cs b/Server/DiscordBot.cs
index e11813a..e2c4765 100644
--- a/Server/DiscordBot.cs
+++ b/Server/DiscordBot.cs
@@ -46,11 +46,21 @@ public class DiscordBot {
         }
     }
 
+    private static List<string> SplitMessage(string message, int maxSizePerElem = 2000)
+    {
+        List<string> result = new List<string>();
+        for (int i = 0; i < message.Length; i += maxSizePerElem) 
+        {
+            result.Add(message.Substring(i, message.Length - i < maxSizePerElem ? message.Length - i : maxSizePerElem));
+        }
+        return result;
+    }
+
     private async void Log(string source, string level, string text, ConsoleColor _) {
         try {
             if (DiscordClient != null && LogChannel != null) {
-                await DiscordClient.SendMessageAsync(LogChannel,
-                    $"```{Logger.PrefixNewLines(text, $"{level} [{source}]")}```");
+                foreach (string mesg in SplitMessage(Logger.PrefixNewLines(text, $"{level} [{source}]"), 1994)) //room for 6 '`'
+                    await DiscordClient.SendMessageAsync(LogChannel, $"```{mesg}```");
             }
         } catch (Exception e) {
             // don't log again, it'll just stack overflow the server!
@@ -83,18 +93,25 @@ public class DiscordBot {
                 if (args.Author.IsCurrent) return;
                 try {
                     DiscordMessage msg = args.Message;
+                    string? resp = null;
                     if (string.IsNullOrEmpty(Prefix)) {
                         await msg.Channel.TriggerTypingAsync();
-                        await msg.RespondAsync(string.Join('\n',
-                            CommandHandler.GetResult(msg.Content).ReturnStrings));
+                        resp = string.Join('\n', CommandHandler.GetResult(msg.Content).ReturnStrings);
                     } else if (msg.Content.StartsWith(Prefix)) {
                         await msg.Channel.TriggerTypingAsync();
-                        await msg.RespondAsync(string.Join('\n',
-                            CommandHandler.GetResult(msg.Content[Prefix.Length..]).ReturnStrings));
+                        resp = string.Join('\n', CommandHandler.GetResult(msg.Content[Prefix.Length..]).ReturnStrings);
                     } else if (msg.Content.StartsWith(mentionPrefix)) {
                         await msg.Channel.TriggerTypingAsync();
-                        await msg.RespondAsync(string.Join('\n',
-                            CommandHandler.GetResult(msg.Content[mentionPrefix.Length..]).ReturnStrings));
+                        resp = string.Join('\n', CommandHandler.GetResult(msg.Content[mentionPrefix.Length..]).ReturnStrings);
+                    }
+                    if (resp != null)
+                    {
+                        foreach (string mesg in SplitMessage(resp))
+                            await msg.RespondAsync(mesg);
+                    }
+                    else
+                    {
+                        await msg.RespondAsync("Couldn't determine a response for the query.");
                     }
                 } catch (Exception e) {
                     Logger.Error(e);
diff --git a/Server/Program.cs b/Server/Program.cs
index 045a05d..e6882f2 100644
--- a/Server/Program.cs
+++ b/Server/Program.cs
@@ -196,6 +196,12 @@ server.PacketHandler = (c, p) => {
     return true;
 };
 
+#region Testing
+//CommandHandler.RegisterCommand("testreallylongcommand", args => {
+//    return new string('a', 7000);
+//});
+#endregion
+
 CommandHandler.RegisterCommand("rejoin", args => {
     if (args.Length  == 0) {
         return "Usage: rejoin <* | usernames...>";