lauchmelder/BloodMagic
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Mob Sacrifice Array no longer kills bosses or players (#1371)

Browse source 
This is solving the issue I raised in https://github.com/WayofTime/BloodMagic/issues/1368
Added a single check to line 123 of src/main/java/WayofTime/bloodmagic/alchemyArray/AlchemyArrayEffectMobSacrifice.java.
The check makes sure the effective health of the currently targeted entity is above 0. So bosses and players are no longer set to -1 health when the array finishes crafting.
This commit is contained in:
KohaiKhaos 2018-07-22 08:54:46 -05:00 committed by Nick Ignoffo
parent 7aadc505ab
commit 5ac52fa8c9
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/main/java/WayofTime/bloodmagic/alchemyArray/AlchemyArrayEffectMobSacrifice.java
View file

@ -120,7 +120,7 @@ public class AlchemyArrayEffectMobSacrifice extends AlchemyArrayEffect
for (EntityLivingBase living : livingEntities)
{
double health = getEffectiveHealth(living);
if (healthAvailable > 0)
if (healthAvailable > 0 && health > 0)
{
healthAvailable -= health;
living.getEntityWorld().playSound(null, living.posX, living.posY, living.posZ, SoundEvents.BLOCK_FIRE_EXTINGUISH, SoundCategory.BLOCKS, 0.5F, 2.6F + (living.getEntityWorld().rand.nextFloat() - living.getEntityWorld().rand.nextFloat()) * 0.8F);
@ -230,4 +230,4 @@ public class AlchemyArrayEffectMobSacrifice extends AlchemyArrayEffect
{
return new AlchemyArrayEffectMobSacrifice(key);
}
}
}