Merge pull request #6259 from Citadel-Station-13/upstream-merge-36711

[MIRROR] Fixed sometimes using items on full storage items

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

@@ -423,9 +423,7 @@
 		return	//Robots can't interact with storage items.
 
 	if(!can_be_inserted(W, 0 , user))
-		if(contents.len >= storage_slots) //don't use items on the backpack if they don't fit
-			return 1
-		return 0
+		return 1
 
 	handle_item_insertion(W, 0 , user)
 

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

@@ -207,7 +207,9 @@
 /obj/structure/closet/attackby(obj/item/W, mob/user, params)
 	if(user in src)
 		return
-	if(!src.tool_interact(W,user))
+	if(src.tool_interact(W,user))
+		return 1 // No afterattack
+	else
 		return ..()
 
 /obj/structure/closet/proc/tool_interact(obj/item/W, mob/user)//returns TRUE if attackBy call shouldnt be continued (because tool was used/closet was of wrong type), FALSE if otherwise