[NO GBP] Fixes oil ignition, sanity checks blood DNA and adds some logging to it (#92221)

## About The Pull Request

Oil blood will no longer try to ignite twice when hit with a welder,
runtiming on the second try.
Also added crashes when ``add_blood_DNA`` is passed non-list or lists
with null assoc values, as both are invalid and cause crashes, because I
need to hunt down a stupid bug.

## Changelog
🆑
fix: Trying to ignite oil with a welding tool no longer runtimes due to
it trying to get ignited twice
/🆑
This commit is contained in:
SmArtKar
2025-07-26 04:22:41 +02:00
committed by GitHub
parent 2867d84a0a
commit de70de4d7c
2 changed files with 5 additions and 2 deletions
@@ -114,6 +114,11 @@
. = ..()
if (isnull(blood_DNA_to_add))
return .
if (!islist(blood_DNA_to_add))
CRASH("add_blood_DNA on [src] ([type]) has been passed a non-list blood_DNA_to_add ([blood_DNA_to_add])!")
for (var/blood_key in blood_DNA_to_add)
if (isnull(blood_DNA_to_add[blood_key]))
CRASH("add_blood_DNA on [src] ([type]) has been passed bad blood_DNA_to_add ([blood_key] - [blood_DNA_to_add[blood_key]] key-value pair)!")
cached_blood_color = null
cached_blood_emissive = null
if (forensics)