InventoryClickEvent

Hi, Im trying to make a plugin where a cusom inventory opens when a block is right clicked, everything seems to be working however I wondered if there was a way to find the location of the block clicked through Inventory Click Event.

Heres My event:

Code:java

@EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        Player player = (Player) event.getWhoClicked();
        ItemStack clicked = event.getCurrentItem();
        Inventory inventory = event.getInventory();
        if (inventory.getName().equals(Main.myInventory.getName())) {
        if (clicked.getType() == Material.IRON_BLOCK) {   
        event.setCancelled(true);   
        player.closeInventory();
        //InventoryHolder test = event.getInventory().getHolder(); 
        //Location loc = (Location) ((Block)test).getLocation();    <- Tryed this gives Null Pointer Exception
              }
        else if (clicked.getType() == Material.GOLD_BLOCK) {   
          event.setCancelled(true);   
            player.closeInventory();
        }
        }
        }

Many thanks.

 

Anyone?

 

There is no way to add secret data to an inventory... however you could do a workaround and set a decoy block in the corner of the inventory with data on it (via lore or NBT tags)...

 

Thanks, I would never of thought of this however is there away to add lore or NBT data that the player cant see? The player wouldn't be able to edit or change the inventory so thats fine.

 

Custom NBT can be added to a stack (the player won't see it), however this cannot be accomplished through the Bukkit API. To do it, you'd either have to use the NMS classes representing NBT, or you could use reflection. I know Comphenix has ProtocolLib, which contains NBT manipulation wrapper classes (as well as other MC reflection goodies), or you could just do it raw.

Non-cached, slightly inefficient example of setting an NBT tag via reflection:

Code:java

 
ItemStack item = ...; // This is the ItemStack you want to hold NBT data
 
try{
String nmsPackage = "net.minecraft.server." + Bukkit.getServer().getClass().getPackage().getName()
.split(Pattern.quote(ClassUtils.PACKAGE_SEPARATOR))[3]; // Get the NMS package
// This example uses an integer array data tag to store the data - this is good for block locations
Constructor<?> nbtIntegerArrayCtor = Class.forName(nmsPackage + ".NBTTagIntArray").getDeclaredConstructor(int[].class);
Object dataTag = nbtIntegerArrayCtor.newInstance(new int[]{0,0,0}); // The int[] contains the data you want the NBT tag to store
Object nmsItemStack = Class.forName(Bukkit.getServer().getClass().getPackage().getName() + ".inventory.CraftItemStack").getMethod("asNMSCopy", ItemStack.class).invoke(null, item);
Object compound = nmsItemStack.getClass().getField("tag").get(nmsItemStack); // The actual NBT data tag instance
if(compound == null){
// Create it a datatag if it doesn't have one
compound = Class.forName(nmsPackage + ".NBTTagCompound").getDeclaredConstructor().newInstance();
}
Class.forName(nmsPackage + ".NBTTagCompound").getDeclaredMethod("set", String.class, nbtIntegerArrayCtor.getDeclaringClass().getSuperclass() /* NBTBase */).invoke(compound, "WHATEVER_YOU_WANT_YOUR_NBT_TAG_TO_BE", dataTag);
nmsItemStack.getClass().getField("tag").set(nmsItemStack, compound);
 
// We've successfully set an NBT tag
// Now, we must reassign the ItemStack to a CraftItemStack containing it
// After this, it is no longer safe to use 'item' with ItemMeta, as it will OVERWRITE this NBT tag
item = (ItemStack) Class.forName(Bukkit.getServer().getClass().getPackage().getName() + ".inventory.CraftItemStack").getDeclaredMethod("asCraftMirror",
nmsItemStack.getClass()).invoke(null, nmsItemStack);
}catch(Exception ex){
// Error occurred during reflection, deal with it
}
 

As you can see, to actually set NBT data, you need to access NMS, which I do here using reflection. You can read this saved data from a CraftItemStack if you can get the NMS handle field, the tag instance, and so on.

EDIT: I've been reading through the CraftBukkit source, and it looks like using any NBT tag that does not exist in ItemMeta will be overwritten by the clone/mirror methods in CraftItemStack. However, assuming it does not crash the client, I think you can add invalid data to valid NBT tags, assuming you use the same NBT type, which will not be overwritten as long as you don't use ItemMeta with the NMS modified stack.