mtcontrib/pipeworks
1
0
Fork 2
mirror of https://github.com/mt-mods/pipeworks.git synced 2024-12-28 03:40:25 +01:00
Code Issues Releases Wiki Activity

patch for #124 (#125) async race condition fix

Browse Source 
* restructure avoiding excessive indentation

* possibly fix #124

* async race condition fix
This commit is contained in:
Luke aka SwissalpS 2024-05-13 12:00:17 +02:00 committed by GitHub
parent 1169cff163
commit 94442e87bb
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 32 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
compat-chests.lua
View File

@ -47,21 +47,27 @@ if minetest.get_modpath("default") then
-- get the fields from the chest formspec, we can do this bc. newest functions are called first -- get the fields from the chest formspec, we can do this bc. newest functions are called first
-- https://github.com/minetest/minetest/blob/d4b10db998ebeb689b3d27368e30952a42169d03/doc/lua_api.md?plain=1#L5840 -- https://github.com/minetest/minetest/blob/d4b10db998ebeb689b3d27368e30952a42169d03/doc/lua_api.md?plain=1#L5840
minetest.register_on_player_receive_fields(function(player, formname, fields) minetest.register_on_player_receive_fields(function(player, formname, fields)
if formname == "default:chest" then if fields.quit or formname ~= "default:chest" then
return
end
local pn = player:get_player_name() local pn = player:get_player_name()
local pos = default.chest.open_chests[pn].pos local chest_open = default.chest.open_chests[pn]
if not chest_open then
-- chest already closed before formspec
return
end
local pos = chest_open.pos
local chest = pos and minetest.get_node(pos) local chest = pos and minetest.get_node(pos)
local is_pipeworks_chest = chest and pipeworks.chests[chest] local is_pipeworks_chest = chest and pipeworks.chests[chest]
if is_pipeworks_chest and not fields.quit and pipeworks.may_configure(pos, player) then if is_pipeworks_chest and pipeworks.may_configure(pos, player) then
-- Pipeworks Switch -- Pipeworks Switch
fs_helpers.on_receive_fields(pos, fields) fs_helpers.on_receive_fields(pos, fields)
minetest.show_formspec(player:get_player_name(), minetest.show_formspec(pn,
"default:chest", "default:chest",
default.chest.get_chest_formspec(pos)) default.chest.get_chest_formspec(pos))
end end
-- Do NOT return true here, the callback from default still needs to run -- Do NOT return true here, the callback from default still needs to run
return false return false
end
end) end)
local connect_sides = {left = 1, right = 1, back = 1, bottom = 1, top = 1} local connect_sides = {left = 1, right = 1, back = 1, bottom = 1, top = 1}
@ -152,17 +158,18 @@ elseif minetest.get_modpath("hades_chests") then
-- get the fields from the chest formspec, we can do this bc. newest functions are called first -- get the fields from the chest formspec, we can do this bc. newest functions are called first
-- https://github.com/minetest/minetest/blob/d4b10db998ebeb689b3d27368e30952a42169d03/doc/lua_api.md?plain=1#L5840 -- https://github.com/minetest/minetest/blob/d4b10db998ebeb689b3d27368e30952a42169d03/doc/lua_api.md?plain=1#L5840
minetest.register_on_player_receive_fields(function(player, formname, fields) minetest.register_on_player_receive_fields(function(player, formname, fields)
if formname == "hades_chests:chest_locked" then if fields.quit or formname ~= "hades_chests:chest_locked" then
return
end
local pn = player:get_player_name() local pn = player:get_player_name()
local pos = open_chests[pn] local pos = open_chests[pn]
if not fields.quit and pos and pipeworks.may_configure(pos, player) then if pos and pipeworks.may_configure(pos, player) then
-- Pipeworks Switch -- Pipeworks Switch
fs_helpers.on_receive_fields(pos, fields) fs_helpers.on_receive_fields(pos, fields)
minetest.show_formspec(pn, "hades_chests:chest_locked", get_locked_chest_formspec(pos)) minetest.show_formspec(pn, "hades_chests:chest_locked", get_locked_chest_formspec(pos))
end end
-- Do NOT return true here, the callback from hades still needs to run (if they add one) -- Do NOT return true here, the callback from hades still needs to run (if they add one)
return false return false
end
end) end)
local connect_sides = {left = 1, right = 1, back = 1, bottom = 1, top = 1} local connect_sides = {left = 1, right = 1, back = 1, bottom = 1, top = 1}