Deny an item from switching between inventories

I'm working on an virtual chest plugin.

I want to add a blacklist of items that cannot be taken or put into the virtual chest.
This is the view of the virtual chest:

http://pbrd.co/14uTWhF

I want a player to not be able to take the wooden sword and put it into his/her own inventory.

I've tried to use InventoryClickEvent, but when I cancel it, I can never put the item back again and it will be forever on my cursor, until I close the inventory and it drops on the ground. By dropping it on the ground, a player has the item now and that is just what I don't want.

The code that I use now:

Code:java

@EventHandler
public void onInventoryClick(InventoryClickEvent event) {
Inventory inv = event.getInventory();
Player player = (Player) event.getWhoClicked();
 
// Not a inventory we want to check for.
if (!inv.getTitle().equalsIgnoreCase("Global Server Inventory"))
return;
 
List<Integer> blacklist = plugin.getMainConfig().getBlackList();
 
if (event.getCursor() == null)
return;
 
ItemStack item = event.getCursor().clone();
 
if (blacklist.contains(item.getTypeId())) {
 
event.setCancelled(true);
 
player.sendMessage(ChatColor.RED
+ "You cannot transfer this item because it is blacklisted on this server.");
return;
}
}

 

getCursor is the selected item attached to the cursor, getCurrentItem however is the item that the player has clicked.

 

The problem is that when a player clicks an item, the item gets 'onto' getCursor() and getCurrentItem() returns air.

 

Staartvin

Exactly, so if you want to have the item be clickable, but not movable, you want to get if the currentItem is a certain type, and if so, cancel the event.

 

getCurrentItem() always returns air for me.

 

Staartvin

ItemStack item = event.getCurrentItem().getType();

How about that, and then check?

 

valon750

Hmm, it seems to work now. It is not returning air anymore. Thanks!
Why isn't it returning air anymore?

 

Staartvin

Hmm, not too sure, as from the looks of the code you're searching for the itemID, I'll have to take a better look at that in the morning if I get chance before work.