prevent crash if get_recipe_index() is passed a nil or apparent garbage ins...

Vanessa Ezekowitz

2015-01-12 bc0ac581aa2b79cd48e4b2083d8c7890ace580aa

 technic/machines/register/recipes.lua

@@ -1,18 +1,23 @@

technic.recipes = {cooking = {numitems = 1}}
function technic.register_recipe_type(typename, desc, numitems)
   numitems = numitems or 1
   if unified_inventory and unified_inventory.register_craft_type then
technic.recipes = { cooking = { input_size = 1, output_size = 1 } }
function technic.register_recipe_type(typename, origdata)
   local data = {}
   for k, v in pairs(origdata) do data[k] = v end
   data.input_size = data.input_size or 1
   data.output_size = data.output_size or 1
   if unified_inventory and unified_inventory.register_craft_type and data.output_size == 1 then
      unified_inventory.register_craft_type(typename, {
         description = desc,
         height = numitems,
         width = 1,
         description = data.description,
         width = data.input_size,
         height = 1,
      })
   end
   technic.recipes[typename] = {numitems = numitems, recipes = {}}
   data.recipes = {}
   technic.recipes[typename] = data
end

local function get_recipe_index(items)
   if not items or type(items) ~= "table" then return false end
   local l = {}
   for i, stack in ipairs(items) do
      l[i] = ItemStack(stack):get_name()
@@ -26,16 +31,26 @@
   for i, stack in ipairs(data.input) do
      data.input[i] = ItemStack(stack):to_string()
   end
   data.output = ItemStack(data.output):to_string()
   if type(data.output) == "table" then
      for i, v in ipairs(data.output) do
         data.output[i] = ItemStack(data.output[i]):to_string()
      end
   else
      data.output = ItemStack(data.output):to_string()
   end
   
   local recipe = {time = data.time, input = {}, output = data.output}
   local index = get_recipe_index(data.input)
   if not index then
      print("[Technic] ignored registration of garbage recipe!")
      return
   end
   for _, stack in ipairs(data.input) do
      recipe.input[ItemStack(stack):get_name()] = ItemStack(stack):get_count()
   end
   
   technic.recipes[typename].recipes[index] = recipe
   if unified_inventory then
   if unified_inventory and technic.recipes[typename].output_size == 1 then
      unified_inventory.register_craft({
         type = typename,
         output = data.output,
@@ -65,12 +80,15 @@
      end
   end
   local index = get_recipe_index(items)
   if not index then
      print("[Technic] ignored registration of garbage recipe!")
      return
   end
   local recipe = technic.recipes[typename].recipes[index]
   if recipe then
      local new_input = {}
      for i, stack in ipairs(items) do
         if stack:get_count() < recipe.input[stack:get_name()] then
            print(stack:get_name())
            return nil
         else
            new_input[i] = ItemStack(stack)