mt-mods/unified_inventory_plus-cd2025
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

prevent reorganization of items with "stack_max = 1"

Browse Source 
some of them have associated metadata that gets cleare in the reorganization

fixes #8
This commit is contained in:
BuckarooBanzay 2020-06-23 08:03:52 +02:00
parent 2b09f0f0a0
commit e286047f5e
1 changed files with 14 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
craft_organize.lua
View File

@ -49,7 +49,7 @@ end
-- Organize items in the craft inventory following a pattern
local function craft_organize(player, formname, fields)
local function craft_organize(player, formname, fields)
local pattern_id
for k, v in pairs(fields) do
pattern_id = k:match("craft_organize_(.*)")
@ -62,30 +62,39 @@ local function craft_organize(player, formname, fields)
local player_name = player:get_player_name()
local craft_list = player_inv:get_list("craft")
local is_creative = minetest.get_player_privs(player_name).creative
-- Organize only on 1 type of item
local only_one_type, type_name = get_type_infos(craft_list)
if not type_name then return end -- craft is empty
if not only_one_type then minetest.chat_send_player(player_name, "You can only organize one type of item.") return end
local itemdef = minetest.registered_items[type_name]
if itemdef and itemdef.stack_max == 1 then
-- disallow non-stackable items
-- most of them have assiciated metadata which gets cleared in the reordering
minetest.chat_send_player(player_name, "You can only organize stackable items.")
return
end
-- Don't exceed 9*99 for non creative players. It shouldn't happen but avoids potential losses then
local total_amount = get_total_amount(craft_list)
if not is_creative and total_amount > 891 then minetest.chat_send_player(player_name, "There are too many items to organize ! Have less than 9 x 99 items.") return end
local res = {ItemStack(type_name),ItemStack(type_name),ItemStack(type_name),ItemStack(type_name),ItemStack(type_name),ItemStack(type_name),ItemStack(type_name),ItemStack(type_name),ItemStack(type_name)}
for i=1,9 do res[i]:set_count(0) end -- Doing this because using empty ItemStack in list constructor crashes the game :S
local pattern = unified_inventory_plus.craft_patterns[tonumber(pattern_id)].pattern
local nb_stacks = 0
for i in pairs(pattern) do nb_stacks = nb_stacks + 1 end
local stack_size = math.floor(total_amount / nb_stacks)
if not is_creative then stack_size = math.min(stack_size, ItemStack(type_name):get_stack_max()) end -- limit stacks to get_stack_max() for non creatives
local remaining = total_amount - nb_stacks * stack_size -- no % nb_stacks if limit: remaining could be greater than a stack
for i=1,nb_stacks do
res[pattern[i]]:add_item(type_name.." "..stack_size)
end
player_inv:set_list("craft", res)
place_item_in_stacks(player, "craft", type_name, remaining)
end