Merge branch 'master' of yunohost.local:minetest-mods/gauges into nalc-1.2-dev

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

CHANGELOG.md

@@ -1,43 +0,0 @@
-# Changelog
-
-All notable changes to this project will be documented in this file.
-
-The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
-and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
-
-## [Unreleased]
-
-## [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
-
-- Improve performance by localizing the `vector.distance()` function.
-
-## 1.0.0 - 2020-01-01
-
-- Initial versioned release.
-
-[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
-[1.0.2]: https://github.com/minetest-mods/gauges/compare/v1.0.1...v1.0.2
-[1.0.3]: https://github.com/minetest-mods/gauges/compare/v1.0.2...v1.0.3

LICENSE.md

@@ -1,6 +1,6 @@
 # MIT License
 
-Copyright © 2014-2020 4aiman, Hugo Locurcio and contributors
+Copyright © 2014-2019 4aiman, Hugo Locurcio and contributors
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

README.md

@@ -1,7 +1,5 @@
 # Gauges
 
-![Preview](https://content.minetest.net/uploads/GBoKauWSou.png)
-
 Gauges for [Minetest](https://www.minetest.net/), a free and open source infinite
 world block sandbox game.
 
@@ -20,7 +18,9 @@ git clone https://github.com/minetest-mods/gauges.git
 
 You can also
 [download a ZIP archive](https://github.com/minetest-mods/gauges/archive/master.zip)
-of Gauges.
+of Gauges. If you do so, you will need to extract the archive then rename
+the resulting folder from `gauges-master` to `gauges` – this is
+**absolutely** required, as the mod won't work otherwise.
 
 ### Enable the mod
 
@@ -60,12 +60,11 @@ 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 requires Minetest 5.0.0 or newer to work as expected.
 
 ## License
 
-Copyright © 2014-2020 4aiman, Hugo Locurcio and contributors
+Copyright © 2014-2019 4aiman, Hugo Locurcio and contributors
 
 - Code is licensed under the MIT license, see
   [`LICENSE.md`](LICENSE.md) for details.

init.lua

@@ -1,69 +1,66 @@
 -- gauges: Adds health/breath bars above players
 --
--- Copyright © 2014-2020 4aiman, Hugo Locurcio and contributors - MIT License
+-- Copyright © 2014-2019 4aiman, Hugo Locurcio and contributors - MIT License
 -- See `LICENSE.md` included in the source distribution for details.
 
-if minetest.settings:get_bool("health_bars") == false or
-		not minetest.settings:get_bool("enable_damage")
-then return end
+local hp_bar = {
+	physical = false,
+	collisionbox = {x = 0, y = 0, z = 0},
+	visual = "sprite",
+	textures = {"20.png"}, -- The texture is changed later in the code
+	visual_size = {x = 1.5, y = 0.09375, z = 1.5}, -- Y value is (1 / 16) * 1.5
+	wielder = nil,
+}
 
--- Localize the vector distance function for better performance,
--- as it's called on every step
-local vector_distance = vector.distance
+function vector.sqdist(a, b)
+	local dx = a.x - b.x
+	local dy = a.y - b.y
+	local dz = a.z - b.z
+	return dx * dx + dy * dy + dz * dz
+end
 
-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
+function hp_bar:on_step(dtime)
+	local wielder = self.wielder and minetest.get_player_by_name(self.wielder)
 
-		-- 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
+	if
+		wielder == nil or
+		vector.sqdist(wielder:get_pos(), self.object:get_pos()) > 3
+	then
+		self.object:remove()
+		return
+	end
 
-		entity:set_attach(player, "", {x=0, y=height, z=0}, {x=0, y=0, z=0})
-		entity:get_luaentity().wielder = player
+	local hp = wielder:get_hp()
+	local breath = wielder:get_breath()
+
+	self.object:set_properties({
+		textures = {
+			"health_" .. tostring(hp) .. ".png^breath_" .. tostring(breath) .. ".png",
+		},
+	})
+end
+
+minetest.register_entity("gauges:hp_bar", hp_bar)
+
+local function add_HP_gauge(name)
+	local player = minetest.get_player_by_name(name)
+	local pos = player:get_pos()
+	local ent = minetest.add_entity(pos, "gauges:hp_bar")
+
+	if ent ~= nil then
+		ent:set_attach(player, "", {x = 0, y = 19, z = 0}, {x = 0, y = 0, z = 0})
+		ent = ent:get_luaentity()
+		ent.wielder = player:get_player_name()
 	end
 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,
+if
+	minetest.settings:get_bool("enable_damage") and
+	minetest.settings:get_bool("health_bars") ~= false
+then
+	minetest.register_on_joinplayer(function(player)
+		minetest.after(1, add_HP_gauge, player:get_player_name())
+	end)
+end
 
-	on_step = function(self)
-		local player = self.wielder
-		local gauge = self.object
-
-		if 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)
-			return
-		end
-
-		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
-			gauge:set_properties({
-				textures = {
-					"health_"..hp..".png^"..
-					"breath_"..breath..".png"
-				}
-			})
-			self.hp = hp
-			self.breath = breath
-		end
-	end
-})
-
-minetest.register_on_joinplayer(function(player)
-	minetest.after(1, add_gauge, player)
-end)
+minetest.log("action", "[gauges] loaded.")