Solved Commands not Working...?

So, I am creating a magical items plugin that works a lot off of a lot of event driven code. When testing a particular item, I noticed that none of the commands would work. Here is my onCommand code. Anyone have any suggestions (I can include my onPlayerInteractEvent code too if it helps)?

Code:java

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args[]) {
        Player player = (Player) sender;
        if(label.equalsIgnoreCase("gxorb")) {
            if(rct == false) {
                player.sendMessage(ChatColor.GREEN + "The Pulsar orb has been turned on!");
                rct = true;
            } else {
                rct = false;
                player.sendMessage(ChatColor.GREEN + "The Pulsar Orb has been turned off!");
            }
        }
        return false;
    }

 

Did you add the command to your plugin.yml?

 

Yes. I have essentials as well on my testing server, and it appears in the /help, but it just doesn't run the command. It even appears in the command prompt.

Anyone else (Bump)?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

In your onEnable() put

<classname> <abbreviation> = new <classname>();
getCommand("cmd").setExecutor("<abbreviation>");

Eg
Commands cmd = new Commands();
getCommand("test").setExecutor(cmd);

 

Someone just recently told me that @Override tags don't do anything and there is no need for them. Well, this is exactly one of the cases where putting @Override before your onCommand method would tell you that the fourth parameter of the method should be a single-dimensional String array, not a two-dimensional one.

 

MrPotatoMuncher
What exactly does that do? I don't mean to insult your help, but that doesn't look like it would effect my commands.

Rocoty
I don't have a @Override. Have any other ideas?

 

Jugglernaught
As Rocoty already implied, your last parameter is wrong. Please check out the java docs on that topic.


PS:

Please, look into switch statements, they might help you when creating a more complex command class. http://docs.oracle.com/javase/tutorial/java/nutsandbolts/switch.html

 

Do Return True;

 

That is quite irrelevant to the question.

KraZ__ If he has that method in the main class that wouldn't make any difference at all. Further, it wouldn't solve the problem entirely. Read my first post in this thread.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Issue is not returning true on success, this method will always return false.

 

I can agree to that, definitely.

Whether the command does what it is supposed to is independent on whether the method returns true or false.