Merge branch 'master' into donut

code/modules/food_and_drinks/kitchen_machinery/icecream_vat.dm

@@ -131,7 +131,6 @@
 					I.add_ice_cream(flavour_name, beaker.reagents)
 				else
 					I.add_ice_cream(flavour_name)
-				I.add_ice_cream(flavour_name)
 				if(I.reagents.total_volume < 10)
 					I.reagents.add_reagent("sugar", 10 - I.reagents.total_volume)
 				updateDialog()
@@ -229,7 +228,7 @@
 	desc = "Delicious waffle cone, but no ice cream."
 	icon = 'icons/obj/kitchen.dmi'
 	icon_state = "icecream_cone_waffle" //default for admin-spawned cones, href_list["cone"] should overwrite this all the time
-	list_reagents = list(/datum/reagent/consumable/nutriment = 4)
+	list_reagents = list("nutriment" = 4)
 	tastes = list("cream" = 2, "waffle" = 1)
 	var/ice_creamed = 0
 	var/cone_type
@@ -252,7 +251,7 @@
 	desc = "Delicious [cone_name] cone, but no ice cream."
 
 
-/obj/item/reagent_containers/food/snacks/icecream/proc/add_ice_cream(flavour_name, datum/reagents/R = null)
+/obj/item/reagent_containers/food/snacks/icecream/proc/add_ice_cream(flavour_name, datum/reagents/R)
 	name = "[flavour_name] icecream"
 	switch (flavour_name) // adding the actual reagents advertised in the ingredient list
 		if ("vanilla")
@@ -311,9 +310,11 @@
 	qdel(src)
 
 /obj/machinery/icecream_vat/AltClick(mob/living/user)
+	. = ..()
 	if(!istype(user) || !user.canUseTopic(src, BE_CLOSE, FALSE, NO_TK))
 		return
 	replace_beaker(user)
+	return TRUE
 
 /obj/machinery/icecream_vat/proc/replace_beaker(mob/living/user, obj/item/reagent_containers/new_beaker)
 	if(beaker)

code/modules/food_and_drinks/kitchen_machinery/microwave.dm

@@ -186,8 +186,10 @@
 	..()
 
 /obj/machinery/microwave/AltClick(mob/user)
+	. = ..()
 	if(user.canUseTopic(src, !issilicon(usr)))
 		cook()
+		return TRUE
 
 /obj/machinery/microwave/ui_interact(mob/user)
 	. = ..()