From 66c926b54088ef2b98cdd67fc8ba2a398f7ab4f0 Mon Sep 17 00:00:00 2001 From: MichiRecRoom <1008889+LikeLakers2@users.noreply.github.com> Date: Sat, 8 Mar 2025 14:43:19 -0500 Subject: [PATCH] The radioactive mutation now ensures it has an owner before modifying the radioactivity (#89805) ## About The Pull Request https://github.com/Monkestation/Monkestation2.0/pull/5690 exposed the fact that `/datum/mutation/human/radioactive/modify()` assumes we have an owner. However, the parent proc, `/datum/mutation/human/modify()`, does not make this assumption - and as such, `/datum/mutation/human/radioactive/modify()` shouldn't either. This PR modifies `/datum/mutation/human/radioactive/modify()` to check if our `owner` variable is set (and valid!) before applying the radioactivity changes. --- code/datums/mutations/radioactive.dm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/code/datums/mutations/radioactive.dm b/code/datums/mutations/radioactive.dm index b2a17b53412..e7dc2557eb5 100644 --- a/code/datums/mutations/radioactive.dm +++ b/code/datums/mutations/radioactive.dm @@ -24,7 +24,8 @@ /datum/mutation/human/radioactive/modify() . = ..() - make_radioactive(owner) + if(!QDELETED(owner)) + make_radioactive(owner) /** * Makes the passed mob radioactive, or if they're already radioactive,