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

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

CHANGELOG.md

@@ -7,36 +7,6 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
-## [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
-
-- Fix an issue with gauges occasionally no longer appearing above a player.
-
-### Changed
-
-- Improved the reliability of the Minetest version check (for compatibility with 0.4.17).
-
-## [1.0.2] - 2020-01-16
-
-### Fixed
-
-- Fix crash when creating an entity if a player leaves the server quickly.
-
-### Changed
-
-- Restore Minetest 0.4.17 support. The version check works only with `minetest_game` with the `player_api` mod added in Minetest 5.0.
-
 ## [1.0.1] - 2020-01-01
 
 ### Changed
@@ -47,8 +17,5 @@ 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
-[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
+[Unreleased]: https://github.com/minetest-mods/gauges/compare/v1.0.1...HEAD
 [1.0.1]: https://github.com/minetest-mods/gauges/compare/v1.0.0...v1.0.1

README.md

@@ -60,8 +60,9 @@ on that line.
 
 ## Version compatibility
 
-Gauges currently supports Minetest 5.0 and later, as well as 0.4.17.
-Issues arising in versions older than 5.0 will generally not be fixed.
+Gauges is currently primarily tested with Minetest 5.1.0. It may or may not work
+with newer or older versions. Issues arising in versions older than 5.0.0
+will generally not be fixed.
 
 ## License
 

init.lua

@@ -3,70 +3,42 @@
 -- Copyright © 2014-2020 4aiman, Hugo Locurcio and contributors - MIT License
 -- See `LICENSE.md` included in the source distribution for details.
 
-local enabled = minetest.settings:get_bool("health_bars") ~= false
-if enabled then
-	enabled = minetest.settings:get_bool("enable_damage")
-end
+if minetest.settings:get_bool("health_bars") == false or
+		not minetest.settings:get_bool("enable_damage")
+then return end
 
--- Localize this functions for better performance,
--- as it's called on every step
+-- Localize the vector distance function 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 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")
-
-		-- 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
-
-		entity:set_attach(player, "", {x=0, y=height, z=0}, {x=0, y=0, z=0})
-		entity:get_luaentity().wielder = player
-	end
-end
 
 minetest.register_entity("gauges:hp_bar", {
 	visual = "sprite",
 	visual_size = {x=1, y=1/16, z=1},
+	-- The texture is changed later in the code
 	textures = {"blank.png"},
 	collisionbox = {0},
 	physical = 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
-			gauge:remove()
-			return
-		elseif vector_distance(player:get_pos(), gauge:get_pos()) > 3 then
-			gauge:remove()
-			add_gauge(player)
+		if not player or
+				not minetest.is_player(player) or
+				vector_distance(player:get_pos(), self.object:get_pos()) > 3
+		then
+			self.object:remove()
 			return
 		end
 
-		local hp = min(player:get_hp(), 20)
-		local breath = min(player:get_breath(), max_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}
+			self.object:set_properties({
+				textures = {
+					"health_"..tostring(hp)..".png^"..
+					"breath_"..tostring(breath)..".png"
+				}
 			})
 			self.hp = hp
 			self.breath = breath
@@ -74,8 +46,15 @@ minetest.register_entity("gauges:hp_bar", {
 	end
 })
 
-if enabled then
-	minetest.register_on_joinplayer(function(player)
-		minetest.after(1, add_gauge, player)
-	end)
+local function add_gauge(player)
+	local entity = minetest.add_entity(player:get_pos(), "gauges:hp_bar")
+
+	entity:set_attach(player, "", {x=0, y=19, z=0}, {x=0, y=0, z=0})
+	entity:get_luaentity().wielder = player
 end
+
+minetest.register_on_joinplayer(function(player)
+	minetest.after(1, add_gauge, player)
+end)
+
+minetest.log("action", "[gauges] loaded.")

mod.conf

@@ -1,3 +1,2 @@
 name = gauges
 description = Adds health and breath gauges above players.
-min_minetest_version = 0.4.17