addressed comments from pr, chat command usage now supports multiple outputs.

Author: sp00ktober

NebulaWorld/Chat/Commands/ClearCommandHandler.cs

@@ -19,9 +19,9 @@ public void Execute(ChatWindow window, string[] parameters)
             window.ClearChat(message => message.MessageType.IsCommandMessage());
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return $"[all|commands]";
+            return new string[] { $"[all|commands]" };
         }
 
         public string GetDescription()

NebulaWorld/Chat/Commands/HelpCommandHandler.cs

@@ -43,9 +43,16 @@ private static string GetCommandDetailsOutput(string commandName)
             string aliasList = ChatCommandRegistry.GetCommandAliases(commandName).Join(FullCommandName);
 
             StringBuilder sb = new StringBuilder();
+            foreach(string usage in handler.GetUsage())
+            {
+                sb.Append($"Usage: {FullCommandName(commandName)} {usage}\n");
+            }
+            string usageString = sb.ToString();
+
+            sb.Clear();
             sb.Append($"Command {commandName} - {handler.GetDescription()}\n");
             sb.Append($"Aliases: {aliasList}\n");
-            sb.Append($"Usage: {FullCommandName(commandName)} {handler.GetUsage()}");
+            sb.Append($"{usageString}");
             return sb.ToString();
         }
 
@@ -82,9 +89,9 @@ public string GetDescription()
             return "Get list of existing commands and their usage";
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "[command name]";
+            return new string[] { "[command name]" };
         }
     }
 }

NebulaWorld/Chat/Commands/IChatCommandHandler.cs

@@ -16,7 +16,7 @@ public interface IChatCommandHandler
         /// <summary>
         /// Provide argument usage (If needed) without starting with command name
         /// </summary>
-        string GetUsage();
+        string[] GetUsage();
     }
 
     public class ChatCommandUsageException : Exception

NebulaWorld/Chat/Commands/InfoCommandHandler.cs

@@ -151,9 +151,9 @@ public string GetDescription()
             return "Get information about server";
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "[full]";
+            return new string[] { "[full]" };
         }
     }
 }

NebulaWorld/Chat/Commands/NavigateCommandHandler.cs

@@ -91,16 +91,17 @@ public void Execute(ChatWindow window, string[] parameters)
                 }
             }
 
-            window.SendLocalChatMessage("Failed to start navigation, please check your input.", ChatMessageType.CommandOutputMessage);
+            window.SendLocalChatMessage("Failed to start navigation, please check your input.", ChatMessageType.CommandErrorMessage);
         }
 
         public string GetDescription()
         {
             return "Start navigating to a specified destination";
         }
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "[planet name | planet id | star name | star id | clear]\r\nUsage: /n player [player id | playr name]";
+            return new string[] {   "<planet name | planet id | star name | star id | clear>",
+                                    "player <player id | player name>" };
         }
     }
 }

NebulaWorld/Chat/Commands/PingCommandHandler.cs

@@ -37,9 +37,9 @@ public string GetDescription()
             return "Test command";
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "[time delay (seconds)]";
+            return new string[] { "[time delay (seconds)]" };
         }
     }
 }

NebulaWorld/Chat/Commands/SystemCommandHandler.cs

@@ -38,15 +38,15 @@ public void Execute(ChatWindow window, string[] parameters)
                 resp = "Could not find given star '" + input + "'";
             }
 
-            window.SendLocalChatMessage(resp, ChatMessageType.CommandOutputMessage);
+            window.SendLocalChatMessage(resp, ChatMessageType.CommandErrorMessage);
         }
         public string GetDescription()
         {
             return "List planets in a system";
         }
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "[star name]";
+            return new string[] { "[star name]" };
         }
     }
 }

NebulaWorld/Chat/Commands/WhisperCommandHandler.cs

@@ -52,9 +52,9 @@ public string GetDescription()
             return $"Send direct message to player. Use {ChatCommandRegistry.CommandPrefix}who for valid user names";
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "<player> <message>";
+            return new string[] { "<player> <message>" };
         }
 
         public static void SendWhisperToLocalPlayer(string sender, string mesageBody)

NebulaWorld/Chat/Commands/WhoCommandHandler.cs

@@ -29,9 +29,9 @@ public string GetDescription()
             return "List all players";
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "";
+            return new string[] { "" };
         }
 
         public static string BuildResultPayload(IPlayerData[] allPlayers, ILocalPlayer hostPlayer)

NebulaWorld/Chat/Commands/XConsoleCommandHandler.cs

@@ -33,9 +33,9 @@ public string GetDescription()
             return "Execute developer console command";
         }
 
-        public string GetUsage()
+        public string[] GetUsage()
         {
-            return "[XConsole command]";
+            return new string[] { "[XConsole command]" };
         }
     }
 }