Some style cleaning

This commit is contained in:
kilbith 2016-02-02 20:17:29 +01:00
parent c36590b46f
commit ad8e84f249
2 changed files with 38 additions and 71 deletions

View File

@ -117,9 +117,7 @@ function enchanting.put(_, listname, _, stack)
end end
function enchanting.on_take(pos, listname) function enchanting.on_take(pos, listname)
if listname == "tool" then if listname == "tool" then enchanting.formspec(pos, nil) end
enchanting.formspec(pos, nil)
end
end end
function enchanting.construct(pos) function enchanting.construct(pos)

View File

@ -39,8 +39,7 @@ worktable.defs = {
-- Tools allowed to be repaired. -- Tools allowed to be repaired.
worktable.repairable_tools = [[ worktable.repairable_tools = [[
pick, axe, shovel, sword, hoe pick, axe, shovel, sword, hoe, armor, shield
armor, shield
]] ]]
function worktable.get_recipe(item) function worktable.get_recipe(item)
@ -49,12 +48,9 @@ function worktable.get_recipe(item)
item = item:sub(7)..":white" item = item:sub(7)..":white"
elseif minetest.registered_items["default:"..item:sub(7)] then elseif minetest.registered_items["default:"..item:sub(7)] then
item = item:gsub("group:", "default:") item = item:gsub("group:", "default:")
else else for node, def in pairs(minetest.registered_items) do
for node, def in pairs(minetest.registered_items) do if def.groups[item:match("[^,:]+$")] then item = node end
if def.groups[item:match("[^,:]+$")] then end
item = node
end
end
end end
end end
return item return item
@ -65,11 +61,8 @@ function worktable.craftguide_formspec(meta, pagenum, item, recipe_num, filter,
local npp, i, s = 8*3, 0, 0 local npp, i, s = 8*3, 0, 0
local pagemax = math.floor((inv_size - 1) / npp + 1) local pagemax = math.floor((inv_size - 1) / npp + 1)
if pagenum > pagemax then if pagenum > pagemax then pagenum = 1
pagenum = 1 elseif pagenum == 0 then pagenum = pagemax end
elseif pagenum == 0 then
pagenum = pagemax
end
local formspec = [[ size[8,6.6;] local formspec = [[ size[8,6.6;]
tablecolumns[color;text;color;text] tablecolumns[color;text;color;text]
@ -89,10 +82,9 @@ function worktable.craftguide_formspec(meta, pagenum, item, recipe_num, filter,
for _, name in pairs(worktable.craftguide_main_list(meta, filter, tab_id)) do for _, name in pairs(worktable.craftguide_main_list(meta, filter, tab_id)) do
if s < (pagenum - 1) * npp then if s < (pagenum - 1) * npp then
s = s + 1 s = s + 1
else else if i >= npp then break end
if i >= npp then break end
formspec = formspec.."item_image_button["..(i%8)..",".. formspec = formspec.."item_image_button["..(i%8)..","..
(math.floor(i/8)+1)..";1,1;"..name..";"..name..";]" (math.floor(i/8)+1)..";1,1;"..name..";"..name..";]"
i = i + 1 i = i + 1
end end
end end
@ -102,14 +94,14 @@ function worktable.craftguide_formspec(meta, pagenum, item, recipe_num, filter,
local items_num = #minetest.get_all_craft_recipes(item) local items_num = #minetest.get_all_craft_recipes(item)
if recipe_num > items_num then recipe_num = 1 end if recipe_num > items_num then recipe_num = 1 end
if items_num > 1 then if items_num > 1 then formspec = formspec..
formspec = formspec.."button[0,6;1.6,1;alternate;Alternate]".. "button[0,6;1.6,1;alternate;Alternate]"..
"label[0,5.5;Recipe "..recipe_num.." of "..items_num.."]" "label[0,5.5;Recipe "..recipe_num.." of "..items_num.."]"
end end
local type = minetest.get_all_craft_recipes(item)[recipe_num].type local type = minetest.get_all_craft_recipes(item)[recipe_num].type
if type == "cooking" then if type == "cooking" then formspec = formspec..
formspec = formspec.."image[3.75,4.6;0.5,0.5;default_furnace_fire_fg.png]" "image[3.75,4.6;0.5,0.5;default_furnace_fire_fg.png]"
end end
local items = minetest.get_all_craft_recipes(item)[recipe_num].items local items = minetest.get_all_craft_recipes(item)[recipe_num].items
@ -123,14 +115,14 @@ function worktable.craftguide_formspec(meta, pagenum, item, recipe_num, filter,
return "" return ""
end end
for i, v in pairs(items) do for i, v in pairs(items) do formspec = formspec..
formspec = formspec.."item_image_button["..((i-1) % width + 4.5)..",".. "item_image_button["..((i-1) % width + 4.5)..","..
(math.floor((i-1) / width + (6 - math.min(2, rows))))..";1,1;".. (math.floor((i-1) / width + (6 - math.min(2, rows))))..";1,1;"..
worktable.get_recipe(v)..";"..worktable.get_recipe(v)..";"..is_group(v).."]" worktable.get_recipe(v)..";"..worktable.get_recipe(v)..";"..is_group(v).."]"
end end
formspec = formspec.."item_image_button[2.5,5;1,1;"..item..";"..item..";"..yield.."]".. formspec = formspec.."item_image_button[2.5,5;1,1;"..item..";"..item..";"..yield.."]"..
"image[3.5,5;1,1;gui_furnace_arrow_bg.png^[transformR90]" "image[3.5,5;1,1;gui_furnace_arrow_bg.png^[transformR90]"
end end
meta:set_string("formspec", formspec) meta:set_string("formspec", formspec)
@ -143,7 +135,6 @@ local function tab_category(tab_id)
minetest.registered_tools, minetest.registered_tools,
minetest.registered_craftitems minetest.registered_craftitems
} }
return id_category[tab_id] or id_category[1] return id_category[tab_id] or id_category[1]
end end
@ -254,18 +245,14 @@ function worktable.fields(pos, _, fields)
worktable.craftguide_main_list(meta, filter, tonumber(fields.tabs)) worktable.craftguide_main_list(meta, filter, tonumber(fields.tabs))
worktable.craftguide_formspec(meta, 1, nil, 1, filter, tonumber(fields.tabs)) worktable.craftguide_formspec(meta, 1, nil, 1, filter, tonumber(fields.tabs))
elseif fields.prev or fields.next then elseif fields.prev or fields.next then
if fields.prev then if fields.prev then pagenum = pagenum - 1
pagenum = pagenum - 1 else pagenum = pagenum + 1 end
else
pagenum = pagenum + 1
end
worktable.craftguide_formspec(meta, pagenum, nil, 1, filter, tab_id) worktable.craftguide_formspec(meta, pagenum, nil, 1, filter, tab_id)
else else for item in pairs(fields) do
for item in pairs(fields) do if minetest.get_craft_recipe(item).items then
if item:find(":") and minetest.get_craft_recipe(item).items then worktable.craftguide_formspec(meta, pagenum, item, 1, filter, tab_id)
worktable.craftguide_formspec(meta, pagenum, item, 1, filter, tab_id) end
end end
end
end end
end end
@ -277,9 +264,7 @@ end
local function trash_delete(pos) local function trash_delete(pos)
local inv = minetest.get_meta(pos):get_inventory() local inv = minetest.get_meta(pos):get_inventory()
minetest.after(0, function() minetest.after(0, function() inv:set_stack("trash", 1, "") end)
inv:set_stack("trash", 1, "")
end)
end end
function worktable.put(pos, listname, _, stack) function worktable.put(pos, listname, _, stack)
@ -294,26 +279,21 @@ function worktable.put(pos, listname, _, stack)
if listname == "trash" then trash_delete(pos) end if listname == "trash" then trash_delete(pos) end
return stack:get_count() return stack:get_count()
end end
return 0 return 0
end end
function worktable.take(_, listname, _, stack, player) function worktable.take(_, listname, _, stack, player)
if listname == "forms" then if listname == "forms" then
local inv = player:get_inventory() local free_space = player:get_inventory():room_for_item("main", stack:get_name())
if inv:room_for_item("main", stack:get_name()) then if free_space then return -1 end
return -1
end
return 0 return 0
end end
return stack:get_count() return stack:get_count()
end end
function worktable.move(pos, _, _, to_list, _, count) function worktable.move(pos, _, _, to_list, _, count)
if to_list == "storage" then if to_list == "storage" or to_list == "trash" then
return count if to_list == "trash" then trash_delete(pos) end
elseif to_list == "trash" then
trash_delete(pos)
return count return count
end end
return 0 return 0
@ -321,19 +301,15 @@ end
function worktable.get_output(inv, input, name) function worktable.get_output(inv, input, name)
if inv:is_empty("input") then if inv:is_empty("input") then
inv:set_list("forms", {}) inv:set_list("forms", {}) return end
return
end
local output = {} local output = {}
for _, n in pairs(worktable.defs) do for _, n in pairs(worktable.defs) do
local count = math.min(n[2] * input:get_count(), input:get_stack_max()) local count = math.min(n[2] * input:get_count(), input:get_stack_max())
local item = name.."_"..n[1] local item = name.."_"..n[1]
if not n[3] then item = "stairs:"..n[1].."_"..name:match(":(.*)") end if not n[3] then item = "stairs:"..n[1].."_"..name:match(":(.*)") end
output[#output+1] = item.." "..count output[#output+1] = item.." "..count
end end
inv:set_list("forms", output) inv:set_list("forms", output)
end end
@ -352,9 +328,7 @@ function worktable.on_take(pos, listname, index, stack)
if listname == "input" then if listname == "input" then
if stack:get_name() == input:get_name() then if stack:get_name() == input:get_name() then
worktable.get_output(inv, input, stack:get_name()) worktable.get_output(inv, input, stack:get_name())
else else inv:set_list("forms", {}) end
inv:set_list("forms", {})
end
elseif listname == "forms" then elseif listname == "forms" then
input:take_item(math.ceil(stack:get_count() / worktable.defs[index][2])) input:take_item(math.ceil(stack:get_count() / worktable.defs[index][2]))
inv:set_stack("input", 1, input) inv:set_stack("input", 1, input)
@ -417,8 +391,7 @@ for _, d in pairs(worktable.defs) do
local player_name = clicker:get_player_name() local player_name = clicker:get_player_name()
if minetest.is_protected(pos, player_name) then if minetest.is_protected(pos, player_name) then
minetest.record_protection_violation(pos, player_name) minetest.record_protection_violation(pos, player_name)
return return end
end
local T = { local T = {
{"nanoslab", nil, 2}, {"nanoslab", nil, 2},
@ -441,6 +414,7 @@ for _, d in pairs(worktable.defs) do
if wield_item == newnode.."_"..x[1] then if wield_item == newnode.."_"..x[1] then
if not x[2] then x[2] = x[1] end if not x[2] then x[2] = x[1] end
local pointed_nodebox = minetest.get_node(pos).name:match("(%w+)$") local pointed_nodebox = minetest.get_node(pos).name:match("(%w+)$")
if x[2] == pointed_nodebox then if x[2] == pointed_nodebox then
if x[3] then newnode = newnode.."_"..worktable.defs[x[3]][1] end if x[3] then newnode = newnode.."_"..worktable.defs[x[3]][1] end
combined = true combined = true
@ -448,9 +422,7 @@ for _, d in pairs(worktable.defs) do
end end
end end
end end
else else minetest.item_place_node(itemstack, clicker, pointed_thing) end
minetest.item_place_node(itemstack, clicker, pointed_thing)
end
if combined and not minetest.setting_getbool("creative_mode") then if combined and not minetest.setting_getbool("creative_mode") then
itemstack:take_item() itemstack:take_item()
@ -462,9 +434,7 @@ for _, d in pairs(worktable.defs) do
if node:find("meselamp") then if node:find("meselamp") then
if d[3] then if d[3] then
minetest.register_alias("default:meselamp_"..d[1], "default:glass_"..d[1]) minetest.register_alias("default:meselamp_"..d[1], "default:glass_"..d[1])
else else minetest.register_alias("stairs:"..d[1].."_meselamp", "stairs:"..d[1].."_glass") end
minetest.register_alias("stairs:"..d[1].."_meselamp", "stairs:"..d[1].."_glass")
end
elseif worktable.nodes(def) and not d[3] then elseif worktable.nodes(def) and not d[3] then
minetest.register_alias(node.."_"..d[1], "stairs:"..d[1].."_"..node:match(":(.*)")) minetest.register_alias(node.."_"..d[1], "stairs:"..d[1].."_"..node:match(":(.*)"))
end end
@ -476,8 +446,7 @@ minetest.register_abm({
interval = 3, chance = 1, interval = 3, chance = 1,
action = function(pos) action = function(pos)
local inv = minetest.get_meta(pos):get_inventory() local inv = minetest.get_meta(pos):get_inventory()
local tool = inv:get_stack("tool", 1) local tool, hammer = inv:get_stack("tool", 1), inv:get_stack("hammer", 1)
local hammer = inv:get_stack("hammer", 1)
if tool:is_empty() or hammer:is_empty() or tool:get_wear() == 0 then if tool:is_empty() or hammer:is_empty() or tool:get_wear() == 0 then
return return