Problem with InventoryClickEvent

Hi, i have a problem with the bukkit InventoryClickEvent. My plugin add new type of items (all is managed with lore). When the player open the menu, he can click on specified item like the belt to equip it in the storage, and reverse, click on the equipped item to give it back in inventory.

If i do like it :
- Open menu
- Click on MyBelt (like image)
==> Item was equiped
- I press ESC (to close Inventory)
- Open menu again
- Click on MyBelt (like second image)
- Item come back in my inventory
===> It's work

But if i do like it (and i want to do like it) :
- Open menu
- Click on MyBelt
==> Item was equipped
- Click on item to get it
==> NO WORK

I've add some debug messages, to test InventoryClickEvent, and in second exemple (red line), the event was no call, i don't know why.
I give you some parts of my code to understand, cause since few hours i don't know how to fix it. Sorry for my english, thank for your time, and i hope you can help me

My code :

Code:java

    @EventHandler
    public void onClick(InventoryClickEvent e)
    {
        Player p = (Player)e.getWhoClicked();
        ItemStack current = e.getCurrentItem();
        Inventory inv = e.getInventory();
        String invn = p.getOpenInventory().getTitle();
        int raw = e.getRawSlot();
 
        if(inv.getTitle().equalsIgnoreCase("Menu du joueur") || invn.equalsIgnoreCase("Menu du joueur"))
        {
 
            Equipement equip = EquipementStorage.getPlayerEquipement(p);
 
            if(inv.getTitle().equalsIgnoreCase("Menu du joueur") && (raw >= 0 && raw <= 35))
            {
                e.setCancelled(true);
                updateInventoryLater(p);
 
                if(current == null)
                    return;
 
                if(current.getItemMeta() == null)
                    return;
 
                if(current.getItemMeta().getDisplayName() == null)
                    return;
 
                int slot = e.getSlot();
 
                if(slot == 8 || slot ==15 || slot == 24 || slot == 26 || slot == 35)
                {
 
                    if(current.isSimilar(ies1) || current.isSimilar(ies2) || current.isSimilar(ies3) || current.isSimilar(ies4) || current.isSimilar(ies5))
                    {
                        //do nothing
                    }
                    else
                    {
                        boolean full = true;
                        for(ItemStack item : e.getView().getBottomInventory().getContents())
                        {
                            if(item == null)
                            {
                                full = false;
                                break;
                            }
                            else if(item != null && item.getType() == Material.AIR)
                            {
                                full = false;
                                break;
                            }
                        }
 
                        if(full)
                        {
                            //Inventaire plein
                            p.sendMessage(ChatColor.RED+"Is full!");
                            return;
                        }
 
                        EquipementStorage.setPlayerEquipement(p, equip);
                        /** Checkup**/
                        EffectItems.checkEquipedPlayerStats(p);
                        p.closeInventory();
                        openPlayerMenu(p);
                        updateInventoryLater(p);
                    }
                    return;
                }
            }
        }
 
            //If click in my inventory (bottom)
            else if(raw >= 36 && raw <= 71)
            {
                if(e.getClick() == ClickType.RIGHT && LevelRestrict.hasLevelToUse(current, p))
                {
                    if(current == null)
                        return;
 
                    ItemMeta im = current.getItemMeta();
                    if(!im.hasLore())
                        return;
                    List<String> lore = im.getLore();
 
                    if(lore != null && lore.size() >= 2)
                    {
                        ItemStack retitem = null;
                        int copie = 0;
 
                        if(lore.get(0).contains("Belt") || lore.get(1).contains("Belt"))
                        {
                            //get bottom inventory
                            Inventory binv = e.getView().getBottomInventory();
 
                            //Stack all copy of item in one stack and remove one amount to equip it
                            for(int i = 0; i < binv.getSize(); i++)
                            {
                                if(binv.getItem(i) != null && binv.getItem(i).isSimilar(current))
                                {                                   
                                    copie = copie + binv.getItem(i).getAmount();
                                    binv.setItem(i, null);
                                }
                            }
 
                            if(copie > 1)
                            {
                                copie = copie - 1;
                                ItemStack ic = current.clone();
                                ic.setAmount(copie);
                                binv.addItem(ic);
                            }
                            //end of remove one amount
 
                            //Place item in storage
                            ItemStack newit = current.clone();
                            newit.setAmount(1);
                            retitem = equip.getBague();//
                            equip.setItem(5, newit);//
                            EquipementStorage.setPlayerEquipement(p, equip);
                            binv.addItem(retitem);
                            //add stack of item left in bottom inventory
                        }
 
                        updateInventoryLater(p);
                        p.closeInventory();
                        openPlayerMenu(p);
                        return;
                    }
            }
        }
  }

 

Try closing and opening the inventory l, and / or use getRawSlot instead of getSlot

 

Thank for your reply, but i already close and open my inventory :

Code:java

p.closeInventory();
openPlayerMenu(p);

The problem is not the slot. When i click in inventory, anywhere, InventoryClickEvent is call, but after the close and open (openPlayerMenu), the opened inventory no longer call InventoryClickEvent, why, i don't know ...

 

Pingoo
So you mean, when you close and re-open the inventory, the plugin will not function the click events?

 

Yes, is that. When i click on the new inventory, the InventoryClickEvent was no call. I'll try some ideas tonight

 

Pingoo
Instead of using

Code:java

 if(inv.getTitle().equalsIgnoreCase("Menu du joueur") || invn.equalsIgnoreCase("Menu du joueur"))
{

try:

Code:java

if (event.getInventory().getName()
                    .equals(InvClass.getCompassInventory().getName())) {

Replace "InvClass" with your class name that you are calling the event for. E.g for the code I provided InvClass would be my class with the inventory.