Chest still opens when event is cancelled.

So I have this code:

Code:

if(event.getAction().equals(Action.RIGHT_CLICK_BLOCK) && event.getClickedBlock().getType().equals(Material.CHEST) && chestGroup.contains(p.getName())){
            event.setCancelled(true);
            Location loc = event.getClickedBlock().getLocation();
            Chest chest = (Chest) loc.getBlock().getState();
            Inventory inv = chest.getInventory();
            p.openInventory(inv);
            p.sendMessage("You opened an invisible chest");
        }

The issue is, the chest still opens event though the event is cancelled. The interesting part of this is that the event is cancelled, because when the the chest code is removed the chest does not open. Any ideas on how I could stop this from happening?

 

Try setting the priority to highest

 

@EventHandler
public void onPlayerRightClickEvent(Action event){
CODEEEEEEEE
}

 

This is the only way I have been able to do it without it glitching. not ideal but you don't really have another option.

 

The issue isn't that the event isn't being cancelled, because when the code looks like this

Code:

if(event.getAction().equals(Action.RIGHT_CLICK_BLOCK) && event.getClickedBlock().getType().equals(Material.CHEST) && chestGroup.contains(p.getName())){
            event.setCancelled(true);
            Location loc = event.getClickedBlock().getLocation();
            //Chest chest = (Chest) loc.getBlock().getState();
            //Inventory inv = chest.getInventory();
            //p.openInventory(inv);
            p.sendMessage("You opened an invisible chest");
        }

The chest does not open. Opening the inventory seems to open the chest.

 

I don't know if this is just me but...
aren't you just cancelling the event and then getting the cancelled chest location, getting the cancelled chest, getting the cancelled chest's inventory and then making the player open it?
So essentially opening the cancelled chest as normal?
If I am completely wrong, please tell me...

 

The point of this is to hide the opening animation and sound effect by getting the location of the chest and opening the inventory. It is designed so invisible admins can open chests without players seeing it.

 

OK, thank you. I thought I was missing something!

 

DONT use .equals. Use ==.
.equals is for strings (and some other things)

For example:

Code:

event.getClickedBlock().getType() == Material.CHEST

 

Grave digging, but no it's not... .equals is used to compare objects, not just strings.