Minecraft Forums

I have tried the following but it doesn't work :

	@Override
	protected void onFoodEaten(ItemStack stack, World worldIn,
			EntityPlayer entityPlayer) {
		// TODO Auto-generated method stub
		super.onFoodEaten(stack, worldIn, entityPlayer);
		
		
		entityPlayer.capabilities.allowFlying = true;
		//entityPlayer.capabilities.isFlying = true;
		entityPlayer.sendPlayerAbilities();
		
		
	}

I've checked using a breakpoint and for some reason OnFoodEaten isn't getting called.

Does your item class extend ItemFood? You need to if you haven't.

If you did extend ItemFood:

• Did you override the onItemUseFinish() method? (You probably shouldn't override it, but that is the method that is supposed to call onFoodEaten()).
• The only other thing I can think of is maybe you imported the wrong World class or something so that the method prototype doesn't match the parent class.

It extends ItemFood.

try posting your item's class

Here is the class

package org.dagwoodland.purple_lab.items;

import org.dagwoodland.purple_lab.init.PurpleLabItems;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemFood;
import net.minecraft.item.ItemStack;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

public class ItemChocolateBubbly extends ItemFood {

	public ItemChocolateBubbly(int amount, float saturation,
			boolean isWolfFood, int effectAmplifier) {
		
		super(amount, saturation, isWolfFood);
		
		// TODO Auto-generated constructor stub
        this.setMaxDamage(0);
        this.setAlwaysEdible();
        this.setPotionEffect(1, 10, effectAmplifier, 1.0f);
        //this.setPotionEffect(8, 10, effectAmplifier, 1.0f);
        this.maxStackSize = 16;
	}

	
	
	@Override
	protected void onFoodEaten(ItemStack stack, World worldIn,
			EntityPlayer entityPlayer) {
		// TODO Auto-generated method stub
		
		super.onFoodEaten(stack, worldIn, entityPlayer);
		
		
		entityPlayer.capabilities.allowFlying = true;
		//entityPlayer.capabilities.isFlying = true;
		entityPlayer.sendPlayerAbilities();
		
		
	}
}

I tried it and for me it works.

There is only a little problem with your code, you have to check that it will be executed only server side with a check over the flight code:

if(!worldIn.isRemote) {
	entityPlayer.capabilities.allowFlying = true;
	entityPlayer.sendPlayerAbilities();
}

Thanks, but onFoodEaten still isn't getting called.

Quote from dagwoodland»

Thanks, but onFoodEaten still isn't getting called.

I'm not sure what that means, sorry I'm fairly new to java.

I have registered the item using that class.

I would highly recommend learning Java properly before trying to make a mod. You need to at least understand the basics of OO programming like classes and instances.

Upload your item registration code to Gist or Pastebin and link it here.

I can not believe how silly I was, I've been trying to get it to work for the last 3 days... and in the end I forgot to change the item register to the new class. Thanks everyone for your help!