Stop quarry duplicating items

The quarry was digging via dig_node and also manually putting the node's
drops into the tube system.  This assumed that dig_node would attempt
to put the drops in the player's inventory, doing nothing if there is
no such inventory.  With the item_drop mod installed, dig_node would
instead turn the node into an item entity, so the quarry's strategy would
duplicate the item, making it appear both as an item entity in situ and
as an item in the tube.  Instead, the quarry must use remove_node when
it manually processes the drops, just like the pipeworks node breaker.
This commit is contained in:
Zefram 2014-05-18 21:03:58 +01:00 committed by RealBadAngel
parent 623fcae4a4
commit 4d20e4473e

@ -107,15 +107,10 @@ local function quarry_dig(pos, center, size)
end end
dig_y = digpos.y dig_y = digpos.y
local node = minetest.get_node(digpos) local node = minetest.get_node(digpos)
drops = minetest.get_node_drops(node.name, "") local node_def = minetest.registered_nodes[node.name] or { diggable = false }
minetest.dig_node(digpos) if node_def.diggable and ((not node_def.can_dig) or node_def.can_dig(digpos, nil)) then
if minetest.get_node(digpos).name == node.name then minetest.remove_node(digpos)
-- We tried to dig something undigable like a drops = minetest.get_node_drops(node.name, "")
-- filled chest. Notice that we check for a node
-- change, not for air. This is so that we get drops
-- from things like concrete posts with platforms,
-- which turn into regular concrete posts when dug.
drops = {}
end end
elseif not (dig_y < pos.y - quarry_max_depth) then elseif not (dig_y < pos.y - quarry_max_depth) then
dig_y = dig_y - 16 dig_y = dig_y - 16