I am trying to make it so that when an entity wears my custom armor, it is immune to magic damage, but the code that i have is not working.

 

public void onBurnDamage(LivingAttackEvent event)
{
    if(event.entity instanceof EntityPlayer)
    {
    	EntityPlayer player = (EntityPlayer)event.entity;
    	if(event.source.equals(DamageSource.lava) || event.source.equals(DamageSource.inFire) || event.source.equals(DamageSource.onFire)){
    		ItemStack head = player.inventory.armorInventory[0];
    		ItemStack chest = player.inventory.armorInventory[1];
    		ItemStack legs = player.inventory.armorInventory[2];
    		ItemStack feet = player.inventory.armorInventory[3];
    		if(head != null && chest != null && legs != null && feet != null){
    			ItemStack helmet = new ItemStack((Item) Item.itemRegistry.getObject(MainClass.fieryHelmet));
    			ItemStack chestplate = new ItemStack((Item) Item.itemRegistry.getObject(MainClass.fieryChestplate));
    			ItemStack Leggs = new ItemStack((Item) Item.itemRegistry.getObject(MainClass.fieryLeggings));
    			ItemStack boots = new ItemStack((Item) Item.itemRegistry.getObject(MainClass.fieryBoots));
    			if(player.getEquipmentInSlot(1) == helmet && player.getEquipmentInSlot(2) == chestplate && player.getEquipmentInSlot(3) == Leggs && player.getEquipmentInSlot(4) == boots){
    				event.setCanceled(true);
    			}
    		}
        }
    }
}

 

If someone can figure out what I am doing wrong that would be great!

 

Also, I am on 1.7 because it is what most other mods I plan on combining with this are on and what I am used to.

You are comparing ItemStacks not items, ItemStacks hold much more information that is different between between each one, and items there should only ever be one instance of an item in a mod.

Almost every method i have tried has required an ItemStack not Item, so I tried to fix it by making the Items into ItemStacks, but didnt work.

The ItemStacks have a method called getItem()

// Just to get an ItemStack variable 
// An ItemStack with one apple in it
ItemStack stack = new ItemStack(Items.apple, 1);
stackItem = stack.getItem();

Ok. I used that information and came up with a bit of new code:

 

public void onBurnDamage(LivingAttackEvent event)
{
    if(event.entity instanceof EntityPlayer)
    {
    	EntityPlayer player = (EntityPlayer)event.entity;
    	if(event.source.equals(DamageSource.lava) || event.source.equals(DamageSource.inFire) || event.source.equals(DamageSource.onFire)){
    		ItemStack head = player.inventory.armorInventory[0];
    		ItemStack chest = player.inventory.armorInventory[1];
    		ItemStack legs = player.inventory.armorInventory[2];
    		ItemStack feet = player.inventory.armorInventory[3];
    		if(head != null && chest != null && legs != null && feet != null){
    			ItemStack helmet = player.getEquipmentInSlot(1);
    			ItemStack chestplate = player.getEquipmentInSlot(2);
    			ItemStack Leggs = player.getEquipmentInSlot(3);
    			ItemStack boots = player.getEquipmentInSlot(4);
    			if(helmet.getItem() == MainClass.fieryHelmet && chestplate.getItem() == MainClass.fieryChestplate && Leggs.getItem() == MainClass.fieryLeggings && boots.getItem() == MainClass.fieryBoots){
    				event.setCanceled(true);
    			}
    		}
        }
    }
}

 

Still take damage from fire.

Try reversing the order so head = 4 not 1

Didn't work. Any other possibilities, or other methods to use instead of onBurnDamage?

Well that is an Event did you subscribe it using

MinecraftForge.EVENT_BUS.register(new EventHandler());

 

And your method needs to have @SubcribeEvent above it

No, i was unaware I had to do that; where do I put the

MinecraftForge.EVENT_BUS.register(new EventHandler());

?

In your mods init

Got it working. Thanks Very Much!

Hey I have a question for you AimeryCM, does it still render the fire?

Probably will still render the mob being on fire, it will just give no shits.

(ZombiePigs and Blazes still render being on fire--say, by being in lava--they just don't take damage)