From 85b0bf124202285907a21c25f33fd52eb2edcbf7 Mon Sep 17 00:00:00 2001
From: Iajret Creature
 <122297233+AnArgonianLizardThatStealsPRs@users.noreply.github.com>
Date: Wed, 15 Nov 2023 12:28:10 +0300
Subject: [PATCH] [MIRROR] Further Prevention of Disposals Qdeletion [MDB
 IGNORE] (#634)

* Further Prevention of Disposals Qdeletion (#79714)

---------

Co-authored-by: SkyratBot <59378654+SkyratBot@users.noreply.github.com>
Co-authored-by: san7890 <the@san7890.com>
---
 code/modules/recycling/disposal/eject.dm  | 38 ++++++++++++++++++-----
 code/modules/recycling/disposal/outlet.dm |  4 +--
 2 files changed, 32 insertions(+), 10 deletions(-)

diff --git a/code/modules/recycling/disposal/eject.dm b/code/modules/recycling/disposal/eject.dm
index febd4f5d604..523f034703c 100644
--- a/code/modules/recycling/disposal/eject.dm
+++ b/code/modules/recycling/disposal/eject.dm
@@ -2,11 +2,33 @@
  * General proc used to expel a holder's contents through src (for bins holder is also the src).
  */
 /obj/proc/pipe_eject(obj/holder, direction, throw_em = TRUE, turf/target, throw_range = 5, throw_speed = 1)
-	var/turf/src_T = get_turf(src)
-	for(var/A in holder)
-		var/atom/movable/AM = A
-		AM.forceMove(src_T)
-		SEND_SIGNAL(AM, COMSIG_MOVABLE_PIPE_EJECTING, direction)
-		if(throw_em && !QDELETED(AM))
-			var/turf/T = target || get_offset_target_turf(loc, rand(5)-rand(5), rand(5)-rand(5))
-			AM.throw_at(T, throw_range, throw_speed)
+	var/turf/origin_turf = get_turf(src)
+	var/turf/target_turf
+	if(isnull(target)) // done up here as a safety
+		target_turf = get_offset_target_turf(loc, rand(5) - rand(5), rand(5) - rand(5))
+	else
+		target_turf = target
+
+	if(QDELETED(origin_turf))
+		stack_trace("pipe_eject() attempted to operate on a qdeleted turf! In order to avoid sending things to nullspace, we are going to send everything directly to the target turf instead.")
+		origin_turf = target_turf
+
+	var/list/contents_to_eject = holder.contents
+	var/list/contents_to_throw = list()
+
+	for(var/atom/movable/thing in contents_to_eject)
+		thing.forceMove(origin_turf)
+		SEND_SIGNAL(thing, COMSIG_MOVABLE_PIPE_EJECTING, direction)
+		if(QDELETED(thing))
+			continue
+
+		contents_to_throw += thing
+
+	if(!throw_em)
+		return
+
+	for(var/atom/movable/throwable as anything in contents_to_throw)
+		if(isnull(target)) // we want the thrown things to be spread out a bit if we weren't given a target
+			target_turf = get_offset_target_turf(loc, rand(5) - rand(5), rand(5) - rand(5))
+
+		throwable.throw_at(target_turf, throw_range, throw_speed)
diff --git a/code/modules/recycling/disposal/outlet.dm b/code/modules/recycling/disposal/outlet.dm
index 4327567fb17..513a263160d 100644
--- a/code/modules/recycling/disposal/outlet.dm
+++ b/code/modules/recycling/disposal/outlet.dm
@@ -60,9 +60,9 @@
 	if((start_eject + 30) < world.time)
 		start_eject = world.time
 		playsound(src, 'sound/machines/warning-buzzer.ogg', 50, FALSE, FALSE)
-		addtimer(CALLBACK(src, PROC_REF(expel_holder), H, TRUE), 20)
+		addtimer(CALLBACK(src, PROC_REF(expel_holder), H, TRUE), 2 SECONDS)
 	else
-		addtimer(CALLBACK(src, PROC_REF(expel_holder), H), 20)
+		addtimer(CALLBACK(src, PROC_REF(expel_holder), H), 2 SECONDS)
 
 /obj/structure/disposaloutlet/proc/expel_holder(obj/structure/disposalholder/H, playsound=FALSE)
 	if(playsound)