Skip to content

Blocks the clone pod from creating MOB_ROBOTIC mobs #6031

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

Ziiro
Copy link
Contributor

@Ziiro Ziiro commented Mar 24, 2025

About The Pull Request

Seems a bit silly that the clone pod can clone MOB_ROBOTIC, this fixes that

Why It's Good For The Game

Fixes a bit of a bug/oversight - while you technically can't scan an IPC body, you can scan the brain. And if you do a clone from a brain scan it can spit out an IPC body. This fixes that.

Changelog

🆑
fix: Clone pods can no longer create MOB_ROBOTIC bodies
/:cl:

Copy link
Member

@Absolucy Absolucy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't you just prevent scanning robots in the first place?

@Ziiro
Copy link
Contributor Author

Ziiro commented Mar 27, 2025

shouldn't you just prevent scanning robots in the first place?

We do:

if(!istype(dna) || HAS_TRAIT(mob_occupant, TRAIT_NO_DNA_COPY))
scantemp = "<font class='bad'>Unable to locate valid genetic data.</font>"
playsound(src, 'sound/machines/terminal_prompt_deny.ogg', 50, 0)
return

But we don't check the brain:

if(isbrain(mob_occupant))
dna = B.stored_dna

Because the brain doesn't have a mob attached we can't check that. At beast I might be able to see if we scan for ORGAN_ROBOTIC on the brain. But I would still want to make sure we're not printing MOB_ROBOTICS from the pod to make double sure.

@@ -133,10 +133,14 @@

//Start growing a human clone in the pod!
/obj/machinery/clonepod/proc/growclone(clonename, ui, mutation_index, mindref, blood_type, datum/species/mrace, list/features, factions, list/quirks, datum/bank_account/insurance, list/traumas, empty)
var/mob/living/carbon/human/H = new /mob/living/carbon/human(src)
H.hardset_dna(ui, mutation_index, null, clonename, blood_type, mrace, features)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoops.

tired-wired pushed a commit that referenced this pull request Mar 31, 2025
Reverts #6031

this was implemented in a way that could cause IMMENSE lag.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance perfectly balanced as all things should be
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants