Toggle armour with command

Hey guys,
Today I have a question with toggling my disco armour with a command such as /disco on and /disco off. However, it does not seem to be working. Here is my code.

Code:java

    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event) {
        Player p = event.getPlayer();
        ItemStack blaze = new ItemStack(Material.BLAZE_POWDER);
        ItemMeta im = blaze.getItemMeta();
        im.setDisplayName("§a§lDisco Armour");
        blaze.setItemMeta(im);
        p.getInventory().addItem(blaze);
        p.updateInventory();
 
    }
 
    @EventHandler
    public void onPlayerQuit(PlayerQuitEvent event) {
        Player p = event.getPlayer();
        p.getInventory().remove(Material.BLAZE_POWDER);
        disco.remove(p.getName());
 
    }
 
    @EventHandler(priority = EventPriority.HIGH)
    public void onPlayerUse(PlayerInteractEvent event) {
        Player o = event.getPlayer();
 
        Action a = event.getAction();
        if (a == Action.RIGHT_CLICK_AIR) {
            if (o.getItemInHand().getType() == Material.BLAZE_POWDER
                    && o.hasPermission("disco.use")) {
                if (o.getItemInHand().getItemMeta().getDisplayName()
                        .equals("§a§lDisco Armour")) {
 
                    if (disco.contains(o.getName())) {
                        o.sendMessage(prefix
                                + "Your disco armour has been disabled!");
                        disco.remove(o.getName());
                        o.getInventory().setHelmet(
                                new ItemStack(Material.AIR, 1));
                        o.getInventory().setChestplate(
                                new ItemStack(Material.AIR, 1));
                        o.getInventory().setLeggings(
                                new ItemStack(Material.AIR, 1));
                        o.getInventory().setBoots(
                                new ItemStack(Material.AIR, 1));
                        o.updateInventory();
                    } else if (!disco.contains(o.getName())) {
                        disco.add(o.getName());
                        o.sendMessage(prefix
                                + "Your disco armour has been enabled!");
                        o.getInventory().setHelmet(
                                new ItemStack(Material.LEATHER_HELMET));
                        o.getInventory().setChestplate(
                                new ItemStack(Material.LEATHER_CHESTPLATE));
                        o.getInventory().setLeggings(
                                new ItemStack(Material.LEATHER_LEGGINGS));
                        o.getInventory().setBoots(
                                new ItemStack(Material.LEATHER_BOOTS));
                        Bukkit.getServer()
                                .getScheduler()
                                .scheduleSyncRepeatingTask(this,
                                        new Runnable() {
                                            public Random r = new Random();
 
                                            public void run() {
                                                Color c = Color.fromRGB(
                                                        r.nextInt(255),
                                                        r.nextInt(255),
                                                        r.nextInt(255));
 
                                                for (Player p : Bukkit
                                                        .getServer()
                                                        .getOnlinePlayers()) {
                                                    if (p.getInventory()
                                                            .getHelmet() != null
                                                            && p.getInventory()
                                                                    .getHelmet()
                                                                    .getType() == Material.LEATHER_HELMET) {
                                                        p.getInventory()
                                                                .setHelmet(
                                                                        getColorArmor(
                                                                                Material.LEATHER_HELMET,
                                                                                c));
                                                    }
 
                                                    if (p.getInventory()
                                                            .getChestplate() != null
                                                            && p.getInventory()
                                                                    .getChestplate()
                                                                    .getType() == Material.LEATHER_CHESTPLATE) {
                                                        p.getInventory()
                                                                .setChestplate(
                                                                        getColorArmor(
                                                                                Material.LEATHER_CHESTPLATE,
                                                                                c));
                                                    }
 
                                                    if (p.getInventory()
                                                            .getLeggings() != null
                                                            && p.getInventory()
                                                                    .getLeggings()
                                                                    .getType() == Material.LEATHER_LEGGINGS) {
                                                        p.getInventory()
                                                                .setLeggings(
                                                                        getColorArmor(
                                                                                Material.LEATHER_LEGGINGS,
                                                                                c));
                                                    }
 
                                                    if (p.getInventory()
                                                            .getBoots() != null
                                                            && p.getInventory()
                                                                    .getBoots()
                                                                    .getType() == Material.LEATHER_BOOTS) {
                                                        p.getInventory()
                                                                .setBoots(
                                                                        getColorArmor(
                                                                                Material.LEATHER_BOOTS,
                                                                                c));
                                                    }
                                                }
                                            }
                                        }, 0, 1);
                    }
                }
            }
        }
    }
 
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
    }
 
    public boolean onCommand(CommandSender sender, Command cmd,
            String commandLabel, String[] args) {
        Player player = (Player) sender;
        if (commandLabel.equalsIgnoreCase("disco")) {
            if (args.length == 0) {
                sender.sendMessage(ChatColor.BLUE
                        + "==========================================");
                sender.sendMessage(ChatColor.GOLD
                        + "This plugin was developed by Glasseater84");
                sender.sendMessage(ChatColor.BLUE
                        + "==========================================");
            }
            if (args.length == 1) {
                if (args[0].equalsIgnoreCase("on")
                        || args[0].equalsIgnoreCase("enable")) {
                    if (disco.contains(player.getName())) {
                        player.sendMessage(prefix
                                + "Your disco armour is already enabled!");
                        return false;
                    }
                    if (!disco.contains(player.getName())) {
                        disco.add(player.getName());
                        player.sendMessage(prefix
                                + "Your disco armour is now enabled.");
 
                    }
 
                }
                if (args[0].equalsIgnoreCase("off")
                        || args[0].equalsIgnoreCase("disable")) {
                    if (!disco.contains(player.getName())) {
                        player.sendMessage(prefix
                                + "Your disco armour is already disabled!");
                        return false;
                    }
                    if (disco.contains(player.getName())) {
                        disco.add(player.getName());
                        player.sendMessage(prefix
                                + "Your disco armour is now disabled.");
 
                    }
 
                }
            }
            return false;
 
        }
        return false;
    }
 
    public ItemStack getColorArmor(Material m, Color c) {
        ItemStack i = new ItemStack(m, 1);
        LeatherArmorMeta meta = (LeatherArmorMeta) i.getItemMeta();
        meta.setColor(c);
        i.setItemMeta(meta);
        return i;
    }
}
 

