Weirdly, if I checked the isCancelled() property on the event before applying the damage then the damage would never get applied. Unconditionally applying damage here seems to work fine.
the method was just missing the entityLiving.attackEntityFrom statement. I guess I brought this change from another branch locally by accident But the check for cancelled can't just be removed.
Resolves #165
Weirdly, if I checked the
isCancelled()
property on the event before applying the damage then the damage would never get applied. Unconditionally applying damage here seems to work fine.