Getting the Data of A Clicked Item in Inventory

Hey you guys, I would really appreciate if you guys could help me out with something.

Code:java

    public static org.bukkit.inventory.Inventory classSelect;
    public static org.bukkit.inventory.Inventory unranked;
    public static org.bukkit.inventory.Inventory vip;
 
    public ItemStack white = new ItemStack(Material.INK_SACK, 1, (short)15);{
    ItemMeta wmeta = white.getItemMeta();
    ArrayList<String> w = new ArrayList<String>();
    wmeta.setDisplayName(ChatColor.YELLOW + "Default Classes");
    w.add(ChatColor.GOLD + "Everyone can access these classes!");
    wmeta.setLore(w);
    white.setItemMeta(wmeta);
}

This works just fine, but the next snippet of code is where it starts to mess up...

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler
      public void onInventoryClick(InventoryClickEvent event) {
      HumanEntity player = event.getWhoClicked();
      @SuppressWarnings("unused")
    MaterialData data = player.getItemInHand().getData();
      ItemMeta wmeta = white.getItemMeta();
      if(event.getInventory().getName().equals(classSelect.getName())) {
          if(event.getCurrentItem().getItemMeta() == wmeta){
              event.setCancelled(true);
                  ((Player) player).openInventory(unranked);
                ((Player) player).updateInventory();
                ((Player) player).playSound(player.getLocation(), Sound.LEVEL_UP, 2F, 1F);
          }
        }
    }

And as you can see, what I am trying to do is open up another inventory, when you click inside of the "classSelect" inventory. What Problem I am running into though is that the items inside of "classSelect" are the same item but with a short of 15, and I need help differentiating those so it can open up the correct inventory when I click on it.

Here is an example of of it: Right-click stick and it opens up classSelect(Which works) click the bonemeal in that inventory(this is where it doesn't work), it cancels and opens up the unranked inventory(That's the part that is the kicker)

Bump

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

 

mamifsidtect
Don't get the item...get the physical items

if one equals the other, do somehting

 

JHG0 And how would I go about doing that? Example code if you could?

 

JHG0 Huh?

 

You won't be able to compare two item meta objects, what exactly are you trying to do? ItemMeta contains lore, display name, enchantments, etc. you would need to compare the piece you care about directly,
This is also completely different from data. If you want to compare data, the easiest way is probably just comparing durability, though it's deprecated.

 

So I would maybe do something like this? NathanWolf
Edit: Tested...doesn't work. Help?

Code:java

 @SuppressWarnings("deprecation")
@EventHandler
public void onInventoryClick(InventoryClickEvent event) {
HumanEntity player = event.getWhoClicked();
@SuppressWarnings("unused")
MaterialData data = player.getItemInHand().getData();
if(event.getInventory().getName().equals(classSelect.getName())) {
if(event.getCurrentItem().getItemMeta().equals(white)){
event.setCancelled(true);
((Player) player).openInventory(unranked);
((Player) player).updateInventory();
((Player) player).playSound(player.getLocation(), Sound.LEVEL_UP, 2F, 1F);
}
}
}

 

mamifsidtect A quick thing you should probably know when working with inventory click event:
Event#getCurrentItem() returns the ItemStack within the slot that the player's mouse if hovering over. This means that, with the way you are currently doing this, your method has a chance to return as null. This chance is increase greatly if the size of the ItemStack clicked on is 1 because it will mean that even a right click (which is meant to just half the stack) will grab the entire slot's contents and make getCurrentItems() return value null.

 

What exactly are you trying to compare?

 

Thank you for that JRL1004 and NathanWolf I'm trying to see if they grabbed bonemeal or if they grabbed the orange or green dye, this might help [pic]

 

mamifsidtect Event#getCursor().getItemMeta().getDisplayname() and compare the strings?

 

JRL1004 Let me test that really quick

Alright so I did this instead

Code:

    @SuppressWarnings("deprecation")
    @EventHandler
      public void onInventoryClick(InventoryClickEvent event) {
        HumanEntity player = event.getWhoClicked();
        if(event.getInventory().getName().equals(classSelect.getName())) {
            if(event.getCursor().getItemMeta().getDisplayName() == white.getItemMeta().getDisplayName()){
              event.setCancelled(true);
              ((Player) player).openInventory(unranked);
              ((Player) player).updateInventory();
              ((Player) player).playSound(player.getLocation(), Sound.LEVEL_UP, 2F, 1F);
          }
        }
    }

And it turns out, my Eclipse is all screwed up for some reason, and sometimes the class file doesn't get changed to the current, saved .class that I edit. So it is way out of sync, it works now. The only problem is, is that when you click on the bonemeal, it lets you put it in your inventory, but then it gets bounced out and then drops on the ground, and then it opens the unranked inventory JRL1004

JRL1004 NathanWolf JHG0 Thanks for all of your guys' help, I eventually got it xD And it is functioning now!

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

 

mamifsidtect Are you running 1.7.2? There was a bug in 1.7.2 about that. Solution is to close the inventory then open the new one

 

xTigerRebornx Actually instead of using "event.getCursor()" I used "event.getCurrentItem()" lol

 

mamifsidtect Ah