Buckets use liquid_pointable=true

This commit is contained in:
parent d00f5a695f
commit 7c01e34c46
1 changed files with 52 additions and 115 deletions

View File

@ -17,7 +17,6 @@ local vector = vector
local math = math local math = math
local string = string local string = string
local raycast = minetest.raycast
local get_node = minetest.get_node local get_node = minetest.get_node
local set_node = minetest.set_node local set_node = minetest.set_node
local add_node = minetest.add_node local add_node = minetest.add_node
@ -87,24 +86,6 @@ end
local pointable_sources = {} local pointable_sources = {}
local function bucket_raycast(user)
local pos = user:get_pos()
pos.y = pos.y + user:get_properties().eye_height
local look_dir = user:get_look_dir()
look_dir = vector.multiply(look_dir, 5)
local pos2 = vector.add(pos, look_dir)
local ray = raycast(pos, pos2, false, true)
if ray then
for pointed_thing in ray do
if pointed_thing and pointable_sources[get_node(pointed_thing.above).name] then
return {under=pointed_thing.under,above=pointed_thing.above}
end
end
end
return nil
end
local function get_node_place(source_place, place_pos) local function get_node_place(source_place, place_pos)
local node_place local node_place
if type(source_place) == "function" then if type(source_place) == "function" then
@ -152,65 +133,48 @@ local function get_bucket_drop(itemstack, user, take_bucket)
end end
end end
local function on_place_bucket(itemstack, user, pointed_thing, def) local function on_place_bucket(itemstack, user, pointed_thing)
-- Must be pointing to node -- Must be pointing to node
if pointed_thing.type ~= "node" then if pointed_thing.type ~= "node" then
return return
end end
-- Call on_rightclick if the pointed node defines it -- Call on_rightclick if the pointed node defines it
local new_stack = mcl_util.call_on_rightclick(itemstack, user, pointed_thing) local new_stack = mcl_util.call_on_rightclick(itemstack, user, pointed_thing)
if new_stack then if new_stack then
return new_stack return new_stack
end end
local undernode = get_node(pointed_thing.under) for i = 1, 2 do
local abovenode = get_node(pointed_thing.above) pos = i == 1 and pointed_thing.under or pointed_thing.above
local node = get_node(pos)
local node_def = registered_nodes[node.name]
local bucket_def = mcl_buckets.buckets[itemstack:get_name()]
if registered_nodes[undernode.name] and registered_nodes[undernode.name].buildable_to or get_item_group(undernode.name, "cauldron") == 1 then if node_def and node_def.buildable_to or get_item_group(node.name, "cauldron") == 1 then
local result, take_bucket = get_extra_check(def.extra_check, pointed_thing.under, user) local result, take_bucket = get_extra_check(bucket_def.extra_check, pos, user)
if result then if result then
local node_place = get_node_place(def.source_place, pointed_thing.under) local node_place = get_node_place(bucket_def.source_place, pos)
local pns = user:get_player_name() local pns = user:get_player_name()
-- Check protection -- Check protection
if is_protected(pointed_thing.under, pns) then if is_protected(pos, pns) then
record_protection_violation(pointed_thing.under, pns) record_protection_violation(pos, pns)
return itemstack return itemstack
end end
-- Place liquid -- Place liquid
place_liquid(pointed_thing.under, node_place) place_liquid(pos, node_place)
-- Update doc mod -- Update doc mod
if mod_doc and doc.entry_exists("nodes", node_place) then if mod_doc and doc.entry_exists("nodes", node_place) then
doc.mark_entry_as_revealed(user:get_player_name(), "nodes", node_place) doc.mark_entry_as_revealed(user:get_player_name(), "nodes", node_place)
end
end end
return get_bucket_drop(itemstack, user, take_bucket)
end end
return get_bucket_drop(itemstack, user, take_bucket)
elseif registered_nodes[abovenode.name] and registered_nodes[abovenode.name].buildable_to or get_item_group(abovenode.name, "cauldron") == 1 then
local result, take_bucket = get_extra_check(def.extra_check, pointed_thing.above, user)
if result then
local node_place = get_node_place(def.source_place, pointed_thing.above)
local pns = user:get_player_name()
-- Check protection
if is_protected(pointed_thing.above, pns) then
record_protection_violation(pointed_thing.above, pns)
return itemstack
end
-- Place liquid
place_liquid(pointed_thing.above, node_place)
-- Update doc mod
if mod_doc and doc.entry_exists("nodes", node_place) then
doc.mark_entry_as_revealed(user:get_player_name(), "nodes", node_place)
end
end
return get_bucket_drop(itemstack, user, take_bucket)
else
return itemstack
end end
return itemstack
end end
@ -226,18 +190,15 @@ local function on_place_bucket_empty(itemstack, user, pointed_thing)
return new_stack return new_stack
end end
local node = get_node(pointed_thing.under)
local nn = node.name
local new_bucket local new_bucket
local liquid_node = bucket_raycast(user) local under = pointed_thing.under
if liquid_node then local node_name = get_node(under).name
if is_protected(liquid_node.above, user:get_player_name()) then if pointable_sources[node_name] then
record_protection_violation(liquid_node.above, user:get_player_name()) if is_protected(under, user:get_player_name()) then
record_protection_violation(under, user:get_player_name())
end end
local liquid_name = get_node(liquid_node.above).name if node_name then
if liquid_name then local liquid_def = mcl_buckets.liquids[node_name]
local liquid_def = mcl_buckets.liquids[liquid_name]
if liquid_def then if liquid_def then
-- Fill bucket, but not in Creative Mode -- Fill bucket, but not in Creative Mode
-- FIXME: remove this line -- FIXME: remove this line
@ -248,42 +209,42 @@ local function on_place_bucket_empty(itemstack, user, pointed_thing)
liquid_def.on_take(user) liquid_def.on_take(user)
end end
end end
add_node(liquid_node.above, {name="air"}) add_node(under, {name="air"})
sound_take(nn, liquid_node.above) sound_take(node_name, under)
if mod_doc and doc.entry_exists("nodes", liquid_name) then if mod_doc and doc.entry_exists("nodes", node_name) then
doc.mark_entry_as_revealed(user:get_player_name(), "nodes", liquid_name) doc.mark_entry_as_revealed(user:get_player_name(), "nodes", node_name)
end end
if new_bucket then if new_bucket then
return give_bucket(new_bucket, itemstack, user) return give_bucket(new_bucket, itemstack, user)
end end
else else
minetest.log("error", string.format("[mcl_buckets] Node [%s] has invalid group [_mcl_bucket_pointable]!", liquid_name)) minetest.log("error", string.format("[mcl_buckets] Node [%s] has invalid group [_mcl_bucket_pointable]!", node_name))
end end
end end
return itemstack return itemstack
else else
-- FIXME: replace this ugly code by cauldrons API -- FIXME: replace this ugly code by cauldrons API
if nn == "mcl_cauldrons:cauldron_3" then if node_name == "mcl_cauldrons:cauldron_3" then
-- Take water out of full cauldron -- Take water out of full cauldron
set_node(pointed_thing.under, {name="mcl_cauldrons:cauldron"}) set_node(under, {name="mcl_cauldrons:cauldron"})
if not is_creative_enabled(user:get_player_name()) then if not is_creative_enabled(user:get_player_name()) then
new_bucket = ItemStack("mcl_buckets:bucket_water") new_bucket = ItemStack("mcl_buckets:bucket_water")
end end
sound_take("mcl_core:water_source", pointed_thing.under) sound_take("mcl_core:water_source", under)
elseif nn == "mcl_cauldrons:cauldron_3r" then elseif node_name == "mcl_cauldrons:cauldron_3r" then
-- Take river water out of full cauldron -- Take river water out of full cauldron
set_node(pointed_thing.under, {name="mcl_cauldrons:cauldron"}) set_node(under, {name="mcl_cauldrons:cauldron"})
if not is_creative_enabled(user:get_player_name()) then if not is_creative_enabled(user:get_player_name()) then
new_bucket = ItemStack("mcl_buckets:bucket_river_water") new_bucket = ItemStack("mcl_buckets:bucket_river_water")
end end
sound_take("mclx_core:river_water_source", pointed_thing.under) sound_take("mclx_core:river_water_source", under)
elseif nn == "mcl_cauldrons:cauldron_3_lava" then elseif node_name == "mcl_cauldrons:cauldron_3_lava" then
set_node(pointed_thing.under, {name="mcl_cauldrons:cauldron"}) set_node(under, {name="mcl_cauldrons:cauldron"})
if not is_creative_enabled(user:get_player_name()) then if not is_creative_enabled(user:get_player_name()) then
new_bucket = ItemStack("mcl_buckets:bucket_lava") new_bucket = ItemStack("mcl_buckets:bucket_lava")
end end
sound_take("mcl_core:lava_source", pointed_thing.under) sound_take("mcl_core:lava_source", under)
end end
if new_bucket then if new_bucket then
return give_bucket(new_bucket, itemstack, user) return give_bucket(new_bucket, itemstack, user)
@ -292,34 +253,6 @@ local function on_place_bucket_empty(itemstack, user, pointed_thing)
return itemstack return itemstack
end end
controls.register_on_press(function(player, key)
if key ~= "RMB" then
return
end
local wielded_item = player:get_wielded_item()
local itemname = wielded_item:get_name()
local def = mcl_buckets.buckets[itemname]
if itemname == "mcl_buckets:bucket_empty" then
local pointed_thing = mcl_util.get_pointed_thing(player, true)
if not pointed_thing then
return
end
wielded_item = on_place_bucket_empty(wielded_item, player, pointed_thing)
elseif def then
local pointed_thing = mcl_util.get_pointed_thing(player, false)
if not pointed_thing then
return
end
wielded_item = on_place_bucket(wielded_item, player, pointed_thing, def)
end
player:set_wielded_item(wielded_item)
end)
function mcl_buckets.register_liquid(def) function mcl_buckets.register_liquid(def)
for _,source in ipairs(def.source_take) do for _,source in ipairs(def.source_take) do
mcl_buckets.liquids[source] = { mcl_buckets.liquids[source] = {
@ -348,6 +281,8 @@ function mcl_buckets.register_liquid(def)
inventory_image = def.inventory_image, inventory_image = def.inventory_image,
stack_max = 1, stack_max = 1,
groups = def.groups, groups = def.groups,
liquids_pointable = true,
on_place = on_place_bucket,
_on_dispense = function(stack, pos, droppos, dropnode, dropdir) _on_dispense = function(stack, pos, droppos, dropnode, dropdir)
local buildable = registered_nodes[dropnode.name].buildable_to or dropnode.name == "mcl_portals:portal" local buildable = registered_nodes[dropnode.name].buildable_to or dropnode.name == "mcl_portals:portal"
if not buildable then return stack end if not buildable then return stack end
@ -370,6 +305,8 @@ minetest.register_craftitem("mcl_buckets:bucket_empty", {
_tt_help = S("Collects liquids"), _tt_help = S("Collects liquids"),
inventory_image = "bucket.png", inventory_image = "bucket.png",
stack_max = 16, stack_max = 16,
liquids_pointable = true,
on_place = on_place_bucket_empty,
_on_dispense = function(stack, pos, droppos, dropnode, dropdir) _on_dispense = function(stack, pos, droppos, dropnode, dropdir)
-- Fill empty bucket with liquid or drop bucket if no liquid -- Fill empty bucket with liquid or drop bucket if no liquid
local collect_liquid = false local collect_liquid = false