Revert "Close storage UI when item becomes inaccessible (#14536)" (#15126)

This reverts commit f54b6904b9.

code/game/objects/items/weapons/storage/secure.dm

@@ -147,7 +147,6 @@
 		else
 			if((href_list["type"] == "R") && (emagged == 0) && (!l_setshort))
 				locked = 1
-				hide_from_all()
 				overlays = null
 				code = null
 				close(usr)

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

@@ -24,7 +24,6 @@
 	var/allow_quick_gather	//Set this variable to allow the object to have the 'toggle mode' verb, which quickly collects all items from a tile.
 	var/collection_mode = 1;  //0 = pick one at a time, 1 = pick all on tile
 	var/use_sound = "rustle"	//sound played when used. null for no sound.
-	var/list/active_users = list() // list of ckey(user.key), who is viewing the inventory?
 
 	/// What kind of [/obj/item/stack] can this be folded into. (e.g. Boxes and cardboard)
 	var/foldable = null
@@ -130,8 +129,6 @@
 	user.client.screen += closer
 	user.client.screen += contents
 	user.s_active = src
-	if(user.key)
-		active_users[ckey(user.key)] = TRUE
 	return
 
 /obj/item/storage/proc/hide_from(mob/user as mob)
@@ -143,19 +140,8 @@
 	user.client.screen -= src.contents
 	if(user.s_active == src)
 		user.s_active = null
-	if(user.key)
-		active_users.Remove(ckey(user.key))
 	return
 
-/obj/item/storage/proc/hide_from_all()
-	for(var/K in active_users)
-		var/client/C = get_client_by_ckey(K)
-		if(C)
-			var/mob/M = C.mob
-			if(M)
-				hide_from(M)
-
-
 /obj/item/storage/proc/open(mob/user as mob)
 	if(src.use_sound)
 		playsound(src.loc, src.use_sound, 50, 1, -5)

code/game/objects/structures/crates_lockers/closets.dm

@@ -103,9 +103,6 @@
 		if(!I.anchored)
 			I.forceMove(src)
 			itemcount++
-			if(istype(I, /obj/item/storage))
-				var/obj/item/storage/S = I
-				S.hide_from_all()
 
 	for(var/mob/M in loc)
 		if(itemcount >= storage_capacity)

code/game/objects/structures/crates_lockers/crates.dm

@@ -81,9 +81,6 @@
 				continue
 		O.forceMove(src)
 		itemcount++
-		if(istype(O, /obj/item/storage))
-			var/obj/item/storage/S = O
-			S.hide_from_all()
 
 	icon_state = icon_closed
 	src.opened = FALSE

code/modules/mob/living/carbon/carbon.dm

@@ -629,9 +629,6 @@ GLOBAL_LIST_INIT(ventcrawl_machinery, list(/obj/machinery/atmospherics/unary/ven
 			return
 
 	if(thrown_thing)
-		if(istype(thrown_thing, /obj/item/storage))
-			var/obj/item/storage/S = thrown_thing
-			S.hide_from_all()
 		visible_message("<span class='danger'>[src] has thrown [thrown_thing].</span>")
 		newtonian_move(get_dir(target, src))
 		thrown_thing.throw_at(target, thrown_thing.throw_range, thrown_thing.throw_speed, src, null, null, null, move_force)

code/modules/recycling/disposal.dm

@@ -103,8 +103,8 @@
 		to_chat(user, "You can't place that item inside the disposal unit.")
 		return
 
-	var/obj/item/storage/S = I
-	if(istype(S))
+	if(istype(I, /obj/item/storage))
+		var/obj/item/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].")
@@ -134,8 +134,6 @@
 	if(!user.drop_item())
 		return
 	if(I)
-		if(istype(S))
-			S.hide_from_all()
 		I.forceMove(src)
 
 	to_chat(user, "You place \the [I] into the [src].")

code/modules/research/destructive_analyzer.dm

@@ -81,9 +81,6 @@ Note: Must be placed within 3 tiles of the R&D Console
 		busy = 1
 		loaded_item = O
 		O.loc = src
-		if(istype(O, /obj/item/storage))
-			var/obj/item/storage/S = O
-			S.hide_from_all()
 		to_chat(user, "<span class='notice'>You add the [O.name] to the [src.name]!</span>")
 		flick("d_analyzer_la", src)
 		spawn(10)