Trying to add a potion effect to the player when he attacks something. It does add the potion effect, but it shows up with a duration of 0:00 and it never goes away, also the icon isn't there.  Not sure why. Any ideas?

 

 

Here's my event code.  I left out a bunch of conditionals, they are commented out while I'm debugging this now.

public class EventHook
{
  @ForgeSubscribe
  public void entityAttacked(LivingAttackEvent event)
  {
    if( !event.worldObj.isRemote ) return;
    EntityPlayer player = Minecraft.getMinecraft().thePlayer;
    
    player.addPotionEffect(new PotionEffect(Potion.heal.getID(), 1, 1));
  }
}

Its because you're applying the potion to a client version of the player

So how would I get the server version of the player?

@ForgeSubscribe
public void LivingAttackEvent(LivingAttackEvent event)
{		
	if(event.entityLiving instanceof EntityPlayerMP)
	{
		EntityPlayerMP player = (EntityPlayerMP) event.entityLiving;
		player.addPotionEffect(new PotionEffect(Potion.heal.getId(), 200, 1));
	}

}

 

It should work like this (or similar, this is for 1.6.2).

Hmm, just tried that. It gave me an error when I attacked a chicken. Couldn't cast chicken to EntityPlayerMP. Also tried

 

EntityPlayerMP player = (EntityPlayerMP) event.source.getEntity();

 

this gave me a casting error as well. EntityClientPlayerMP cannot be cast to EntityPlayerMP. Forge keeps confusing me.