From 8a67b210f14236275a196d791531115329ac6435 Mon Sep 17 00:00:00 2001
From: crazyraio2 <44246096+crazyraio2@users.noreply.github.com>
Date: Fri, 17 Jul 2020 09:27:31 -0300
Subject: [PATCH] now it actually ejects sheets too

---
 sandcode/code/game/mecha/mech_fabricator.dm | 22 +++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/sandcode/code/game/mecha/mech_fabricator.dm b/sandcode/code/game/mecha/mech_fabricator.dm
index 73e2e72702..396da01ed5 100644
--- a/sandcode/code/game/mecha/mech_fabricator.dm
+++ b/sandcode/code/game/mecha/mech_fabricator.dm
@@ -44,6 +44,11 @@
 	RefreshParts() //Recalculating local material sizes if the fab isn't linked
 	return ..()
 
+/obj/machinery/mecha_part_fabricator_adv/examine(mob/user)
+	. = ..()
+	if(in_range(user, src) || isobserver(user))
+		. += "<span class='notice'>The status display reads: Storing up to <b>[rmat.local_size]</b> material units.<br>Material consumption at <b>[component_coeff*100]%</b>.<br>Build time reduced by <b>[100-time_coeff*100]%</b>.</span>"
+
 /obj/machinery/mecha_part_fabricator_adv/RefreshParts()
 	var/T = 0
 
@@ -64,11 +69,6 @@
 		T += Ml.rating
 	time_coeff = round(initial(time_coeff) - (initial(time_coeff)*(T))/5,0.01)
 
-/obj/machinery/mecha_part_fabricator_adv/examine(mob/user)
-	. = ..()
-	if(in_range(user, src) || isobserver(user))
-		. += "<span class='notice'>The status display reads: Storing up to <b>[rmat.local_size]</b> material units.<br>Material consumption at <b>[component_coeff*100]%</b>.<br>Build time reduced by <b>[100-time_coeff*100]%</b>.<span>"
-
 /obj/machinery/mecha_part_fabricator_adv/emag_act()
 	. = ..()
 	if(obj_flags & EMAGGED)
@@ -136,13 +136,14 @@
 		for(var/mat_id in materials.materials)
 			var/datum/material/M = mat_id
 			var/amount = materials.materials[mat_id]
+			var/ref = REF(M)
 			output += "<span class=\"res_name\">[M.name]: </span>[amount] cm&sup3;"
 			if(amount >= MINERAL_MATERIAL_AMOUNT)
-				output += "<span style='font-size:80%;'>- Remove \[<a href='?src=[REF(src)];remove_mat=1;material=[REF(M)]'>1</a>\]"
+				output += "<span style='font-size:80%;'>- Remove \[<a href='?src=[REF(src)];remove_mat=1;material=[ref]'>1</a>\]"
 				if(amount >= (MINERAL_MATERIAL_AMOUNT * 10))
-					output += " | \[<a href='?src=[REF(src)];remove_mat=10;material=[REF(M)]'>10</a>\]"
-				output += " | \[<a href='?src=[REF(src)];remove_mat=50;material=[REF(M)]'>All</a>\]</span>"
-			output += "<br/>"
+					output += " | \[<a href='?src=[REF(src)];remove_mat=10;material=[ref]'>10</a>\]"
+				output += " | \[<a href='?src=[REF(src)];remove_mat=50;material=[ref]'>50</a>\]</span>"
+			output += "<br>"
 	else
 		output += "<font color='red'>No material storage connected, please contact the quartermaster.</font><br>"
 	return output
@@ -427,7 +428,8 @@
 					break
 
 	if(href_list["remove_mat"] && href_list["material"])
-		eject_sheets(href_list["material"], text2num(href_list["remove_mat"]))
+		var/datum/material/Mat = locate(href_list["material"])
+		eject_sheets(Mat, text2num(href_list["remove_mat"]))
 
 	updateUsrDialog()
 	return