Command not registering?

I am having a bit of a problem with my plugin. So far, I have made it to the point of where I have 2 kits, fisherman and PvP, and fisherman is working perfectly fine with a GUI and everything. Strangely enough, its the PvP Kit I am having problems with. When I type /pvp on my server, absolutely nothing happens. I tried debugging already and nothing showed up. I cant figure out whats going on and I hope someone can help me fix it. Thanks!

Main Class:

Code:java

package me.TCOB055.MultiKits;
 
import java.util.ArrayList;
 
import org.bukkit.Bukkit;
import org.bukkit.event.Listener;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
    public GUIListener guilisten = new GUIListener(this);
    public Events Events = new Events(this);
    //array lists for kits
    ArrayList<String> PvP = new ArrayList<String>();
    ArrayList<String> Fisherman = new ArrayList<String>();
    ArrayList<String> kitused = new ArrayList<String>();
    //end of array lists
 
    public void onEnable() {
        commands();
        events();
    }
 
    public void commands(){
        getCommand("Fisherman").setExecutor(new Fisherman(this));
        getCommand("PvP").setExecutor(new PvP(this));
        getCommand("kit").setExecutor(new Kit(this));
 
    }
 
    public void events(){
        PluginManager pm = getServer().getPluginManager();
        pm.registerEvents(Events, this);
        pm.registerEvents(guilisten, this);
 
    }
}
 

PvP Class:

Code:java

package me.TCOB055.MultiKits;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
 
public class PvP implements CommandExecutor{
 
    private Main plugin;
    public PvP(Main instance){
        plugin = instance;
    }
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String [] args){
        Player p = (Player) sender;
        if(commandLabel.equalsIgnoreCase("PvP")){
            if(!plugin.PvP.contains(p.getName())) {
                if(!plugin.kitused.contains(p.getName())){
                    plugin.PvP.add(p.getName());
                    plugin.kitused.add(p.getName());
                    p.getInventory().clear();
                    ItemStack PvP = new ItemStack(Material.DIAMOND_SWORD);
                    PvP.addEnchantment(Enchantment.DAMAGE_ALL, 1);
                    p.getInventory().addItem(PvP);
                    p.getInventory().setHelmet(new ItemStack(Material.IRON_HELMET));
                    p.getInventory().setChestplate(new ItemStack(Material.IRON_CHESTPLATE));
                    p.getInventory().setLeggings(new ItemStack(Material.IRON_LEGGINGS));
                    p.getInventory().setBoots(new ItemStack(Material.IRON_BOOTS));
                    for(int i = 0; i < 30; i++){
                        p.getInventory().addItem(new ItemStack(Material.MUSHROOM_SOUP));
                    }
                } else {
                    p.sendMessage(ChatColor.RED + "You have already chosen a kit!");
                }
 
                }else {
                    p.sendMessage(ChatColor.RED + "You have already chosen the PVP kit!");
                }
                }
 
 
        return false;
    }
}
 

Event Class:

Code:java

package me.TCOB055.MultiKits;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.PlayerFishEvent;
 
public class Events implements Listener {
    private Main plugin;
    public Events(Main instance){
        plugin = instance; }
 
    @EventHandler
    public void onPlayerHitFishingrodscorpion(final PlayerFishEvent event) {
            final Player player = event.getPlayer();
            if (event.getCaught() instanceof Player) {
                    final Player caught = (Player) event.getCaught();
                    if (plugin.Fisherman.contains(player.getName())) {
                            if (player.getItemInHand().getType() == Material.FISHING_ROD) {
                                    player.sendMessage(ChatColor.AQUA + "You have fished " + ChatColor.BOLD + ChatColor.RED + caught.getName() + ChatColor.AQUA + " to you!");
                                    caught.teleport(player.getLocation());
                            }
                    }
            }
    }
    @EventHandler
    public void onDeath(PlayerDeathEvent e){
        Player p = e.getEntity();
        if(plugin.kitused.contains(p.getName())){
            plugin.kitused.remove(p.getName());
            if(plugin.Fisherman.contains(p.getName())){
                plugin.Fisherman.remove(p.getName());
                if(plugin.PvP.contains(p.getName())){
                    plugin.PvP.remove(p.getName());
                }
            }
        }
    }
}

