Merge remote-tracking branch 'upstream/master' into nalc-1.2-dev

- Place à nouveau la barre au dessus de la tête du joueur.

.github/workflows/build.yml

@@ -1,20 +0,0 @@
-name: build
-on: [push, pull_request]
-
-jobs:
-  luacheck:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v3
-    - name: Install dependencies
-      run: |
-        sudo apt-get update -qq
-        sudo apt-get install -qqq luarocks
-    - name: Install LuaCheck and pre-commit
-      run: |
-        pip3 install pre-commit
-        luarocks install --local luacheck
-    - name: Run LuaCheck with pre-commit
-      run: |
-        export PATH="$HOME/.luarocks/bin:$PATH"
-        pre-commit run --all-files

.luacheckrc

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

.travis.yml

@@ -0,0 +1,16 @@
+dist: bionic
+language: python
+
+python:
+  - 3.7.1
+
+install:
+  - sudo apt-get update -qq
+  - sudo apt-get install -qqq luarocks
+  - pip3 install pre-commit
+  - luarocks install --local luacheck
+
+script:
+  # All linters are run with pre-commit hooks
+  - export PATH="$HOME/.luarocks/bin:$PATH"
+  - pre-commit run --all-files

CHANGELOG.md

@@ -7,25 +7,6 @@ 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
-
-- Support for Minetest 5.3 and improved Minetest version definitions.
-
-### Changed
-
-- Gauge entities are now removed when the mod is disabled or when the server switches to creative mode.
-
 ## [1.0.3] - 2020-02-15
 
 ### Fixed
@@ -56,8 +37,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 - Initial versioned release.
 
-[Unreleased]: https://github.com/minetest-mods/gauges/compare/v1.0.4...HEAD
-[1.0.4]: https://github.com/minetest-mods/gauges/compare/v1.0.3...v1.0.4
+[Unreleased]: https://github.com/minetest-mods/gauges/compare/v1.0.3...HEAD
 [1.0.3]: https://github.com/minetest-mods/gauges/compare/v1.0.2...v1.0.3
 [1.0.2]: https://github.com/minetest-mods/gauges/compare/v1.0.1...v1.0.2
 [1.0.1]: https://github.com/minetest-mods/gauges/compare/v1.0.0...v1.0.1

init.lua

@@ -3,61 +3,43 @@
 -- 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
+if minetest.settings:get_bool("health_bars") == false or
+		not minetest.settings:get_bool("enable_damage")
+then return end
 
-local enabled = minetest.settings:get_bool("health_bars") ~= false
-if enabled then
-	enabled = minetest.settings:get_bool("enable_damage")
-end
-
--- Localize this functions for better performance,
+-- Localize the vector distance function for better performance,
 -- as it's called on every step
 local vector_distance = vector.distance
-local max = {
-	breath = 11,
-	hp = 20,
-}
-
-local mt_5 = minetest.features.object_independent_selectionbox
 
 local function add_gauge(player)
 	if player and player:is_player() then
 		local entity = minetest.add_entity(player:get_pos(), "gauges:hp_bar")
+		local height = 19
 
-		-- Check Minetest version and set required entity heigh
-		-- (The entity height offset was changed in Minetest 5.0.0)
-		local height = mt_5 and 19 or 9
+		-- Check for Minetest 0.4.17 and adjust the entity height if needed
+		-- (The entity height offset was changed in Minetest 5.0.0.)
+		local version = tonumber(minetest.get_version().string:sub(1, 1))
+		if version and version < 5 then
+			height = 9
+		end
 
 		entity:set_attach(player, "", {x=0, y=height, z=0}, {x=0, y=0, z=0})
 		entity:get_luaentity().wielder = player
 	end
 end
 
--- credit:
--- https://github.com/minetest/minetest/blob/master/builtin/game/statbars.lua#L31-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
 		local gauge = self.object
 
-		if not enabled or
-				not player or not player:is_player() then
+		if not player or not player:is_player() then
 			gauge:remove()
 			return
 		elseif vector_distance(player:get_pos(), gauge:get_pos()) > 3 then
@@ -66,23 +48,15 @@ minetest.register_entity("gauges:hp_bar", {
 			return
 		end
 
-		local hp = scaleToDefault(player, "hp")
-		local breath = scaleToDefault(player, "breath")
+		local hp = player:get_hp() <= 20 and player:get_hp() or 20
+		local breath = player:get_breath() <= 10 and player:get_breath() or 11
 
 		if self.hp ~= hp or self.breath ~= breath then
-			local health_t = "health_"..hp..".png"
-			local breath_t = "breath_"..breath..".png"
-
-			if hp == 0 then
-				health_t = "blank.png"
-			end
-
-			if breath == max.breath then
-				breath_t = "blank.png"
-			end
-
 			gauge:set_properties({
-				textures = {health_t.."^"..breath_t}
+				textures = {
+					"health_"..hp..".png^"..
+					"breath_"..breath..".png"
+				}
 			})
 			self.hp = hp
 			self.breath = breath
@@ -90,8 +64,8 @@ minetest.register_entity("gauges:hp_bar", {
 	end
 })
 
-if enabled then
-	minetest.register_on_joinplayer(function(player)
-		minetest.after(1, add_gauge, player)
-	end)
-end
+minetest.register_on_joinplayer(function(player)
+	minetest.after(1, add_gauge, player)
+end)
+
+minetest.log("action", "[gauges] loaded.")