1.12.2 How to make a life steal enchantment by percentage [SOLVED]

[Enderlook]

Actually, I have done one:

	@Override
	public void onEntityDamaged (EntityLivingBase user, Entity target, int level) {
		user.heal(1 + level);
	}

...

But I have a problem, this heals the player a flat amount. My idea is to heal the player a percentage of the damage dealt to the target. How can I get the amount of damaged provoked to the enemy in order to calculate my life steal?

Edited August 9, 2018 by Enderlook

[Animefan8888]

4 hours ago, Enderlook said:

But I have a problem, this heals the player a flat amount. My idea is to heal the player a percentage of the damage dealt to the target. How can I get the amount of damaged provoked to the enemy in order to calculate my life steal?

You're not able to apply your Enchantment Effect in that method. You will need to subscribe to the LivingDamageEvent or LivingAttackEvent.

[Enderlook]

55 minutes ago, Animefan8888 said:

You're not able to apply your Enchantment Effect in that method. You will need to subscribe to the LivingDamageEvent or LivingAttackEvent.

Ok, I've tried with this code (and the same with LivingDamageEvent) but it seems don't work:

	@SubscribeEvent
	public void vampire(LivingAttackEvent event) {
		Object attacker = event.getSource().getTrueSource();
		if (attacker instanceof EntityLivingBase) {
			EntityLivingBase entityAttacker = (EntityLivingBase)attacker;
			if(!entityAttacker.getEntityWorld().isRemote) {
				int level = EnchantmentHelper.getEnchantmentLevel(ENCHANTEMNT_VAMPIRISM, entityAttacker.getHeldItemMainhand());
				entityAttacker.heal(event.getAmount() * (level / 5));
			}	
		}	
	}

What am I doing wrong?

[Animefan8888]

12 minutes ago, Enderlook said:

What am I doing wrong?

Is your event registered?

[Enderlook]

14 minutes ago, Animefan8888 said:

Is your event registered?

... isn't "@SubscribeEvent" the way to register the event? I've another event registered in that way (for another enchantment) and they work. All of them are registered in the same class under the "@EventBusSubscriber(modid=ExtraCraft.MOD_ID)"

[Terrails]

If you use @EventBusSubscriber you have to make the event method static

[Enderlook]

16 minutes ago, Terrails said:

If you use @EventBusSubscriber you have to make the event method static

Ups, I've forgotten that. And which method is better? Register them like I do, or in the other way?

[Animefan8888]

7 minutes ago, Enderlook said:

Ups, I've forgotten that. And which method is better? Register them like I do, or in the other way?

It ultimately doesn't matter.