[1.8] onItemUseFinish called twice?

[61352151511]

In 1.7 I had an item called "Lunchbox" which you could put in a crafting table with food items to fill up. You could then "eat" the lunchbox in order to take some food stored from it. In my item class the onEaten method from Item was overridden, in 1.8 however the onFoodEaten method is in the ItemFood class so I decided to switch to onItemUseFinish (Changing to extending ItemFood would cause more problems for me)

 

This is the method I have

@Override public ItemStack onItemUseFinish(ItemStack stack, World world, EntityPlayer player) {
	int PlayerFood = player.getFoodStats().getFoodLevel();
	if (PlayerFood < 20) {
		float StoredFoodA = 0F;
		if (stack.getTagCompound() == null) {
			stack.setTagCompound(new NBTTagCompound());
		}
		try {
			StoredFoodA = stack.getTagCompound().getFloat("Food_Stored");
		} catch (NullPointerException e) {
			StoredFoodA = 0F;
		}
		int StoredFood = (int) Math.floor(StoredFoodA);
		int FoodToGive = 20 - PlayerFood;
		LogHelper.warn("*****");
		LogHelper.warn(StoredFood + "Food Stored");
		LogHelper.warn(FoodToGive + "FTG");
		if (FoodToGive > StoredFood) {
			LogHelper.warn("Exceeds stored food");
			FoodToGive = StoredFood;
			LogHelper.warn(FoodToGive + "FTG");
		}
		LogHelper.warn("*****");
		player.getFoodStats().addStats(PlayerFood + FoodToGive, FoodToGive > 0 ? 20F : 0F);
		stack.getTagCompound().setFloat("Food_Stored", StoredFood - FoodToGive);
	}
	return stack;
}

 

In the console this is spit out after "eating" once

[08:17:54] [server thread/WARN] [RandomUtilities]: *****
[08:17:54] [server thread/WARN] [RandomUtilities]: 12Food Stored
[08:17:54] [server thread/WARN] [RandomUtilities]: 20FTG
[08:17:54] [server thread/WARN] [RandomUtilities]: Exceeds stored food
[08:17:54] [server thread/WARN] [RandomUtilities]: 12FTG
[08:17:54] [server thread/WARN] [RandomUtilities]: *****
[08:17:54] [Client thread/WARN] [RandomUtilities]: *****
[08:17:54] [Client thread/WARN] [RandomUtilities]: 12Food Stored
[08:17:54] [Client thread/WARN] [RandomUtilities]: 20FTG
[08:17:54] [Client thread/WARN] [RandomUtilities]: Exceeds stored food
[08:17:54] [Client thread/WARN] [RandomUtilities]: 12FTG
[08:17:54] [Client thread/WARN] [RandomUtilities]: *****

 

Any help appreciated

[Degubi]

I think 1 print for client, and one for server.

[clienthax]

I think 1 print for client, and one for server.

yep.

[61352151511]

That makes sense. Should have looked harder, however what I can't figure out is why the hunger is being overfilled then... I'll have to check that out, thanks for pointing that out.