So the only problem is disabling and enabling with the command! Thank you in advance!
P.S I still want the item to work when you click it!

 

You are not doing anything when you run the command! Make a method for equipping the armor!

 

CoolGuy2001 Ok, but, I equip the armor from line 87 down - So what would I do - re-type everything from there?

bump?

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

 

Nope, cut the equipping from the event. Then make a new method like equipArmor(). Paste the code in the method. Call the method equipArmor() in the event, AND the command!

 

CoolGuy2001
He is doing something.

glasseater
Registered events?

 

CraftCreeper6 Yes, my events are registered.

CoolGuy2001 CraftCreeper6 Ok, so I have created my method here:

Code:java

public void setArmor(PlayerInteractEvent e){
    Player p = e.getPlayer();
        p.sendMessage(prefix + "Your disco armour has been enabled!");
        p.getInventory().setHelmet(new ItemStack(Material.LEATHER_HELMET));
        p.getInventory().setChestplate(new ItemStack(Material.LEATHER_CHESTPLATE));
        p.getInventory().setLeggings(new ItemStack(Material.LEATHER_LEGGINGS));
        p.getInventory().setBoots(new ItemStack(Material.LEATHER_BOOTS));
    }
 

But the method cannot be called to 'o' because we use that in the event.

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

 

Look at some basic Java tutorials. Do not put the event in the method. You should not be basing the event off your method. Here is a hint:

Code:

public void equipArmor(Player p)
{
    //Do stuff here
}