Replaces lastassailant weakref deletion with setting to null

Fixes bugs caused by the lastassailant weakref getting deleted, which would break any other objects that were using that weakref.

Author: noirogen

code/__HELPERS/logging.dm

@@ -95,7 +95,7 @@
 	if (ismob(target))
 		target.attack_log += text("\[[time_stamp()]\] <font color='orange'>Has been [what_done] by [user_txt][object_txt].[intent_txt][addition_txt]</font>")
 		if (!iscarbon(user))
-			QDEL_NULL(target.lastassailant)
+			target.lastassailant = null
 
 	if (ismob(user) && ismob(target))
 		target.assaulted_by(user)

code/modules/runescape/runescape_pvp.dm

@@ -59,16 +59,7 @@ var/list/non_standard_maint_areas = list(
 
 /mob/proc/assaulted_by(var/mob/M,var/weak_assault=FALSE)
 	if(!iscarbon(M))
-		var/ref_in_use = FALSE
-		if(istype(src, /mob/living/simple_animal/hostile))
-			var/mob/living/simple_animal/hostile/H = src
-			for(var/datum/weakref/friend in H.friends)
-				if(friend.get() == lastassailant.get())
-					lastassailant = null
-					ref_in_use = TRUE
-					break;
-		if(!ref_in_use)
-			QDEL_NULL(lastassailant)
+		lastassailant = null
 	else
 		lastassailant = makeweakref(M)