From e4c3ca5fb83ded810c808110ae43d2fa3f46cd09 Mon Sep 17 00:00:00 2001
From: Perakp <perhal90@hotmail.com>
Date: Fri, 6 Dec 2013 13:00:11 +0200
Subject: [PATCH] Janiborgs can no longer place their mop, light replacer or
 trash bag in the janicart. Fixes #1954 Language fix "this the" -> "the" in
 storage.dm

---
 .../objects/items/weapons/storage/storage.dm  |  2 +-
 code/game/objects/structures/janicart.dm      | 84 +++++++++----------
 2 files changed, 42 insertions(+), 44 deletions(-)

diff --git a/code/game/objects/items/weapons/storage/storage.dm b/code/game/objects/items/weapons/storage/storage.dm
index 5ecba49ee27..4a3d973d609 100644
--- a/code/game/objects/items/weapons/storage/storage.dm
+++ b/code/game/objects/items/weapons/storage/storage.dm
@@ -208,7 +208,7 @@
 
 	if(W.w_class > max_w_class)
 		if(!stop_messages)
-			usr << "<span class='notice'>[W] is too big for this [src].</span>"
+			usr << "<span class='notice'>[W] is too big for [src].</span>"
 		return 0
 
 	var/sum_w_class = W.w_class
diff --git a/code/game/objects/structures/janicart.dm b/code/game/objects/structures/janicart.dm
index 5f38e215ee6..342d2288f6c 100644
--- a/code/game/objects/structures/janicart.dm
+++ b/code/game/objects/structures/janicart.dm
@@ -25,64 +25,62 @@
 	usr << "It contains [reagents.total_volume] unit\s of liquid!"
 	//everything else is visible, so doesn't need to be mentioned
 
+/obj/structure/janitorialcart/proc/wet_mop(obj/item/weapon/mop, mob/user)
+	if(reagents.total_volume < 1)
+		user << "[src] is out of water!</span>"
+	else
+		reagents.trans_to(mop, 5)	//
+		user << "<span class='notice'>You wet [mop] in [src].</span>"
+		playsound(loc, 'sound/effects/slosh.ogg', 25, 1)
+
+/obj/structure/janitorialcart/proc/put_in_cart(obj/item/I, mob/user)
+	user.drop_item()
+	I.loc = src
+	update_icon()
+	updateUsrDialog()
+	user << "<span class='notice'>You put [I] into [src].</span>"
+	return I
 
 /obj/structure/janitorialcart/attackby(obj/item/I, mob/user)
-	if(istype(I, /obj/item/weapon/storage/bag/trash) && !mybag)
-		user.drop_item()
-		mybag = I
-		I.loc = src
-		update_icon()
-		updateUsrDialog()
-		user << "<span class='notice'>You put [I] into [src].</span>"
+	var/fail_msg = "<span class='notice'>There is already one of those in [src].</span>"
 
-	else if(istype(I, /obj/item/weapon/mop))
+	if(istype(I, /obj/item/weapon/mop))
 		if(I.reagents.total_volume < I.reagents.maximum_volume)	//if it's not completely soaked we assume they want to wet it, otherwise store it
-			if(reagents.total_volume < 1)
-				user << "[src] is out of water!</span>"
-			else
-				reagents.trans_to(I, 5)	//
-				user << "<span class='notice'>You wet [I] in [src].</span>"
-				playsound(loc, 'sound/effects/slosh.ogg', 25, 1)
-				return
+			wet_mop(I, user)
+			return
+		if(isrobot(user))
+			user << "<span class='notice'>You are a robot. No.</span>"
+			return
 		if(!mymop)
-			user.drop_item()
-			mymop = I
-			I.loc = src
-			update_icon()
-			updateUsrDialog()
-			user << "<span class='notice'>You put [I] into [src].</span>"
+			mymop=put_in_cart(I,user)
+		else user <<  fail_msg;	return
 
-	else if(istype(I, /obj/item/weapon/reagent_containers/spray) && !myspray)
-		user.drop_item()
-		myspray = I
-		I.loc = src
-		update_icon()
-		updateUsrDialog()
-		user << "<span class='notice'>You put [I] into [src].</span>"
-
-	else if(istype(I, /obj/item/device/lightreplacer) && !myreplacer)
-		user.drop_item()
-		myreplacer = I
-		I.loc = src
-		update_icon()
-		updateUsrDialog()
-		user << "<span class='notice'>You put [I] into [src].</span>"
 
+	else if(isrobot(user))
+		user << "<span class='notice'>You are a robot. No.</span>"
+		return
+	else if(istype(I, /obj/item/weapon/storage/bag/trash))
+		if(!mybag)
+			mybag=put_in_cart(I, user)
+		else user <<  fail_msg;	return
+	else if(istype(I, /obj/item/weapon/reagent_containers/spray))
+		if(!myspray)
+			myspray=put_in_cart(I, user)
+		else user << fail_msg;	return
+	else if(istype(I, /obj/item/device/lightreplacer))
+		if(!myreplacer)
+			myreplacer = put_in_cart(I,user)
+		else user << fail_msg;	return
 	else if(istype(I, /obj/item/weapon/caution))
 		if(signs < 4)
-			user.drop_item()
-			I.loc = src
+			put_in_cart(I, user)
 			signs++
-			update_icon()
-			updateUsrDialog()
-			user << "<span class='notice'>You put [I] into [src].</span>"
 		else
 			user << "<span class='notice'>[src] can't hold any more signs.</span>"
-
+			return
 	else if(mybag)
 		mybag.attackby(I, user)
 
-
 /obj/structure/janitorialcart/attack_hand(mob/user)
 	user.set_machine(src)
 	var/dat