Fixed alias parsing for commands. object.toString().split(",") is NOT how you parse a YAML list!

1 files changed, 8 insertions, 4 deletions

diff --git a/src/main/java/org/bukkit/command/PluginCommandYamlParser.java b/src/main/java/org/bukkit/command/PluginCommandYamlParser.java
index 9b4f8493..2305e0ed 100644
--- a/src/main/java/org/bukkit/command/PluginCommandYamlParser.java
+++ b/src/main/java/org/bukkit/command/PluginCommandYamlParser.java
@@ -9,13 +9,13 @@ import org.bukkit.plugin.Plugin;
 
 public class PluginCommandYamlParser {
 
+    @SuppressWarnings("unchecked")
     public static List<Command> parse(Plugin plugin) {
         List<Command> pluginCmds = new ArrayList<Command>();
         Object object = plugin.getDescription().getCommands();
         if (object == null)
             return pluginCmds;
 
-        @SuppressWarnings("unchecked")
         Map<String, Map<String, Object>> map = (Map<String, Map<String, Object>>)object;
 
         if (map != null) {
@@ -34,9 +34,13 @@ public class PluginCommandYamlParser {
 
                 if (aliases != null) {
                     List<String> aliasList = new ArrayList<String>();
-
-                    for(String a : aliases.toString().split(",")) {
-                            aliasList.add(a);
+                    
+                    if (aliases instanceof List) {
+                        for (Object o : (List<Object>)aliases) {
+                            aliasList.add(o.toString());
+                        }
+                    } else {
+                        aliasList.add(aliases.toString());
                     }
 
                     newCmd.setAliases(aliasList);