Fixed a minor bug introduced when I created openfarming, which skips the faction test if an area is openfarming.

And not return false if not a player(ie:mobs api) or unknown node, continue iterate on others areas(minor bug, generaly if 1 area it's protected).
This commit is contained in:
Crabman77 2022-07-09 02:49:12 +02:00
parent 76f8ccf8ab
commit a8cacee8cc

12
api.lua
View File

@ -110,12 +110,12 @@ function areas:canInteract(pos, name)
for _, area in pairs(self:getAreasAtPos(pos)) do for _, area in pairs(self:getAreasAtPos(pos)) do
if area.owner == name or area.open then if area.owner == name or area.open then
return true return true
elseif area.openfarming then end
if area.openfarming then
-- if area is openfarming -- if area is openfarming
local node = minetest.get_node(pos).name
if not minetest.registered_nodes[node] then return false end
local player = minetest.get_player_by_name(name) local player = minetest.get_player_by_name(name)
if not player then return false end local node = minetest.get_node(pos).name
if player and minetest.registered_nodes[node] then
local wstack = player:get_wielded_item():get_name() local wstack = player:get_wielded_item():get_name()
if wstack == "" then wstack = "hand" end if wstack == "" then wstack = "hand" end
@ -128,7 +128,9 @@ function areas:canInteract(pos, name)
if plants[wstack] ~= nil and plants[wstack] == node then if plants[wstack] ~= nil and plants[wstack] == node then
return true return true
end end
elseif areas.factions_available and area.faction_open then end
end
if areas.factions_available and area.faction_open then
if (factions.version or 0) < 2 then if (factions.version or 0) < 2 then
local faction_name = factions.get_player_faction(name) local faction_name = factions.get_player_faction(name)
if faction_name then if faction_name then