From 40812e66e8b0d26fa32e38d7cc89347298fed04d Mon Sep 17 00:00:00 2001
From: SkyratBot <59378654+SkyratBot@users.noreply.github.com>
Date: Thu, 3 Feb 2022 10:58:51 +0100
Subject: [PATCH] [MIRROR] Fixes disposal self deletion [MDB IGNORE] (#11248)

* Fixes disposal self deletion (#64612)

Disposal holders are meant to stack up in the pipe if they're not moving
The check for this is typically done using the active variable when two
holders collide.
Unfortunately when I converted holders over to move loops, I forgot to
set active to false properly.

This means the part of deconstruct code that handled ejecting the
contents of the pipe couldn't really function, since there was more then
one holder stored in the pipe. so all but one of the holders just got
qdel'd. Not good, makes people upset for some reason

This resolves that issue

* Fixes disposal self deletion

Co-authored-by: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com>
---
 code/modules/recycling/disposal/holder.dm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/code/modules/recycling/disposal/holder.dm b/code/modules/recycling/disposal/holder.dm
index 8468ec3e90d..3168a36e631 100644
--- a/code/modules/recycling/disposal/holder.dm
+++ b/code/modules/recycling/disposal/holder.dm
@@ -89,6 +89,7 @@
 	SIGNAL_HANDLER
 	current_pipe = null
 	last_pipe = null
+	active = FALSE
 
 //failsafe in the case the holder is somehow forcemoved somewhere that's not a disposal pipe. Otherwise the above loop breaks.
 /obj/structure/disposalholder/Moved(atom/oldLoc, dir)