Allow mod namespace for door registrations.

This is an adapted version of #861 - by oleastre

Most mods had been calling `doors.register_door() with a door
name that included the "modname:" prefix, and we should continue
to allow mods to do so, without registering the nodenames created
in the "doors:" namespace.

The default case is to use the "modname:" prefix verbatim. If
mods or code calls this function without a prefix, then "doors:"
is automatically used.

Now that the namespace is corrected, the copy replacement ABM is
no longer needed.
This commit is contained in:
Auke Kok 2016-02-20 20:03:09 -08:00 committed by paramat
parent 777088481b
commit 2c355a8642
1 changed files with 21 additions and 46 deletions

View File

@ -119,7 +119,7 @@ function _doors.door_toggle(pos, clicker)
local meta = minetest.get_meta(pos) local meta = minetest.get_meta(pos)
local state = meta:get_int("state") local state = meta:get_int("state")
local def = minetest.registered_nodes[minetest.get_node(pos).name] local def = minetest.registered_nodes[minetest.get_node(pos).name]
local name = def.door.basename local name = def.door.name
if clicker then if clicker then
local owner = meta:get_string("doors_owner") local owner = meta:get_string("doors_owner")
@ -146,7 +146,7 @@ function _doors.door_toggle(pos, clicker)
end end
minetest.swap_node(pos, { minetest.swap_node(pos, {
name = "doors:" .. name .. transform[state + 1][dir+1].v, name = name .. transform[state + 1][dir+1].v,
param2 = transform[state + 1][dir+1].param2 param2 = transform[state + 1][dir+1].param2
}) })
meta:set_int("state", state) meta:set_int("state", state)
@ -174,9 +174,13 @@ local function on_place_node(place_to, newnode, placer, oldnode, itemstack, poin
end end
function doors.register(name, def) function doors.register(name, def)
if not name:find(":") then
name = "doors:" .. name
end
-- replace old doors of this type automatically -- replace old doors of this type automatically
minetest.register_abm({ minetest.register_abm({
nodenames = {"doors:"..name.."_b_1", "doors:"..name.."_b_2"}, nodenames = {name.."_b_1", name.."_b_2"},
interval = 7.0, interval = 7.0,
chance = 1, chance = 1,
action = function(pos, node, active_object_count, active_object_count_wider) action = function(pos, node, active_object_count, active_object_count_wider)
@ -190,14 +194,14 @@ function doors.register(name, def)
} }
local new = replace[l][h] local new = replace[l][h]
-- retain infotext and doors_owner fields -- retain infotext and doors_owner fields
minetest.swap_node(pos, { name = "doors:" .. name .. "_" .. new.type, param2 = p2}) minetest.swap_node(pos, {name = name .. "_" .. new.type, param2 = p2})
meta:set_int("state", new.state) meta:set_int("state", new.state)
-- wipe meta on top node as it's unused -- wipe meta on top node as it's unused
minetest.set_node({x = pos.x, y = pos.y + 1, z = pos.z}, { name = "doors:hidden" }) minetest.set_node({x = pos.x, y = pos.y + 1, z = pos.z}, { name = "doors:hidden" })
end end
}) })
minetest.register_craftitem(":doors:" .. name, { minetest.register_craftitem(":" .. name, {
description = def.description, description = def.description,
inventory_image = def.inventory_image, inventory_image = def.inventory_image,
@ -249,9 +253,9 @@ function doors.register(name, def)
local state = 0 local state = 0
if minetest.get_item_group(minetest.get_node(aside).name, "door") == 1 then if minetest.get_item_group(minetest.get_node(aside).name, "door") == 1 then
state = state + 2 state = state + 2
minetest.set_node(pos, {name = "doors:" .. name .. "_b", param2 = dir}) minetest.set_node(pos, {name = name .. "_b", param2 = dir})
else else
minetest.set_node(pos, {name = "doors:" .. name .. "_a", param2 = dir}) minetest.set_node(pos, {name = name .. "_a", param2 = dir})
end end
minetest.set_node(above, { name = "doors:hidden" }) minetest.set_node(above, { name = "doors:hidden" })
@ -296,9 +300,9 @@ function doors.register(name, def)
def.groups.not_in_creative_inventory = 1 def.groups.not_in_creative_inventory = 1
def.groups.door = 1 def.groups.door = 1
def.drop = "doors:" .. name def.drop = name
def.door = { def.door = {
basename = name, name = name,
sounds = { def.sound_close, def.sound_open }, sounds = { def.sound_close, def.sound_open },
} }
@ -321,12 +325,12 @@ function doors.register(name, def)
def.on_blast = function(pos, intensity) def.on_blast = function(pos, intensity)
minetest.remove_node(pos) minetest.remove_node(pos)
-- hidden node doesn't get blasted away. -- hidden node doesn't get blasted away.
minetest.remove_node({ x = pos.x, y = pos.y + 1, z = pos.z}) minetest.remove_node({x = pos.x, y = pos.y + 1, z = pos.z})
return { "doors:" .. name } return {name}
end end
end end
minetest.register_node(":doors:" .. name .. "_a", { minetest.register_node(":" .. name .. "_a", {
description = def.description, description = def.description,
visual = "mesh", visual = "mesh",
mesh = "door_a.obj", mesh = "door_a.obj",
@ -358,7 +362,7 @@ function doors.register(name, def)
}, },
}) })
minetest.register_node(":doors:" .. name .. "_b", { minetest.register_node(":" .. name .. "_b", {
description = def.description, description = def.description,
visual = "mesh", visual = "mesh",
mesh = "door_b.obj", mesh = "door_b.obj",
@ -392,13 +396,13 @@ function doors.register(name, def)
if def.recipe then if def.recipe then
minetest.register_craft({ minetest.register_craft({
output = "doors:" .. name, output = name,
recipe = def.recipe, recipe = def.recipe,
}) })
end end
_doors.registered_doors["doors:" .. name .. "_a"] = true _doors.registered_doors[name .. "_a"] = true
_doors.registered_doors["doors:" .. name .. "_b"] = true _doors.registered_doors[name .. "_b"] = true
end end
doors.register("door_wood", { doors.register("door_wood", {
@ -461,7 +465,6 @@ function doors.register_door(name, def)
local i = name:find(":") local i = name:find(":")
local modname = name:sub(1, i - 1) local modname = name:sub(1, i - 1)
local doorname = modname .. "_" .. name:sub(i + 1, -1)
if not def.tiles then if not def.tiles then
if def.protected then if def.protected then
def.tiles = {{name = "doors_door_steel.png", backface_culling = true}} def.tiles = {{name = "doors_door_steel.png", backface_culling = true}}
@ -474,35 +477,7 @@ function doors.register_door(name, def)
"will be used instead.") "will be used instead.")
end end
doors.register(doorname, def) doors.register(name, def)
-- these help replace items present and conversion
minetest.register_alias(name, "doors:" .. doorname)
minetest.register_alias(name .. "_t_1", "doors:hidden")
minetest.register_alias(name .. "_t_2", "doors:hidden")
minetest.register_abm({
nodenames = {name .. "_b_1", name .. "_b_2"},
interval = 7.0,
chance = 1,
action = function(pos, node, active_object_count, active_object_count_wider)
local l = tonumber(node.name:sub(-1))
local meta = minetest.get_meta(pos)
local h = meta:get_int("right") + 1
local p2 = node.param2
local replace = {
{{type = "a", state = 0}, {type = "a", state = 3}},
{{type = "b", state = 1}, {type = "b", state = 2}}
}
local new = replace[l][h]
-- retain infotext and doors_owner fields
minetest.swap_node(pos, {name = "doors:" .. doorname .. "_" .. new.type, param2 = p2})
meta:set_int("state", new.state)
-- wipe meta on top node as it's unused
minetest.set_node({x = pos.sx, y = pos.y + 1, z = pos.z}, {name = "doors:hidden"})
end
})
end end
----trapdoor---- ----trapdoor----