Hi, is there a way to make a player immune to explosive damage when they have a certain item in hand?

I did this but it wont work, what did I do wrong?

 

 
public class EntityHurtHandler {

@SubscribeEvent
public void Event(LivingAttackEvent event) {

	EntityPlayer player = Minecraft.getMinecraft().thePlayer;

	if(event.entity instanceof EntityPlayer && event.source.isExplosion() && player.inventory.getStackInSlot(player.inventory.currentItem).getItem() == SoulCraft.creeperBow){
		event.setCanceled(true);	
	}

}

}

Minecraft.getMinecraft().thePlayer is the client side player; use the entity from the event and cast to EntityPlayer:

if (event.entity instanceof EntityPlayer) {
EntityPlayer player = (EntityPlayer) event.entity;
}

This is the new code, but it still doesn't work , the messages never show up. And to make sure my subscribe event works, I did a dummy PlayerTickEvent and it worked. Does anyone know why I cant get this to work?

 

 
package somarani.soulcraft.event;

import somarani.soulcraft.common.SoulCraft;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.util.DamageSource;
import net.minecraftforge.event.entity.living.LivingAttackEvent;
import net.minecraftforge.event.entity.living.LivingHurtEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.common.gameevent.TickEvent.PlayerTickEvent;

public class EntityHurtHandler {

@SubscribeEvent
public void Event(LivingAttackEvent event) {

	System.out.println("test 1");

	if (event.entity instanceof EntityPlayer) {
		EntityPlayer player = (EntityPlayer) event.entity;

		System.out.println("test 2");

	if(event.source.isExplosion() && player.inventory.getStackInSlot(player.inventory.currentItem).getItem() == SoulCraft.creeperBow){
		event.setCanceled(true);	
	}

}
}

}

TickEvent and LivingAttackEvent are not on the same event bus... register to MinecraftForge.EVENT_BUS.

use player.getCurrentHeldItem().getItem() instanceof SoulCraft.creeperBow

 

instead of

 

player.inventory.getStackInSlot(player.inventory.currentItem).getItem() == SoulCraft.creeperBow

 

 

Also ALWAYS check if player.getCurrentHeldItem() is not null! This would be the case if the player holds nothing.

 

EDIT: I should read the stuff I quote ._.