Solved Commands not Executing

I've set up a few different commands with multiple Command Executors and all but one is working. I am setting the executors in

Code:

    private GamemodeSelector gamemodeSelectorExecutor;
    private PlayerKicker kickerExecutor;
    private PlayerBanner bannerExecutor;
    private PlayerTeleporter teleporterExecutor;

public void onEnable(){
    getCommand("selector").setExecutor(gamemodeSelectorExecutor);
   
    getCommand("kicker").setExecutor(kickerExecutor);
   
    getCommand("banhammer").setExecutor(bannerExecutor);
   
    getCommand("teleporter").setExecutor(teleporterExecutor);
}

Then the code in the executor looks like

Code:

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {
        if (cmd.getName().equalsIgnoreCase("kicker") && sender instanceof Player){
            Player player = (Player) sender;

            if (player.hasPermission("cephalstaff.useKicker")){
                openGUI(player);
               
            }
            else {
                player.sendMessage(ChatColor.RED + "You do not have permission to use the kicker");
            }
            return true;
           
        }
        return false;
    }

Ok, never mind about just one, they all aren't working now

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

 

You aren't creating a new instance of any of your command classes.

Keeping them like;

private PlayerKicker kickerExecutor;

and not doing anything with them isn't really doing much.

Create a new instance of them in your onEnable().

 

@Butter1484
Check in your command class have you defined the main class? Like this

Code:java

 private kickerExecutor plugin;
      public kickerExecutor( [ Your main class] plugin) {
      this.plugin = plugin;
}

 

@CoolGamerXD just a correction on your code, you mean
private PlayerKicker.. Not private kickerExecutor

 

If this is solved, mark as Solved.

 

@CoolGamerXD All of the executors are also listeners and are already defined because of that

 

@Butter1484 Registering events and registering commands both are different.

 

Hmmmm doubt it. Show us the code you think is "defining" it.

 

In my main class I have

Code:

new PlayerKicker(this);

Then in the PlayerKicker class I have

Code:

    public PlayerKicker(CephalStaff cephalStaff) {
        cephalStaff.getServer().getPluginManager().registerEvents(this, cephalStaff);
    }

The events are registering but the commands in the class do not work

 

@timtower If you look at my original post you will see I do have that code

 

@timtower The commands are not actually executing

 

Feel free to check out all the code https://bitbucket.org/cephalmcdevteam/cephalstaff/

 

That did it. Thanks @timtower