[SOLVED]onCommand doesn't work?

onCommand always returns false in my plugin, no errors, just false

here is my code:

Code:

package me.tbsman.Lockdown;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerLoginEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class LockdownPlus extends JavaPlugin implements Listener {
    public boolean lockdownmode;
   
    @Override
    public void onEnable() {
    }
    @Override
    public void onDisable() {
    }
    public boolean onCommand(CommandSender sender, Command cmd, String label,String[] args) {
        if (cmd.equals("lockdown") && sender.hasPermission("admin.yay")) {
            if (lockdownmode != true) {
                lockdownmode = true;
                for(Player player : Bukkit.getOnlinePlayers()) {
                    if (!player.hasPermission("admin.yay"))
                        player.kickPlayer("This Server is currently under maintenace");
                }
                sender.sendMessage("The Server is now in " + ChatColor.GOLD + "LOCKDOWN mode");
                return true;
            } else {
                lockdownmode = false;
                sender.sendMessage("The Server is now in " + ChatColor.GOLD + "open mode!");
                return true;
            }
           
           
           
            }
            return false;
        }
       
       
   
    public void onPlayerJoin(PlayerLoginEvent event) {
        if (event.getPlayer().hasPermission("admin.yay") != true && lockdownmode == true) {
            event.getPlayer().kickPlayer("This Server is currently under maintenace");
        }
    }
}

I'm sure I'm just missing something simple.

 

Did you register the command in plugin.yml?
Also, try adding @Override above onCommand.

Also, your event misses the @EventHandler annotation.

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

 

oncommands dont need @EventHandler, I think your first solution is the correct one.

 

You're comparing a Command and a String. You want cmd.getName().equals

 

Thanks, works now. That's what you get when coding at night

 

No, your event needs the annotation.

 

oh, didnt event see that tiny event at the bottom, my bad XD