Commands not working as expected

So, I have the following problem: Whenever I use a command from my plugin, instead of doing what it's supposed to, it simply displays the "usage:" field on the plugin.yml
I really don't know what's wrong, I already checked my code twice but I can't figurethis out. Some help would be greatly appreciated, thanks.

Show Spoiler Hide Spoiler

Code:java

package me.Heracles421.MyMCMusic;
 
import java.io.File;
import java.util.HashMap;
import java.util.Map;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
import com.xxmicloxx.NoteBlockAPI.NBSDecoder;
import com.xxmicloxx.NoteBlockAPI.RadioSongPlayer;
import com.xxmicloxx.NoteBlockAPI.Song;
import com.xxmicloxx.NoteBlockAPI.SongPlayer;
 
public class Main extends JavaPlugin {
    public Map<Integer, Song> tracks = new HashMap<Integer, Song>();
    public static Map<String, SongPlayer> playerMap = new HashMap<String, SongPlayer>();
 
    File trackspath = new File(getDataFolder() + "/tracks");
    String prefix = ChatColor.BLUE + "[MyMCMusic]";
 
    @Override
    public void onEnable(){
        getSongs();
    }
 
    @Override
    public void onDisable(){
        // TODO Insert logic to be performed when the plugin is enabled
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String[] args){
        Player player = (Player)sender;
        try {
            if (cmd.getName().equalsIgnoreCase("mcm") && player.hasPermission("mcm.use")){
                if (args.length <= 2){
                    if (args[0].equalsIgnoreCase("help")){
                        player.sendMessage(prefix + ChatColor.GREEN + "by Heracles421");
                        player.sendMessage("--------------------------");
                        player.sendMessage(ChatColor.RED + "/mcm help " + ChatColor.AQUA + "- Displays this help");
                        player.sendMessage(ChatColor.RED + "/mcm list " + ChatColor.AQUA + "- Lists all available tracks");
                        player.sendMessage(ChatColor.RED + "/mcm play [NUMBER] (PLAYER)" + ChatColor.AQUA + "- Plays the chosen number, optionally for the chosen player.");
                        player.sendMessage(ChatColor.RED + "/mcm stop (PLAYER)" + ChatColor.AQUA + "- Stops the current track, optionally for the chosen player.");
                        return true;
                    }
                    else if (args[0].equalsIgnoreCase("list") && player.hasPermission("mcm.list")) {
                        player.sendMessage("These tracks are availabe for playing:");
                        for (int i = 1; i <= tracks.size(); i++){
                            String song = (tracks.get(i)).toString();
                            player.sendMessage(ChatColor.RED + String.valueOf(i) + "- " + ChatColor.AQUA + song);
                        }
                        return true;
                    }
                    else if (args[0].equalsIgnoreCase("play") && player.hasPermission("mcm.play")) {
                        String song = (tracks.get(args[1])).toString();
                        play(new Player[] {player}, args[1]);
                        player.sendMessage(ChatColor.AQUA + "Now playing: " + ChatColor.GREEN + song);
                        return true;
                    }
                    else if (args[0].equalsIgnoreCase("stop") && player.hasPermission("mcm.stop")) {
                        try {
                            if (args.length == 2 && player.hasPermission("mcm.stop.others")){
                                Player indirect = getServer().getPlayer(args[2]);
                                play(new Player[] {indirect}, args[1]);
                            } else {
                                stopPlaying(new Player[] {player});
                                player.sendMessage(ChatColor.AQUA + "You have been removed from the stream!");
                            }
                        } catch (Exception e){
                            player.sendMessage(ChatColor.DARK_RED + args[1] + ChatColor.AQUA + " is not a player or is offline!");
                        }
                        return true;
                    } else {
                        player.sendMessage(ChatColor.DARK_RED + "Unknown command. Please, refer to the help.");
                    }
                }
                else if (args.length == 3 && player.hasPermission("mcm.play.others")) {
                    try {
                        if (args[0].equalsIgnoreCase("play")){
                            Player indirect = getServer().getPlayer(args[2]);
                            play(new Player[] {indirect}, args[1]);
                            return true;
                        }
 
                    } catch (Exception e) {
                        player.sendMessage(ChatColor.DARK_RED + args[1] + ChatColor.AQUA + " is not a player or is offline!");
                    }
                } else {
                    player.sendMessage(ChatColor.DARK_RED + "Unknown command. Please, refer to the help.");
                }
            }
 
        }
        catch (Exception e)
        {
            player.sendMessage(ChatColor.DARK_RED + "Something went wrong! Please check spelling");
            player.sendMessage(ChatColor.RED + "Error:  " + e.getMessage());
        }
        return false;
    }
 
    // Custom functions
    public void stopPlaying(Player[] player){
        for (Player p : player){
            if (playerMap.containsKey(p.getName())){
                (playerMap.get(p.getName())).removePlayer(p);
            }
            playerMap.remove(p.getName());
        }
    }
 
    public void play(Player[] player, String s){
        SongPlayer sp = new RadioSongPlayer(tracks.get(s));
        sp.setAutoDestroy(true);
        for (Player p : player){
            if (playerMap.containsKey(p.getName())){
                (playerMap.get(p.getName())).removePlayer(p);
            }
            sp.addPlayer(p);
            playerMap.put(p.getName(), sp);
        }
        sp.setPlaying(true);
    }
 
    public void getSongs(){
        if (!trackspath.exists()) {
            getDataFolder().mkdir();
            trackspath.mkdir();
            System.out.println(prefix + "Tracks folder doesn't exist. It will be created...");
        }
 
        File[] listOfFiles = trackspath.listFiles();
 
        for (int i = 0; i < listOfFiles.length; i++){
            if (listOfFiles[i].isFile() && listOfFiles[i] != tracks){
                System.out.println(prefix + "New track found! Importing " + listOfFiles[i].getName());
                tracks.put(i, NBSDecoder.parse(listOfFiles[i].getAbsoluteFile()));
            }
        }
    }
}[/i][/i][/i][/i]

 

Heracles421
Remove all of the "return true;" statements, or at least change them to "return false;". Alternatively, you can remove the usage property in plugin.yml.

 

Nope, it is not working yet. I removed all the return true's in there, and also the usage: field, but now it does not display anything, even though it should

 

*bump*
I still don't know what's the problem, help please!

 

if (args.length <= 2){
There's your problem

 

PatoTheBest Why is it wrong? Maybe I'm just being stupid or blind, or both, but I really don't know why it won't work

 

Heracles421
Perhaps the player doesn't have the permission node? Try removing that clause and see if it works.

 

Adamki11s Nope, it's not the perms part. I removed all the permissions checks and still nothing. let me try to change the args.lenght part, and see what do I get

PatoTheBest args.lenght is not the problem. Something else is causing it and I still haven't found it

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

 

Heracles421
There are several things which must be fixed in your onCommand.

1. You have to use the correct parameters. You are using "CommandSender sender, Command cmd, String[] args" as your params. This is the reason why nothing is working at all, as Bukkit doesn't recognize this method. To fix it, you must do one of two things: add a command executor for each command or add the CommandLabel parameter (It's a string) to make it look like the following: "CommandSender sender, Command cmd, String label, String[] args"
2. You must check if the sender's instance is a player before casting the variable. Add the following line of code in the beginning of the onCommand: if (!(sender instanceof Player)) return false;
3. Remove the big try/catch block. They should be used sparingly and when absolutely necessary. You might want to remove the other try/catch blocks if they aren't needed for sure.
4. When checking "if (args.length <=2)" is wrong. That means the args.length could be 0. You need to make sure that the args.length > 0.

Edit: So many edits so I can keep referring back as I go along, just ignore the edit count.

Also, I am not telling you that everything will work after. I am checking the validity of your code, not the soundness of it.

 

Heracles421
As AoH_Ruthless posted above you are using the wrong method. It's always good practice to use the @Override annotation to prevent this from happening.

 

Adamki11s
Just for clarity's sake, as Heracles421 might not know what the Override annotation does:

• It will not fix the aforementioned issues. It silently tells the parent class "Hey, you are using the method incorrectly". This could be a parameter change, like you did, or a spelling change (onCommnad, for example). It's use is to quickly detect typos / deliberate API changes. So, while he said it "prevents this from happening", a slight error or message is thrown when you use @Override paired with an incorrect method.

 

AoH_Ruthless I'd kiss you if I could.

How was I so stupid to forget the CommandLabel parameter... Anyways, it all seems ok for now. I'll keep an eye on the plugin to see if anything breaks or is not working properly.

 

MooshViolet
What on earth are you talking about? I fixed the issue: there is no event he is listening for at all.