Kit Class:

Code:java

package me.TCOB055.MultiKits;
 
import java.util.ArrayList;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Player;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
 
public class Kit implements CommandExecutor{
        private Main plugin;
        public Kit(Main instance){
            plugin = instance;
        }
        public static Inventory kits = Bukkit.createInventory(null, 18, ChatColor.GOLD + "Kits");
 
        public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
            Player p = (Player) sender;
            if(commandLabel.equalsIgnoreCase("kit")){
                kits.clear();
                p.sendMessage(ChatColor.GREEN + "You have opened your kits!");
 
                ItemStack PvP = new ItemStack(Material.DIAMOND_SWORD);
                ItemMeta PvPMeta = PvP.getItemMeta();
                PvPMeta.setDisplayName(ChatColor.GRAY + "PvP");
                PvP.addEnchantment(Enchantment.DAMAGE_ALL, 1);
                ArrayList<String> PvPLore = new ArrayList<String>();
                PvPLore.add(ChatColor.RED + "Diamond Sword - Sharpness 1");
                PvPLore.add(ChatColor.GRAY + "Full iron armour!");
                PvPMeta.setLore(PvPLore);
                PvP.setItemMeta(PvPMeta);
                kits.setItem(0 , PvP);
 
 
 
                ItemStack Fisherman = new ItemStack(Material.FISHING_ROD);
                ItemMeta FishermanMeta = Fisherman.getItemMeta();
                FishermanMeta.setDisplayName(ChatColor.GOLD + "Fisherman");
                Fisherman.addUnsafeEnchantment(Enchantment.DURABILITY, 10);
                ArrayList<String> FishermanLore = new ArrayList<String>();
                FishermanLore.add(ChatColor.AQUA + "Stone Sword - Sharpness 1");
                FishermanLore.add(ChatColor.GOLD + "Hook your enemies and pull them to you!");
                FishermanMeta.setLore(FishermanLore);
                Fisherman.setItemMeta(FishermanMeta);
                kits.setItem(1, Fisherman);
 
 
                p.openInventory(kits);
 
 
 
            }
            return false;
        }
}
 

GUIListener Class:

Code:java

package me.TCOB055.MultiKits;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.Inventory;
 
public class GUIListener implements Listener{
    private Main plugin;
    public GUIListener(Main instance){
        plugin = instance;
    }
    @EventHandler
    public void onClick(InventoryClickEvent event){
        if ((event.getCurrentItem() != null) && (event.getCurrentItem().getItemMeta() != null)){
            Inventory inv = event.getInventory();
            Player p = (Player)event.getWhoClicked();
            if ((inv.getTitle().equals(ChatColor.GOLD + "Kits"))){
                if (event.getCurrentItem().getItemMeta().getDisplayName().equals(ChatColor.GRAY + "PvP")){
                    Bukkit.dispatchCommand(p, "PvP");
                    event.setCancelled(true);
                    p.closeInventory();
 
                }
                if (event.getCurrentItem().getItemMeta().getDisplayName().equals(ChatColor.GOLD + "Fisherman")){
                    Bukkit.dispatchCommand(p, "Fisherman");
                    event.setCancelled(true);
                    p.closeInventory();
 
                }
 
        }
 
    }
 
}
}
 

 

Did you add your commands in plugin.yml?

 

Yes. I made sure to do that. Im not exactly sure if I entered them in correctly though.
FYI the - means spaces. Forums doesnt really allow it so Im using the - to demonstrate spaces.
name: MultiKits
version: 1.0
main: me.TCOB055.MultiKits.Main
author: TCOB055
description: All the servers kits!
commands:
--Fisherman:
--PvP:
--kit:
DanyBv

 

You have nothing added to the commands.
Add at least one thing, like a description.

commands:
--Fisherman:
----description: (description)
--PvP:
----description: (description)
--kit:
----description: (description)

 

I tried doing that but the PvP command still does not work. jeussa

 

TCO_007
Make sure that if the command is being performed by the PvP.java to return with 'true'. Otherwise bukkit will think the command has not been performed by your plugin and sends you the message 'Unknown command'

Put

Code:java

return true;

between lines 42 and 43 of the PvP.java

 

jeussa I tried that and the command still refuses to work D: