Pushing to review diff

code/modules/reagents/chemistry/machinery/chem_dispenser.dm

@@ -33,7 +33,6 @@
 	var/nopower_state = "dispenser_nopower"
 	var/has_panel_overlay = TRUE
 	var/obj/item/reagent_containers/beaker = null
-	var/list/stored_beakers = list()
 	var/list/dispensable_reagents = list(
 		/datum/reagent/hydrogen,
 		/datum/reagent/lithium,
@@ -363,10 +362,11 @@
 				return 
 			var/reagent = text2path(params["id"])
 			var/datum/reagent/R = beaker.reagents.has_reagent(reagent)
-			if(reagents.total_volume+amount > reagents.maximum_volume)
+			var/potentialAmount = min(amount, R.volume)
+			if(reagents.total_volume+potentialAmount > reagents.maximum_volume)
 				say("Not enough storage space left!")
 				return
-			beaker.reagents.trans_id_to(src, R.type, amount)
+			beaker.reagents.trans_id_to(src, R.type, potentialAmount)
 			work_animation()				
 			. = TRUE
 		
@@ -455,13 +455,13 @@
 		if(M.rating > 3)
 			dispensable_reagents |= upgrade_reagents3
 		switch(M.rating)
-			if(0)
+			if(-INFINITY to 1)
 				dispenceUnit = 5
-			if(1)
-				dispenceUnit = 3
 			if(2)
+				dispenceUnit = 3
+			if(3)
 				dispenceUnit = 2
-			if(3 to INFINITY)
+			if(4 to INFINITY)
 				dispenceUnit = 1
 	powerefficiency = round(newpowereff, 0.01)
 

modular_citadel/code/modules/reagents/chemistry/reagents/fermi_reagents.dm

@@ -345,12 +345,16 @@ datum/reagent/fermi/nanite_b_gone/reaction_obj(obj/O, reac_volume)
 
 //Consumes self on addition and shifts pH
 /datum/reagent/fermi/acidic_buffer/on_new(datapH)
+	if(!holder)
+		return ..()
+	if(holder.reagents_holder_flags & NO_REACT)
+		return..()
 	if(holder.has_reagent(/datum/reagent/stabilizing_agent))
 		return ..()
 	data = datapH
 	if(LAZYLEN(holder.reagent_list) == 1)
 		return ..()
-	holder.pH = ((holder.pH * (holder.total_volume-volume))+(pH * (volume*strength)))/(holder.total_volume + volume)
+	holder.pH = ((holder.pH * (holder.total_volume-volume))+(pH * volume*strength))/holder.total_volume //This is BEFORE removal
 	holder.my_atom.visible_message("<span class='warning'>The beaker fizzes as the pH changes!</b></span>")
 	playsound(holder.my_atom, 'sound/FermiChem/bufferadd.ogg', 50, 1)
 	holder.remove_reagent(type, volume, ignore_pH = TRUE)
@@ -382,12 +386,16 @@ datum/reagent/fermi/nanite_b_gone/reaction_obj(obj/O, reac_volume)
 	strength = 0.4
 
 /datum/reagent/fermi/basic_buffer/on_new(datapH)
+	if(!holder)
+		return ..()
+	if(holder.reagents_holder_flags & NO_REACT)
+		return..()
 	if(holder.has_reagent(/datum/reagent/stabilizing_agent))
 		return ..()
 	data = datapH
 	if(LAZYLEN(holder.reagent_list) == 1)
 		return ..()
-	holder.pH = ((holder.pH * (holder.total_volume-volume))+(pH * (volume*strength)))/(holder.total_volume + volume)
+	holder.pH = ((holder.pH * (holder.total_volume-volume))+(pH * volume*strength))/holder.total_volume //This is BEFORE removal
 	holder.my_atom.visible_message("<span class='warning'>The beaker froths as the pH changes!</b></span>")
 	playsound(holder.my_atom, 'sound/FermiChem/bufferadd.ogg', 50, 1)
 	holder.remove_reagent(type, volume, ignore_pH = TRUE)

tgui/packages/tgui/interfaces/ChemDispenser.js

@@ -226,7 +226,7 @@ export const ChemDispenser = (props, context) => {
 const ChemicalBuffer = Table;
 
 const ChemicalBufferEntry = (props, context) => {
-  const { act } = useBackend(context);
+  const { act, data } = useBackend(context);
   const { chemical, transferTo } = props;
   return (
     <Table.Row key={chemical.id}>
@@ -240,6 +240,7 @@ const ChemicalBufferEntry = (props, context) => {
         <Button
           content="Dispense"
           icon="download"
+          disabled={!!data.recordingRecipe}
           mt={0.5}
           onClick={() => act('unstore', {
             id: chemical.id,
@@ -252,7 +253,7 @@ const ChemicalBufferEntry = (props, context) => {
 const ChemicalBeaker = Table;
 
 const ChemicalBeakerEntry = (props, context) => {
-  const { act } = useBackend(context);
+  const { act, data } = useBackend(context);
   const { chemical, transferTo } = props;
   return (
     <Table.Row key={chemical.id}>
@@ -266,6 +267,7 @@ const ChemicalBeakerEntry = (props, context) => {
         <Button
           content="Store"
           icon="upload"
+          disabled={!!data.recordingRecipe}
           mt={0.5}
           onClick={() => act('store', {
             id: chemical.id,

tgui/packages/tgui/interfaces/ChemMaster.js

@@ -403,7 +403,7 @@ const AnalysisResults = (props, context) => {
         <LabeledList.Item label="Purity">
           {analyzeVars.purityF}
         </LabeledList.Item>
-        {!! data.fermianalyze && ( // why did you do that before? it's fucking bad.
+        {!! data.fermianalyze && ( // why did you do that before? it's bad.
           <Fragment>
             <LabeledList.Item label="Inverse Ratio">
               {analyzeVars.inverseRatioF}