`/datum/action/cooldown/spell/touch/lay_on_hands` has a bad assumption

tgstation / tgstation

The /tg/station branch of SS13.

https://www.tgstation13.org/

GNU Affero General Public License v3.0

1.65k stars 4.83k forks source link

`/datum/action/cooldown/spell/touch/lay_on_hands` has a bad assumption #84521

Open MrMelbert opened 3 weeks ago

MrMelbert commented 3 weeks ago

Issue Summary

https://github.com/tgstation/tgstation/blob/5674c1b96f26413c3636913ddd6150c7a56cefe0/code/datums/mutations/touch.dm#L198-L205

mendicant_organic_limbs is never null, it should be !length

MrMelbert commented 3 weeks ago

also hurtguy.has_status_effect(/datum/status_effect/fire_handler/fire_stacks this isn't technically wrong but it should just be hurtguy.fire_stacks > 0 iirc

carlarctg commented 3 weeks ago

slapped both of these into my pr