Refactor mcl_banners
This commit is contained in:
parent
c58c12ae72
commit
9f76b5e973
3 changed files with 67 additions and 58 deletions
|
@ -247,38 +247,6 @@ for colorid, colortab in pairs(mcl_banners.colors) do
|
|||
dye_to_colorid_mapping[colortab[5]] = colorid
|
||||
end
|
||||
|
||||
-- Create a banner description containing all the layer names
|
||||
mcl_banners.make_advanced_banner_description = function(description, layers)
|
||||
if layers == nil or #layers == 0 then
|
||||
-- No layers, revert to default
|
||||
return ""
|
||||
else
|
||||
local layerstrings = {}
|
||||
for l=1, #layers do
|
||||
-- Prevent excess length description
|
||||
if l > max_layer_lines then
|
||||
break
|
||||
end
|
||||
-- Layer text line.
|
||||
local color = mcl_banners.colors[layers[l].color][6]
|
||||
local pattern_name = patterns[layers[l].pattern].name
|
||||
-- The pattern name is a format string (e.g. “%s Base”)
|
||||
table.insert(layerstrings, string.format(pattern_name, color))
|
||||
end
|
||||
-- Warn about missing information
|
||||
if #layers == max_layer_lines + 1 then
|
||||
table.insert(layerstrings, "And one addional layer")
|
||||
elseif #layers > max_layer_lines + 1 then
|
||||
table.insert(layerstrings, string.format("And %d addional layers", #layers - max_layer_lines))
|
||||
end
|
||||
|
||||
-- Final string concatenations: Just a list of strings
|
||||
local append = table.concat(layerstrings, "\n")
|
||||
description = description .. "\n" .. core.colorize("#8F8F8F", append)
|
||||
return description
|
||||
end
|
||||
end
|
||||
|
||||
-- This is for handling all those complex pattern crafting recipes
|
||||
local banner_pattern_craft = function(itemstack, player, old_craft_grid, craft_inv)
|
||||
if minetest.get_item_group(itemstack:get_name(), "banner") ~= 1 then
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue