From d8e6893e359a635127d4cd2920e324b60154bf89 Mon Sep 17 00:00:00 2001
From: Krausus <vaeusx@gmail.com>
Date: Fri, 8 May 2015 07:20:44 -0400
Subject: [PATCH] Improves dumping of "quick" containers

Allows all "quick" containers to be dumped into a disposal (click) or
onto a table (drag-drop).
---
 .../objects/items/weapons/storage/storage.dm  | 12 ++++++++++
 code/modules/recycling/disposal.dm            | 24 +++++++------------
 2 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/code/game/objects/items/weapons/storage/storage.dm b/code/game/objects/items/weapons/storage/storage.dm
index b22b813d1c1..e705521954d 100644
--- a/code/game/objects/items/weapons/storage/storage.dm
+++ b/code/game/objects/items/weapons/storage/storage.dm
@@ -38,6 +38,18 @@
 			show_to(M)
 			return
 
+		if((allow_quick_empty || allow_quick_gather) && istype(over_object, /obj/structure/table) \
+			&& contents.len && over_object.Adjacent(usr))
+			var/turf/T = get_turf(over_object)
+			hide_from(usr)
+			usr.face_atom(src)
+			usr.visible_message("<span class='notice'>[usr] empties \the [src] onto \the [over_object].</span>",
+				"<span class='notice'>You empty \the [src] onto \the [over_object].</span>")
+			for(var/obj/item/I in contents)
+				remove_from_storage(I, T)
+			update_icon() // For content-sensitive icons
+			return
+
 		if (!( istype(over_object, /obj/screen) ))
 			return ..()
 		if (!(src.loc == usr) || (src.loc && src.loc.loc == usr))
diff --git a/code/modules/recycling/disposal.dm b/code/modules/recycling/disposal.dm
index d1506aa8dbc..5bc8b20a6bb 100644
--- a/code/modules/recycling/disposal.dm
+++ b/code/modules/recycling/disposal.dm
@@ -93,22 +93,14 @@
 			user << "You can't place that item inside the disposal unit."
 			return
 
-		if(istype(I, /obj/item/weapon/storage/bag/trash))
-			var/obj/item/weapon/storage/bag/trash/T = I
-			if(T.contents.len)
-				user << "\blue You empty the bag."
-				for(var/obj/item/O in T.contents)
-					T.remove_from_storage(O,src)
-				T.update_icon()
-				update()
-				return
-
-		if(istype(I, /obj/item/weapon/storage/part_replacer))
-			var/obj/item/weapon/storage/part_replacer/P = I
-			if(P.contents.len)
-				user << "\blue You empty the RPED's contents."
-				for(var/obj/item/O in P.contents)
-					P.remove_from_storage(O,src)
+		if(istype(I, /obj/item/weapon/storage))
+			var/obj/item/weapon/storage/S = I
+			if((S.allow_quick_empty || S.allow_quick_gather) && S.contents.len)
+				S.hide_from(user)
+				user.visible_message("[user] empties \the [S] into \the [src].", "You empty \the [S] into \the [src].")
+				for(var/obj/item/O in S.contents)
+					S.remove_from_storage(O, src)
+				S.update_icon() // For content-sensitive icons
 				update()
 				return