[Solved][1.7.10] When hit set mob on fire

[shmcrae]

I'm trying to make it so when you have the full set of armor on and when a mob or player hits me they are set on fire. Similar to thorns but with fire.

[Failender]

Subscribe to Living Hurt event, check if entity hurt is a player, check if he has full armor on aaaand set em on fire

[shmcrae]

And this is in the armor class?

[Failender]

Subscribe to Living Hurt event

[shmcrae]

Sorry, I'm not understanding.

[Failender]

http://www.minecraftforum.net/forums/mapping-and-modding/mapping-and-modding-tutorials/1571567-forge-1-6-4-1-8-eventhandler-and

[shmcrae]

I tried this

 

@SubscribeEvent

public void LivingHurtEvent(EntityLivingBase entity, DamageSource source, float ammount, EntityPlayer player)

{

if(entity instanceof EntityLiving && player.getCurrentArmor(3) != null && player.getCurrentArmor(3).getItem().equals(PItems.phoenixHelmet) && player.getCurrentArmor(2) != null && player.getCurrentArmor(2).getItem().equals(PItems.phoenixChestplate) && player.getCurrentArmor(1) != null && player.getCurrentArmor(1).getItem().equals(PItems.phoenixLeggings) && player.getCurrentArmor(0) != null && player.getCurrentArmor(0).getItem().equals(PItems.phoenixBoots))

{

entity.setFire(10);

}

}

 

[Choonster]

Read the tutorial properly, event handler methods can only have a single argument (the event that they handle).

[shmcrae]

Ok now i tried this:

 

@SubscribeEvent

public void livingEvent(LivingHurtEvent event)

{

if(event.entityLiving instanceof EntityPlayer && player.getCurrentArmor(3) != null && player.getCurrentArmor(3).getItem().equals(PItems.phoenixHelmet) && player.getCurrentArmor(2) != null && player.getCurrentArmor(2).getItem().equals(PItems.phoenixChestplate) && player.getCurrentArmor(1) != null && player.getCurrentArmor(1).getItem().equals(PItems.phoenixLeggings) && player.getCurrentArmor(0) != null && player.getCurrentArmor(0).getItem().equals(PItems.phoenixBoots))

{

event.entityLiving.setFire(10);

}

}

 

[Draco18s]

And?

[shmcrae]

And when i get hit by a zombie or any mob it doesn't set them on fire.

[Choonster]

Where are you getting

player

from? You must get all data from the event itself.

 

LivingHurtEvent#entityLiving

is the entity that was hurt.

LivingHurtEvent#source

is the

DamageSource

that caused the damage.

DamageSource#getEntity

will get the

Entity

