Requiem icon indicating copy to clipboard operation
Requiem copied to clipboard

Mortal dysmorphia may not kill if the players involved have >100 max HP

Open James103 opened this issue 1 year ago • 4 comments

Mortal dysmorphia always deals 100 HP of damage upon turning mortal if the soul does not belong in the body it's inhabiting; this amount is regardless of the player's max health. https://github.com/Ladysnake/Requiem/blob/5fd09888475e9be2ed8735f3b172b5d0a24df37b/src/main/java/ladysnake/requiem/common/VanillaRequiemPlugin.java#L249

The 100F constant should be changed to player.getMaxHealth() * 5 to ensure the damage source always hits for an amount greater than the player's max health. The damage source itself may need to be changed to act like void damage instead of just magic, further ensuring no amount of protection can reduce the damage amount below the player's max health.

James103 avatar Mar 05 '23 01:03 James103

I was testing this issue, another thing to keep in mind is that the damage would have to bypass totems of undying since those do also allow the issue to persist.

PsychoBreadstick avatar Mar 05 '23 03:03 PsychoBreadstick

I was testing this issue, another thing to keep in mind is that the damage would have to bypass totems of undying since those do also allow the issue to persist.

Void damage already bypasses totems

kristibektashi avatar Mar 05 '23 09:03 kristibektashi

I was unaware of that, thank you.

PsychoBreadstick avatar Mar 05 '23 14:03 PsychoBreadstick

I was unaware of that, thank you.

As of 1.20.1 there is a new generic_kill damage type that afaik bypasses all damage reductions

kristibektashi avatar Aug 13 '23 19:08 kristibektashi