Item not going away

Im trying to do when an item is clicked in the inventory the item you have in hand dissapears heres my code

Code:java

package me.firstminigame;
 
 
import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.Plugin;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.scoreboard.Scoreboard;
import org.bukkit.scoreboard.ScoreboardManager;
import org.bukkit.scoreboard.Team;
 
public class CTO extends JavaPlugin implements Listener {
    public static CTO plugin;
 
 
    ScoreboardManager sbManager;
    Scoreboard sBoard;
 
    Team Red;
    Team Blue;
    Team Yellow;
    Team Green;
 
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
 
 
        sbManager = Bukkit.getScoreboardManager();
        sBoard = sbManager.getNewScoreboard();
 
    Red = sBoard.registerNewTeam("Red");
    Blue = sBoard.registerNewTeam("Blue");
    Yellow = sBoard.registerNewTeam("Yellow");
        Green = sBoard.registerNewTeam("Green");
 
    }       
 
    public static Inventory myInventory = Bukkit.createInventory(null, 9, "Choose your team");
 
 
 
 
    static {
        myInventory.setItem(0, new ItemStack(Material.REDSTONE_BLOCK, 1));
        myInventory.setItem(1, new ItemStack(Material.LAPIS_BLOCK, 1));
        myInventory.setItem(2, new ItemStack (Material.GOLD_BLOCK, 1));
        myInventory.setItem(3, new ItemStack (Material.EMERALD_BLOCK, 1));
        }
 
 
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
    Player player = event.getPlayer();
    Material mat = player.getItemInHand().getType();
    if(event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    if(mat == Material.IRON_BLOCK){ player.openInventory(myInventory);
 
        }
    }
    }
 
    @EventHandler
    public void onPlayerInteract1(PlayerInteractEvent event) {
    Player player = event.getPlayer();
    Material mat = player.getItemInHand().getType();
    if(event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    if(mat == Material.REDSTONE_BLOCK){ player.openInventory(myInventory);
 
        }
    }
    }
 
    @EventHandler
    public void onPlayerInteract2(PlayerInteractEvent event) {
    Player player = event.getPlayer();
    Material mat = player.getItemInHand().getType();
    if(event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    if(mat == Material.LAPIS_BLOCK){ player.openInventory(myInventory);
 
        }
    }
    }
 
    @EventHandler
    public void onPlayerInteract3(PlayerInteractEvent event) {
    Player player = event.getPlayer();
    Material mat = player.getItemInHand().getType();
    if(event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    if(mat == Material.GOLD_BLOCK){ player.openInventory(myInventory);
 
        }
    }
    }
 
    @EventHandler
    public void onPlayerInteract4(PlayerInteractEvent event) {
    Player player = event.getPlayer();
    Material mat = player.getItemInHand().getType();
    if(event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    if(mat == Material.EMERALD_BLOCK){ player.openInventory(myInventory);
 
        }
    }
    }
 
        @SuppressWarnings("deprecation")
        @EventHandler
        public void onInventoryClick(InventoryClickEvent event) {
        final Player player = (Player) event.getWhoClicked();
        ItemStack clicked = event.getCurrentItem();
        Inventory inventory = event.getInventory();
        if (inventory.getName().equals(myInventory.getName())) {
        if (clicked.getType() == Material.REDSTONE_BLOCK) {
 
        event.setCancelled(true);
        player.closeInventory();
        Red.addPlayer(player);
        player.sendMessage("You are now in the Red Team");
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(CTO.getInstance(), new Runnable() {
            public void run() {
                player.setItemInHand(null);
            }
        }, 3);
        player.updateInventory();
        }
        }
        }
 
 
        @SuppressWarnings("deprecation")
        @EventHandler
        public void onInventoryClick1(InventoryClickEvent event) {
        final Player player = (Player) event.getWhoClicked();
        ItemStack clicked = event.getCurrentItem();
        Inventory inventory = event.getInventory();
        if (inventory.getName().equals(myInventory.getName())) {
        if (clicked.getType() == Material.LAPIS_BLOCK) {
        event.setCancelled(true);
        player.closeInventory();
        Blue.addPlayer(player);
        player.sendMessage("You are now in the Blue Team");
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(CTO.getInstance(), new Runnable() {
            public void run() {
                player.setItemInHand(null);
            }
        }, 3);
        player.updateInventory();
        }
        }
        }
        private static Plugin getInstance() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @SuppressWarnings("deprecation")
        @EventHandler
        public void onInventoryClick2(InventoryClickEvent event) {
        final Player player = (Player) event.getWhoClicked();
        ItemStack clicked = event.getCurrentItem();
        Inventory inventory = event.getInventory();
        if (inventory.getName().equals(myInventory.getName())) {
        if (clicked.getType() == Material.GOLD_BLOCK) {
        event.setCancelled(true);
        player.closeInventory();
        Yellow.addPlayer(player);
        player.sendMessage("You are now in the Yellow Team");
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(CTO.getInstance(), new Runnable() {
        public void run() {
            player.setItemInHand(null);
        }
    }, 3);
        player.updateInventory();
        }
        }
        }
 
        @SuppressWarnings("deprecation")
        @EventHandler
        public void onInventoryClick3(InventoryClickEvent event) {
        final Player player = (Player) event.getWhoClicked();
        ItemStack clicked = event.getCurrentItem();
        Inventory inventory = event.getInventory();
        if (inventory.getName().equals(myInventory.getName())) {
        if (clicked.getType() == Material.EMERALD_BLOCK) {
        event.setCancelled(true);
        player.closeInventory();
        Green.addPlayer(player);
        player.sendMessage("You are now in the Green Team");
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(CTO.getInstance(), new Runnable() {
            public void run() {
                player.setItemInHand(null);
            }
        }, 3);
        player.updateInventory();
        }
        }}}
 
 
 
 
 

But when i click the items in the Inventory the item you have in hand doesnt dissapear

 

Well first off you didnt have to make events for each blocks you couldve done or ( || )
And try putting player.updateInventory() inside your runnable

 

Onlineids That didnt work

 

Any errors? If not toss in some debug messages and tell me how far the code gets.

 

Onlineids No errors

 

Toss a couple debug messages in then and tell me when it stops

 

Onlineids I can't do that now I'm not home

 

Then how did u test what I said before -_-

 

Onlineids I left 10 minutes ago

The debug messages all worked Onlineids

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

 

It looks like you DEFINED run() but never CALLED it. I've never used Runnable before, though, so I may be wrong.

 

it maybe ean521

 

PizzaPixel It may be*

Sorry, had to.

 

ean521 No i called the runnable

LOL

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

 

PizzaPixel

Okay then, I just wasn't sure you did, as I never saw it in the code.


EDIT: Where DO you call it?

 

ean521 Its fine

SkillSam Not working

Anyone?

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

 

Well just use the inventory click event to get the player, get the inventory and get the clicked item and set it to air.

Code:java

 @EventHandler
public void onInventoryClick(InventoryClickEvent e){
if(e.getAction() == InventoryAction.PICKUP_ALL || e.getAction() == InventoryAction.PICKUP_ONE){
Player p = (Player) e.getWhoClicked();
if(p.getItemOnCursor().getType() == Material.LAPIS_BLOCK){
p.getItemOnCursor().setType(Material.AIR);
//Do other stuff here with your plugin.
}
if(p.getItemOnCursor().getType() == Material.REDSTONE_BLOCK){
p.getItemOnCursor().setType(Material.AIR);
//Do other stuff here with your plugin.
 
//Etc etc etc...
}
}
}

 

JBoss925 You dont understand what im doing

 

I have no idea what you're doing. Please explain and maybe I can help.

 

Then please.. enlighten us.

 

Mattkx4 Look, Thats not part of my plugin. Your telling me to do a completely different plugin.