Inventory Click Event Help

Hello everyone

I have been using InventoryClickEvent, cancelling it and it works, however players can bypass this by hoovering over the slot and pressing the number on the hotbar.

Is there anyway to cancel that?

Thanks

 

Code:java

 Inventory myInventory = Bukkit.createInventory(null, 9, "myInventory");
 
@EventHandler
public void click(InventoryClickEvent event)
{
 
Inventory inventory = event.getInventory();
 
if(inventory.getName().equalsIgnoreCase(myInventory.getName()))
{
event.setCancelled(true);
}
 
}

hellobrad100 This should fix that for you. If i helped please like my post!

 

Hi

Thanks for the help, but that didnt fix the issue.

I know how to cancel the click event, my problem is the number on the hotbar feature.

So I hover on the place I want the thing in my hotbar to go like so:


Then I press "1" on my keyboard and then the item will go into the place I hover on, like so:


My question is, is there a way to cancel that along with all clicking?
Thanks

 

hellobrad100, Make another event handler with InventoryInteractEvent, They way i showed you, But with that listener instead.

 

Maybe use also InventoryMoveItemEvent ?
I think that may be this event you want to cancel.

http://jd.bukkit.org/rb/doxygen/da/...t_1_1inventory_1_1InventoryMoveItemEvent.html

 

Hartorn, InventoryInteractEvent would be more efficient just because it'll stop all interact events like taking items etc. Just for that inventory.

 

xAstraah :From the doc

I thought when seeing that in the doc that you could not use it, but instead other events extending this one.

But if working, it is a better solution

 

Hartorn, Yeah i always use it for when i code. That's why i said it

 

xAstraah : Never had to use event on Inventory. Good to know

 

I tried this but it dont work at all. Any ideas?

Code:java

@EventHandler
    public static void onMove(InventoryMoveItemEvent event)
    {
        ArrayList<String> lore;
        ItemStack is = new ItemStack(Material.DIAMOND_SWORD);
        ItemMeta iss = is.getItemMeta();
        lore = new ArrayList<String>();
        lore.add(ChatColor.RED + "You can not move this!");
        iss.setLore(lore);
        is.setItemMeta(iss);
 
        if(event.getItem().getItemMeta().getLore() == lore)
        {
            event.setCancelled(true);
        }
    }

 

hellobrad100, Mate you've done it wrong. Don't use lore and stuff to set events cancelled.
Do the way i cancelled my event in my code i sent you using a Inventory and a event.getInventory();

hellobrad100, Try this

Code:java

    Inventory myInventory = Bukkit.createInventory(null, 9, "myInventory");
 
    @EventHandler
    public void click(InventoryInteractEvent event)
    {
 
        Inventory inventory = event.getInventory();
 
        if(inventory.getName().equalsIgnoreCase(myInventory.getName()))
        {
            event.setCancelled(true);
        }
 
    }
 

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

 

xAstraah
I am not using the code above to cancel the event.
I only want that item not to move, everything else is fine.

So the item with the lore doesn't move, but everything else does.

I know about that method, it didn't work as I wanted it to. That is why I am here.

 

hellobrad100, Right, See what you have '== lore' change that to get the Inventory and to get that item. Because at the momment your trying to just find a lore.

 

When Event is called using Hotbar buttons, the clicked item not equals to the moved item.
You need check if event is not cancelled.
If the event is not cancelled check if HotbarButton not equals to -1 (this means that the player used the Hotbar Buttons, because when user do a normal click, HotbarButton = -1)

Code:java

if (event.getHotbarButton() != -1){ }

Later, check the Item of the Player Inventory in the hotbar Button:

Code:java

ItemStack itemMoved = evt.getWhoClicked().getInventory().getItem(evt.getHotbarButton());

And check this item again for Cancel if is necessary.

Full code:

Code:java

if (event.getHotbarButton() != -1){
ItemStack itemMoved = evt.getWhoClicked().getInventory().getItem(evt.getHotbarButton());
//Check if cant move this and cancel.
}