getting clicked item.

Hey. I have an inventory and I want it to do a separate thing when they click on a different block.
Here's my code and it doesn't seem to do anything when I click on stone.

Code:java

    public static Inventory Inv = Bukkit.createInventory(null, 18, F.green + "Select a kit!");
 
    //INVENTORY
    static {
        //TP places
        ItemStack snowblock = new ItemStack(Material.SNOW_BLOCK, 1);
        ItemMeta snowmeta = snowblock.getItemMeta();
        snowmeta.setDisplayName(F.gold + F.bold + "Sweets");
        snowmeta.setLore(Arrays.asList(F.gray + "Teleport to the Sweet shop."));
        snowblock.setItemMeta(snowmeta);
        Inv.setItem(9, snowblock);
        //InMate
        ItemStack stone = new ItemStack(Material.STONE, 1);
        ItemMeta stonemeta = stone.getItemMeta();
        stonemeta.setDisplayName(F.pink + "Inmate (Click me)");
        stonemeta.setLore(Arrays.asList(F.gray + "Kit inmate is the most basic kit."));
        stone.setItemMeta(stonemeta);
        Inv.setItem(1, stone);
 
 
    }
 
    public HashMap<String, Long> inmateCool = new HashMap<String, Long>();
 
    @EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        Player player = (Player) event.getWhoClicked(); // The player that clicked the item
        ItemStack clicked = event.getCurrentItem(); // The item that was clicked
        Inventory inventory = event.getInventory(); // The inventory that was clicked in
        if (inventory.getName().equals(Inv.getName())) { // The inventory is our custom Inventory
            if (clicked.getType() == Material.STONE) { // Stone, the inmate kit
                event.setCancelled(true); // Make it so the item cant be taken

 

Did you register your events correctly, and why do you have a static body?

 

ILLOGICAL USE OF STATIC KEYWORD​

​

​

​

PLEASE LEARN JAVA AND OOP​

Or, you could make the ItemStack and Inventory constant by appending the final keyword...

 

https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/
https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/
https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/
https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/
https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/
https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/
https://forums.bukkit.org/threads/tutorial-create-a-inventory-menu.173571/

SuperOriginal

 

TheGamesHawk2001 Just because a tutorial says so doesn't mean it's the absolute right/effiecient way to do something.

 

Well, Can you give me an alternative?

 

If you take a look on youtube or the forums there are a few tutorials / ways to do it, for example this is a tutorial from Turqmelon located at youtube.

 

TheSuddenlyFamous

Please don't link awful YouTube tutorials, it really degrades the quality of these forums.

TheGamesHawk2001

Add final modifiers, it's something that should be constant anyway.

 

INTIALIZE GODDAMMIT! also check if the inv isnt null before casting also check if the click isnt null


http://jd.bukkit.org/rb/doxygen/d7/deb/classorg_1_1bukkit_1_1plugin_1_1java_1_1JavaPlugin.html

http://jd.bukkit.org/rb/apidocs/org/bukkit/plugin/java/JavaPlugin.html