Merge branch 'github'

If a player teleports far away, their hp_bar can become deactivated and disappear. The mod installs the gauge only when a player joins, so it will be forever missing. `static_save = false` prevents the item from deactivating https://github.com/minetest/minetest/blob/master/src/serverenvironment.cpp#L2028 .
Also, it doesn't make sense to save `gauges:hp_bar` to map meta since it's not a standalone item.

.luacheckrc

@@ -21,6 +21,11 @@ stds.minetest = {
 				"copy",
 			},
 		},
+		math = {
+			fields = {
+				"round",
+			},
+		}
 	}
 }
 

CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+### Changed
+
+- [Health and breath bars now adapt to custom maximum values instead of being hardcoded to 20 and 11 respectively.](https://github.com/minetest-mods/gauges/pull/9)
+
+### Fixed
+
+- [The health bar entity is no longer lost when a player teleports.](https://github.com/minetest-mods/gauges/pull/10)
+  - A side effect is that health bar entities are no longer saved in map metadata.
+
 ## [1.0.4] - 2020-09-27
 
 ### Added

init.lua

@@ -3,6 +3,10 @@
 -- Copyright © 2014-2020 4aiman, Hugo Locurcio and contributors - MIT License
 -- See `LICENSE.md` included in the source distribution for details.
 
+local function round(v)
+	return math.floor(v + 0.5)
+end
+
 local enabled = minetest.settings:get_bool("health_bars") ~= false
 if enabled then
 	enabled = minetest.settings:get_bool("enable_damage")
@@ -11,8 +15,10 @@ end
 -- Localize this functions for better performance,
 -- as it's called on every step
 local vector_distance = vector.distance
-local min = math.min
-local max_breath = minetest.PLAYER_MAX_BREATH_DEFAULT or 11
+local max = {
+	breath = 11,
+	hp = 20,
+}
 
 local mt_5 = minetest.features.object_independent_selectionbox
 
@@ -29,12 +35,21 @@ local function add_gauge(player)
 	end
 end
 
+-- credit: https://github.com/minetest/minetest/blob/6de8d77e17017cd5cc7b065d42566b6b1cd076cc/builtin/game/statbars.lua#L30-L37
+local function scaleToDefault(player, field)
+	-- Scale "hp" or "breath" to supported amount
+	local current = player["get_" .. field](player)
+	local max_display = math.max(player:get_properties()[field .. "_max"], current)
+	return round(current / max_display * max[field])
+end
+
 minetest.register_entity("gauges:hp_bar", {
 	visual = "sprite",
 	visual_size = {x=1, y=1/16, z=1},
 	textures = {"blank.png"},
 	collisionbox = {0},
 	physical = false,
+	static_save = false,
 
 	on_step = function(self)
 		local player = self.wielder
@@ -50,8 +65,8 @@ minetest.register_entity("gauges:hp_bar", {
 			return
 		end
 
-		local hp = min(player:get_hp(), 20)
-		local breath = min(player:get_breath(), max_breath)
+		local hp = scaleToDefault(player, "hp")
+		local breath = scaleToDefault(player, "breath")
 
 		if self.hp ~= hp or self.breath ~= breath then
 			local health_t = "health_"..hp..".png"
@@ -61,7 +76,7 @@ minetest.register_entity("gauges:hp_bar", {
 				health_t = "blank.png"
 			end
 
-			if breath == max_breath then
+			if breath == max.breath then
 				breath_t = "blank.png"
 			end
 
@@ -79,3 +94,5 @@ if enabled then
 		minetest.after(1, add_gauge, player)
 	end)
 end
+
+minetest.log("action", "[gauges] loaded.")