fix(common): remove wrong move_items

Have only copy_items_stack and move_items_stack
This commit is contained in:
oof2win2
2022-05-14 23:22:05 +02:00
parent 49ccd5040a
commit b164e4dd68

View File

@@ -540,7 +540,7 @@ end
--[[-- Moves items to the position and stores them in the closest entity of the type given --[[-- Moves items to the position and stores them in the closest entity of the type given
-- Copies the items by prototype name, but keeps them in the original inventory -- Copies the items by prototype name, but keeps them in the original inventory
@tparam table items items which are to be added to the chests, ['name']=count @tparam table items items which are to be added to the chests, an array of LuaItemStack
@tparam[opt=navies] LuaSurface surface the surface that the items will be moved to @tparam[opt=navies] LuaSurface surface the surface that the items will be moved to
@tparam[opt={0, 0}] table position the position that the items will be moved to {x=100, y=100} @tparam[opt={0, 0}] table position the position that the items will be moved to {x=100, y=100}
@tparam[opt=32] number radius the radius in which the items are allowed to be placed @tparam[opt=32] number radius the radius in which the items are allowed to be placed
@@ -551,7 +551,7 @@ end
move_items(game.player.get_main_inventory().get_contents()) move_items(game.player.get_main_inventory().get_contents())
]] ]]
function Common.move_items(items, surface, position, radius, chest_type) function Common.copy_items_stack(items, surface, position, radius, chest_type)
chest_type = chest_type or 'iron-chest' chest_type = chest_type or 'iron-chest'
surface = surface or game.surfaces[1] surface = surface or game.surfaces[1]
if position and type(position) ~= 'table' then return end if position and type(position) ~= 'table' then return end
@@ -587,12 +587,15 @@ function Common.move_items(items, surface, position, radius, chest_type)
end end
-- Inserts the items into the chests -- Inserts the items into the chests
local last_chest local last_chest
for item_name, item_count in pairs(items) do for i=1,#items do
local chest = next_chest{name=item_name, count=item_count} local item = items[i]
if not chest then return error(string.format('Cant move item %s to %s{%s, %s} no valid chest in radius', item_name, surface.name, p.x, p.y)) end if item.valid_for_read then
Util.insert_safe(chest, {[item_name]=item_count}) local chest = next_chest(item)
if not chest or not chest.valid then return error(string.format('Cant move item %s to %s{%s, %s} no valid chest in radius', item.name, surface.name, p.x, p.y)) end
chest.insert(item)
last_chest = chest last_chest = chest
end end
end
return last_chest return last_chest
end end
@@ -650,7 +653,9 @@ function Common.move_items_stack(items, surface, position, radius, chest_type)
if item.valid_for_read then if item.valid_for_read then
local chest = next_chest(item) local chest = next_chest(item)
if not chest or not chest.valid then return error(string.format('Cant move item %s to %s{%s, %s} no valid chest in radius', item.name, surface.name, p.x, p.y)) end if not chest or not chest.valid then return error(string.format('Cant move item %s to %s{%s, %s} no valid chest in radius', item.name, surface.name, p.x, p.y)) end
chest.insert(item) local empty_stack = chest.get_inventory(defines.inventory.chest).find_empty_stack(item.name)
if not empty_stack then return error(string.format('Cant move item %s to %s{%s, %s} no valid chest in radius', item.name, surface.name, p.x, p.y)) end
empty_stack.transfer_stack(item)
last_chest = chest last_chest = chest
end end
end end