Fix more by-ref issues in DNA

Conflicts:
	code/game/machinery/hydroponics.dm

code/datums/diseases/dna_spread.dm

@@ -39,12 +39,14 @@
 
 				//Save original dna for when the disease is cured.
 				src.original_dna["name"] = affected_mob.real_name
-				src.original_dna["UI"] = affected_mob.dna.UI
-				src.original_dna["SE"] = affected_mob.dna.SE
+				src.original_dna["UI"] = affected_mob.dna.UI.Copy()
+				src.original_dna["SE"] = affected_mob.dna.SE.Copy()
 
 				affected_mob << "\red You don't feel like yourself.."
-				affected_mob.UpdateAppearance(strain_data["UI"])
-				affected_mob.dna.SE = strain_data["SE"]
+				var/list/newUI=strain_data["UI"]
+				var/list/newSE=strain_data["SE"]
+				affected_mob.UpdateAppearance(newUI.Copy())
+				affected_mob.dna.SE = newSE.Copy()
 				affected_mob.dna.UpdateSE()
 				affected_mob.real_name = strain_data["name"]
 				domutcheck(affected_mob)
@@ -56,8 +58,10 @@
 
 /datum/disease/dnaspread/Del()
 	if ((original_dna["name"]) && (original_dna["UI"]) && (original_dna["SE"]))
-		affected_mob.UpdateAppearance(original_dna["UI"])
-		affected_mob.dna.SE = original_dna["SE"]
+		var/list/newUI=original_dna["UI"]
+		var/list/newSE=original_dna["SE"]
+		affected_mob.UpdateAppearance(newUI.Copy())
+		affected_mob.dna.SE = newSE.Copy()
 		affected_mob.dna.UpdateSE()
 		affected_mob.real_name = original_dna["name"]
 

code/game/dna/dna_modifier.dm

@@ -797,7 +797,7 @@
 				if ((buf.types & DNA2_BUF_UE))
 					src.connected.occupant.real_name = buf.dna.real_name
 					src.connected.occupant.name = buf.dna.real_name
-				src.connected.occupant.UpdateAppearance(buf.dna.UI)
+				src.connected.occupant.UpdateAppearance(buf.dna.UI.Copy())
 			else if (buf.types & DNA2_BUF_SE)
 				src.connected.occupant.dna.SE = buf.dna.SE
 				src.connected.occupant.dna.UpdateSE()

code/game/dna/genes/disabilities.dm

@@ -36,6 +36,8 @@
 		M.sdisabilities|=sdisability
 	if(activation_message)
 		M << "\red [activation_message]"
+	else
+		testing("[name] has no activation message.")
 
 /datum/dna/gene/disability/deactivate(var/mob/M, var/connected, var/flags)
 	if(mutation && (mutation in M.mutations))
@@ -46,6 +48,8 @@
 		M.sdisabilities &= ~sdisability
 	if(deactivation_message)
 		M << "\red [deactivation_message]"
+	else
+		testing("[name] has no deactivation message.")
 
 /datum/dna/gene/disability/hallucinate
 	name="Hallucinate"

code/game/machinery/hydroponics.dm

@@ -937,7 +937,6 @@ obj/machinery/hydroponics/attackby(var/obj/item/O as obj, var/mob/user as mob)
 
 	parent.update_tray()
 
-
 /obj/machinery/hydroponics/proc/update_tray(mob/user = usr)
 	harvest = 0
 	lastproduce = age

code/game/objects/items/weapons/dna_injector.dm

@@ -66,7 +66,7 @@
 	if (!(NOCLONE in M.mutations)) // prevents drained people from having their DNA changed
 		if (buf.types & DNA2_BUF_UI)
 			if (!block) //isolated block?
-				M.UpdateAppearance(buf.dna.UI)
+				M.UpdateAppearance(buf.dna.UI.Copy())
 				if (buf.types & DNA2_BUF_UE) //unique enzymes? yes
 					M.real_name = buf.dna.real_name
 					M.name = buf.dna.real_name
@@ -77,7 +77,7 @@
 				uses--
 		if (buf.types & DNA2_BUF_SE)
 			if (!block) //isolated block?
-				M.dna.SE = buf.dna.SE
+				M.dna.SE = buf.dna.SE.Copy()
 				M.dna.UpdateSE()
 			else
 				M.dna.SetSEValue(block,src.GetValue())