Inventory not updating on p.setItemInHand()

I am attempting to make it so that when you right-click with a certain item in your hand, it adds a certain amount of health to your player's health. Everything is working properly, except that it will not remove the item from their inventory. Code follows:

Code:java

if(!doHunger){
    int amountToHeal = f.getHealthBoost();
    int health = p.getHealth() + amountToHeal;
    if(health > 20) health = 20;
    p.setHealth(health);
    System.out.println(p.getItemInHand());
    p.setItemInHand(new ItemStack(Material.AIR));
    p.updateInventory();
    System.out.println(p.getItemInHand());
}

output to console:

Code:

 
23:23:29 [INFO] ItemStack{GRILLED_PORK x 1, UNSPECIFIC_META:{meta-type=UNSPECIFIC, display-name=Instant-Eat Cooked Pork, lore=[Instantly heals 3.0 Hearts,  0.9729892635818012], enchants={DIG_SPEED=1}}}
23:23:29 [INFO] ItemStack{AIR x 0}
 

The output i what I expected. It sets the item to air.
However, the output does not match what happens in-game. The player's inventory does not update and the GRILLED_PORK remains in the inventory.

Any ideas on why this would occur?

[comment redacted]

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

 

what about setting the item using the inventory, like setting the slot yourself

 

try using Player.updateInventory() to see if it works after that (methode is depredated, but still useable)
If it works, then bukkit/minecraft didn't notice inventory was updated
if it wont work, then you made a mistake

 

He already does that in the OP

 

try logging it instead
getLogger().info()
instead of
System.out.println
I need to see what that says after

 

you dont take items like that T.T
p.getItemInHand().setAmount(p.getItemInHand().getAmount() - 1);
Problem solved or no?

 

Setting it to air works for me because I made it so it wouldn't stack
I'll try the line you gave really fast, though.

That does not work.
I still get the same result using this following, new code.

Code:java

 
p.setHealth(health);
getLogger().info(p.getItemInHand().toString());
item.setAmount(item.getAmount() - 1);
p.setItemInHand(item);
p.updateInventory();
getLogger().info(p.getItemInHand().toString());
 

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

 

if the amount is 1 and you have to remove 1 of it, you should set it to AIR

 

The output to console says that it has been set to air, but they player's inventory doesn't get updated, even though I use p.updateInventory()

 

have you tried to just remove the pork from the players hand

example:

Code:

ItemStack rItem = new ItemStack(320, 1);
 
player.getInventory().removeItem(rItem);
player.updateInventory();

 

so does it change just not update?
or does it not change?
is this in an event? if so what is the event priority?

 

It is in an event, and everything in the event works *except* removing the item from their inventory.
It is supposed to run onPlayerInteract, check if right-click, check if a food item, and if so, heal the player accordingly and then consume the item. The last step is the problem

 

are you on 1.4.6, i am noticing a seperate issue trying to update the count in hand while processing a block place.. it doesn't work

 

Yes I think this is a bug. Trying to set stuff inside of RIGHT_CLICK_BLOCK was giving me problems but immediately worked when i changed it to LEFT_CLICK_BLOCK