Commands Not Executing?

Hi there,

I recently got done developing a mini-game plugin no sooner to find out the commands weren't executing. I type the name of the command but doesn't bother to actually execute the command, all it does is display the usage information that I defined in my plugin.yml. Can anyone help me with this? Thanks in advance!

CommandManager:

Code:java

public class CommandManager implements CommandExecutor {
 
private ArrayList<SubCommand> commands = new ArrayList<SubCommand>();
 
public void setup() {
commands.add(new Create());
commands.add(new Delete());
commands.add(new ForceStart());
commands.add(new ForceStop());
commands.add(new Join());
commands.add(new Leave());
commands.add(new Reload());
commands.add(new SetLocation());
}
 
public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
 
if (!(sender instanceof Player)) {
MessageManager.getInstance().msg(sender, MessageType.BAD, "You cannot use Dodgeball from the console!");
return true;
}
 
Player p = (Player) sender;
 
if (cmd.getName().equalsIgnoreCase("dodgeball")) {
if (args.length == 0) {
for (SubCommand c : commands) {
MessageManager.getInstance().msg(p, MessageType.INFO, "/dodgeball " + c.name() + "(" + aliases(c) + ")" + " - " + c.info());
}
return true;
}
 
SubCommand target = get(args[0]);
 
if (target == null) {
MessageManager.getInstance().msg(p, MessageType.BAD, "/dodgeball " + args[0] + " is not a valid subcommand!");
return true;
}
 
ArrayList<String> a = new ArrayList<String>();
a.addAll(Arrays.asList(args));
a.remove(0);
args = a.toArray(new String[a.size()]);
 
try {
target.onCommand(p, args);
}
 
catch (Exception e) {
MessageManager.getInstance().msg(p, MessageType.BAD, "An error has occured: " + e.getCause());
e.printStackTrace();
}
}
return true;
}
 
private String aliases(SubCommand cmd) {
String fin = "";
 
for (String a : cmd.aliases()) {
fin += a + " | ";
}
 
return fin.substring(0, fin.lastIndexOf(" | "));
}
 
private SubCommand get(String name) {
for (SubCommand cmd : commands) {
if (cmd.name().equalsIgnoreCase(name)) return cmd;
for (String alias : cmd.aliases()) if (name.equalsIgnoreCase(alias)) return cmd;
}
return null;
}
}

Main class (setting up CommandManager):

Code:java

CommandManager cm = new CommandManager();
cm.setup();
getCommand("dodgeball").setExecutor(cm);

SubCommands Class:

Code:java

public abstract class SubCommands {
 
public abstract void onCommand(Player p, String[] args);
 
public abstract String name();
 
public abstract String info();
 
public abstract String[] aliases();
 
}

 

Instead of using getCommand within the main class and .setup(), just use it in the constructor:

Code:java

CommandManager cm = new CommandManager(this);
 
/* CommandManager.java */
 
public CommandManager(YourPlugin plugin) {
    SubCommand commands = new SubCommand[] {
        new Create(),
        new Delete(),
        new ForceStart(),
        new ForceStop(),
        new Join(),
        new Leave(),
        new Reload(),
        new SetLocation()
    };
 
    for (SubCommand cmd : commands) {
        this.commands.add(cmd);
        plugin.getCommand(cmd.getName()).setExecutor(this);
    }
}

Also, you are using "SubCommand" but your class is named "SubCommands"

 

Also make sure that all the commands are registered in Plugin.yml