Luggage still takes damage from Chromaticraft Pylons
Shahelion opened this issue ยท 7 comments
As long as it lives, I consider it minor bug (it cannot be seen without health bar mods).
Probably caused by some Chromaticraft hackery (directly manipulating health?), since luggage ignores all damage.
What is its entity name or class? I can make pylons ignore them. Nothing can prevent pylon damage once it decides to attack.
@ReikaKalseki ? How do pylons do damage?
If it's https://github.com/ReikaKalseki/ChromatiCraft/blob/master/TileEntity/Auxiliary/TileEntityPylonTurboCharger.java#L359 then it's not problem (still done with attackEntityFrom
). Same with poison effect. So maybe it's something else.
Anyway, if there is something that may ignore Entity.isEntityInvulnerable
please let me know.
I'm not really fan of special casing and creating hidden dependencies like that. Can you restrict damage to entities that return Entity.isEntityInvulnerable() == false
? There are really only few entities that set it (in Forge it's only FakePlayer and I don't see why that would require damage).
Anyway, if damage is done with https://github.com/ReikaKalseki/ChromatiCraft/blob/c9fbdb44248ade76cd683a31aeff3b0bbcd3fdf3/Auxiliary/ChromaAux.java#L235 then I really don't see that as big problem - it won't kill luggage and health can be restored by picking and placing luggage back.
Can you restrict damage to entities that return Entity.isEntityInvulnerable() == false
No. I do not trust things like Draconic Armor.
Welp, here comes nuclear option: 940569f