From 6d71858bc0dd15c5e55042570c75c9b9c033031b Mon Sep 17 00:00:00 2001
From: Kandiyaki <andrewepperson8@gmail.com>
Date: Sun, 27 Oct 2024 10:22:08 -0400
Subject: [PATCH] fixed the bug for real this time. i hope.

---
 .../EntityEffects/Effects/MakeSyndient.cs     | 19 +++++++++++++++----
 .../Forensics/Systems/ForensicsSystem.cs      |  8 +++++---
 2 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/Content.Server/EntityEffects/Effects/MakeSyndient.cs b/Content.Server/EntityEffects/Effects/MakeSyndient.cs
index 1f7745cb2314ea..3f5cca33fc10fd 100644
--- a/Content.Server/EntityEffects/Effects/MakeSyndient.cs
+++ b/Content.Server/EntityEffects/Effects/MakeSyndient.cs
@@ -66,12 +66,23 @@ public override void Effect(EntityEffectBaseArgs args)
                         }
                     }
                 }
-                //we have all the DNA in the activated subjuzine. get a random one and find the DNA's source.
-                if (dnaDataList.Count > 0)
+
+                String? chosenName = null;
+
+                for(int i=0; i<dnaDataList.Count; i++)
                 {
-                    DnaData chosenOne = dnaDataList[0];
+                    DnaData candidate = dnaDataList[i];
+                    String? candidateName = forensicSys.GetNameFromDNA(candidate.DNA);
 
-                    String chosenName = forensicSys.GetNameFromDNA(chosenOne.DNA);
+                    if (candidateName != null)
+                    {
+                        chosenName = candidateName;
+                    }
+                }
+
+                //we have all the DNA in the activated subjuzine. get a random one and find the DNA's source.
+                if (chosenName!=null)
+                {
                     //we FINALLY have the name of the injector. jesus fuck.
                     //now, we build the role name, description, etc.
 
diff --git a/Content.Server/Forensics/Systems/ForensicsSystem.cs b/Content.Server/Forensics/Systems/ForensicsSystem.cs
index 2ce83e7584d7e1..54b2579e298d53 100644
--- a/Content.Server/Forensics/Systems/ForensicsSystem.cs
+++ b/Content.Server/Forensics/Systems/ForensicsSystem.cs
@@ -135,11 +135,13 @@ public void CopyForensicsFrom(ForensicsComponent src, EntityUid target)
                 dest.Fingerprints.Add(print);
             }
         }
-        public string GetNameFromDNA(string DNA)
+
+        //gets the name of a person from the DNA inside it. RETURNS NULL FOR ANIMALS, OR ANYTHING ELSE WITH UNKNOWN DNA.
+        public string? GetNameFromDNA(string DNA)
         {
             var query = EntityQueryEnumerator<DnaComponent>();
 
-            String outputName = "OH GOD OH FUCK IT'S BROKEN";
+            String? outputName = null;
             //iterate over every DNAcomponent in the server until you find one that matches the given DNA
             while (query.MoveNext(out var sourceUID, out var sourceComp))
             {
@@ -324,7 +326,7 @@ private void OnTransferDnaEvent(EntityUid uid, DnaComponent component, ref Trans
             recipientComp.DNAs.Add(component.DNA);
             recipientComp.CanDnaBeCleaned = args.CanDnaBeCleaned;
         }
-
+        
         #region Public API
 
         /// <summary>