Replace globalstep on playerstep, hide Areas text on free areas

hud.lua

@@ -1,67 +1,60 @@
 -- This is inspired by the landrush mod by Bremaweb
 areas.hud = {}
-areas.hud.refresh = 0
 
-minetest.register_globalstep(function(dtime)
+minetest.register_playerstep(function(dtime, playernames)
+	for _, name in pairs(playernames) do
+		local player = minetest.get_player_by_name(name)
+		if player and player:is_player() then
+			local pos = vector.round(player:get_pos())
+			pos = vector.apply(pos, function(p)
+				return math.max(math.min(p, 2147483), -2147483)
+			end)
+			local areaStrings = {}
 
-	areas.hud.refresh = areas.hud.refresh + dtime
+			for id, area in pairs(areas:getAreasAtPos(pos)) do
-	if areas.hud.refresh > areas.config["tick"] then
+				table.insert(areaStrings, ("%s [%u] (%s%s)")
-		areas.hud.refresh = 0
+						:format(area.name, id, area.owner,
-	else
+						area.open and ":open" or ""))
-		return
+			end
-	end
 
-	for _, player in pairs(minetest.get_connected_players()) do
+			for i, area in pairs(areas:getExternalHudEntries(pos)) do
-		local name = player:get_player_name()
+				local str = ""
-		local pos = vector.round(player:get_pos())
+				if area.name then str = area.name .. " " end
-		pos = vector.apply(pos, function(p)
+				if area.id then str = str.."["..area.id.."] " end
-			return math.max(math.min(p, 2147483), -2147483)
+				if area.owner then str = str.."("..area.owner..")" end
-		end)
+				table.insert(areaStrings, str)
-		local areaStrings = {}
+			end
 
-		for id, area in pairs(areas:getAreasAtPos(pos)) do
+			local areaString = ""
-			table.insert(areaStrings, ("%s [%u] (%s%s)")
+			if #areaStrings > 0 then
-					:format(area.name, id, area.owner,
+				areaString = "Areas:"
-					area.open and ":open" or ""))
+				areaString = areaString.."\n"..
-		end
+					table.concat(areaStrings, "\n")
-
+			end
-		for i, area in pairs(areas:getExternalHudEntries(pos)) do
+			local hud = areas.hud[name]
-			local str = ""
+			if not hud then
-			if area.name then str = area.name .. " " end
+				hud = {}
-			if area.id then str = str.."["..area.id.."] " end
+				areas.hud[name] = hud
-			if area.owner then str = str.."("..area.owner..")" end
+				hud.areasId = player:hud_add({
-			table.insert(areaStrings, str)
+					hud_elem_type = "text",
-		end
+					name = "Areas",
-
+					number = 0xFFFFFF,
-		local areaString = "Areas:"
+					position = {x=0, y=1},
-		if #areaStrings > 0 then
+					offset = {x=8, y=-8},
-			areaString = areaString.."\n"..
+					text = "",
-				table.concat(areaStrings, "\n")
+					scale = {x=200, y=60},
-		end
+					alignment = {x=1, y=-1},
-		local hud = areas.hud[name]
+				})
-		if not hud then
+				hud.oldAreas = areaString
-			hud = {}
+				return
-			areas.hud[name] = hud
+			elseif hud.oldAreas ~= areaString then
-			hud.areasId = player:hud_add({
+				player:hud_change(hud.areasId, "text", areaString)
-				hud_elem_type = "text",
+				hud.oldAreas = areaString
-				name = "Areas",
+			end
-				number = 0xFFFFFF,
-				position = {x=0, y=1},
-				offset = {x=8, y=-8},
-				text = areaString,
-				scale = {x=200, y=60},
-				alignment = {x=1, y=-1},
-			})
-			hud.oldAreas = areaString
-			return
-		elseif hud.oldAreas ~= areaString then
-			player:hud_change(hud.areasId, "text", areaString)
-			hud.oldAreas = areaString
 		end
 	end
-end)
+end, false)
 
 minetest.register_on_leaveplayer(function(player)
 	areas.hud[player:get_player_name()] = nil
 end)
-