that caused the damage, if any (this will be

null

if the damage wasn't caused by an entity).

[shmcrae]

Then how do i check if the player is wearing the full set of armor?

[Choonster]

Check if the hurt entity is a player. If it is, cast it to

EntityPlayer

and then check if it's wearing your armour.

 

If you want the effect to apply to any living entity wearing the armour rather than just players, skip the player check and use

EntityLivingBase#getEquipmentInSlot

to check the armour.

[shmcrae]

Still nothing:

 

@SubscribeEvent

public void livingEvent(LivingHurtEvent event)

{

if(event.entityLiving instanceof EntityPlayer && event.entityLiving.getEquipmentInSlot(1).getItem() == PItems.phoenixHelmet && event.entityLiving.getEquipmentInSlot(2).getItem() == PItems.phoenixChestplate && event.entityLiving.getEquipmentInSlot(3).getItem() == PItems.phoenixLeggings && event.entityLiving.getEquipmentInSlot(4).getItem() == PItems.phoenixBoots)

{

event.source.getEntity().setFire(10);

}

}

 

[YoungErtu]

Still nothing:

 

@SubscribeEvent

public void livingEvent(LivingHurtEvent event)

{

if(event.entityLiving instanceof EntityPlayer && event.entityLiving.getEquipmentInSlot(1).getItem() == PItems.phoenixHelmet && event.entityLiving.getEquipmentInSlot(2).getItem() == PItems.phoenixChestplate && event.entityLiving.getEquipmentInSlot(3).getItem() == PItems.phoenixLeggings && event.entityLiving.getEquipmentInSlot(4).getItem() == PItems.phoenixBoots)

{

event.source.getEntity().setFire(10);

}

}

 

 

try this:

 

@SubscribeEvent
   public void livingEvent(LivingHurtEvent event)
   {
      if(event.entityLiving instanceof EntityPlayer)
      {
    	 EntityPlayer player = (EntityPlayer) event.entityLiving;
    	  if(player.inventory.armorItemInSlot(3) != null && player.inventory.armorItemInSlot(3).getItem() == MODID.HELMET && player.inventory.armorItemInSlot(2) != null && player.inventory.armorItemInSlot(2).getItem() == MODID.CHESTPLATE && player.inventory.armorItemInSlot(1) != null && player.inventory.armorItemInSlot(1).getItem() == MODID.LEGGINGS && player.inventory.armorItemInSlot(0) != null && player.inventory.armorItemInSlot(0).getItem() == MODID.BOOTS) {  
         event.source.getEntity().setFire(10);
      }
   }}

 

this should work fine. dont try to check everything in one if statement.

[shmcrae]

Thank you all for the help

[shmcrae]

It is working with setting entities on fire, but when I take any other damage besides from a mob or a whole bunch of mobs attack me at once it crashes.

 

@SubscribeEvent

public void livingEvent(LivingHurtEvent event)

{

if(event.entityLiving instanceof EntityPlayer)

{

EntityPlayer player = (EntityPlayer) event.entityLiving;

if(player.inventory.armorItemInSlot(3) != null && player.inventory.armorItemInSlot(3).getItem() == PItems.phoenixHelmet && player.inventory.armorItemInSlot(2) != null && player.inventory.armorItemInSlot(2).getItem() == PItems.phoenixChestplate && player.inventory.armorItemInSlot(1) != null && player.inventory.armorItemInSlot(1).getItem() == PItems.phoenixLeggings && player.inventory.armorItemInSlot(0) != null && player.inventory.armorItemInSlot(0).getItem() == PItems.phoenixBoots)

{

event.source.getEntity().setFire(10);

}

}

}

 

 

Crash: http://pastebin.com/SYLDpJDH

[shmcrae]

Anyone?

[YoungErtu]

It is working with setting entities on fire, but when I take any other damage besides from a mob or a whole bunch of mobs attack me at once it crashes.

 

@SubscribeEvent

public void livingEvent(LivingHurtEvent event)

{

if(event.entityLiving instanceof EntityPlayer)

{

EntityPlayer player = (EntityPlayer) event.entityLiving;

if(player.inventory.armorItemInSlot(3) != null && player.inventory.armorItemInSlot(3).getItem() == PItems.phoenixHelmet && player.inventory.armorItemInSlot(2) != null && player.inventory.armorItemInSlot(2).getItem() == PItems.phoenixChestplate && player.inventory.armorItemInSlot(1) != null && player.inventory.armorItemInSlot(1).getItem() == PItems.phoenixLeggings && player.inventory.armorItemInSlot(0) != null && player.inventory.armorItemInSlot(0).getItem() == PItems.phoenixBoots)

{

event.source.getEntity().setFire(10);

}

}

}

 

 

Crash: http://pastebin.com/SYLDpJDH

 

try to check if the damage is by a mob.

because your currently code tries to set everying burning which hurts you, but some of them are no entities so they can't be set on fire, if the player gets drown damage the source will be the water, but water is no entity so check if the damage source is by an entity.

[YoungErtu]

in your method where you set the entity on fire add this:

 

if(event.source.getSourceOfDamage() != null) 
            event.source.getSourceOfDamage().setFire(10);   

[shmcrae]

Thank you