Solved Command is just returning itself

I have this code:

Code:java

 
public class RankCommand implements CommandExecutor{
 
    Main plugin;
 
    public RankCommand(Main instance){
        plugin = instance;
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        if(cmd.getName().equalsIgnoreCase("setrank")){
            if(args.length <= 1){
                sender.sendMessage(ChatColor.GRAY + "Please use " + ChatColor.YELLOW + "/setrank <player> <rank>" + ChatColor.GRAY + ".");
                return true;
            }
            if(args.length == 2){
 
                Player target = Bukkit.getServer().getPlayer(args[0]);
 
                if(target == null){
                    sender.sendMessage(ChatColor.GRAY + "Could not find the player " + ChatColor.YELLOW + args[0]);
                    return true;
                }
 
                if(args[1].equalsIgnoreCase("user")){
                    plugin.rank.put(target.getUniqueId(), "user");
                    return true;
                }
 
                if(args[1].equalsIgnoreCase("admin")){
                    plugin.rank.put(target.getUniqueId(), "admin");
                    return true;
                }
            }
            if(args.length > 2){
                sender.sendMessage(ChatColor.GRAY + "Please use " + ChatColor.YELLOW + "/setrank <player> <rank>" + ChatColor.GRAY + ".");
                return true;
            }
        }
        return true;
    }
}
 
 

And everytime I use the /setrank command, it returns "/setrank <player> <rank>".

My plugin.yml:

Code:java

 
name: RankTest
main: me.stenterstal.pack.Main
version: 1.0
author: stenterstal
commands:
    setrank:
        usage: /<command> <player> <rank>
        description: Applies a rank to the specific player.
 

Anyone that can help me with this?

The goal of the code is to save the string "user" or "admin" bound to the players UUID in a hashmap so I don't have to use an external plugin.

Thanks, Sten.

 

at the end you wrote return true; make that return false;

 

/setrank is still returning the usage in the plugin.yml

 

It returns the usage from the plugin.yml when the command returns false, so make sure you return true in the right places.

 

@CoolDude53 btw why do people put usage in the .yml? ive never seen it used

 

@Corndogoz To keep record what the arguments are whether they are required or not. You can't create a valid command without putting the usage in the plugin.yml.