Solved InventoryCloseEvent ItemStack returning issue

So I was recently developing a shop plugin for a server that requested it and I ran into an issue. I'm not sure if it's just me or what because I haven't been able to test it with others.

Basically, when you're listening for the InventoryCloseEvent, if you loop through the contents of that inventory and try to add it to the player's inventory it gets added, but it's invisible. player.updateInventory(); doesn't fix the issue either. They stay invisible until they are updated through action such as right clicking while holding one of the spots where the item should be.

Can someone else test this and tell me if it's an issue with Bukkit? And if it IS an issue with Bukkit is there even a fix? Here's a snippet of my code.

Code:java

            for(int i = 0; i < 35; i++){
                ItemStack invItem = event.getInventory().getItem(i);
                if(invItem == null) continue;
                if(!gch.getSellables(shopName).contains(Integer.toString(invItem.getTypeId()))){
                    returnItem.add(invItem);
                    event.getInventory().removeItem(invItem);
                    continue;
                }
                cashPayed = (gch.getPayment(shopName, Integer.toString(invItem.getTypeId())) * invItem.getAmount()) + cashPayed;
                itemsSold = itemsSold + invItem.getAmount();
            }
            for(ItemStack item : returnItem){
                player.getInventory().addItem(item);
            }
            player.updateInventory();

 

Just a hunch so this might not work, but try scheduling a task for 1 tick later to add the items to the player's inventory.

 

St3venAU

Well, I fixed it using this method before anything was commented, but a 1 tick delay doesn't fix the issue. I found it has to be at least 3 ticks for it to visibly add the items to the player's inventory. Perhaps it's because it's being indirectly done from outside the event after it finishes? It's weird though, that's for sure.