Merge commit '053c30b'

3d_armor/init.lua

@@ -99,17 +99,7 @@ for _, mod in pairs(skin_mods) do
 		armor.set_skin_mod(mod)
 	end
 end
-if not minetest.get_modpath("moreores") then
-	armor.materials.mithril = nil
-end
-if not minetest.get_modpath("ethereal") then
-	armor.materials.crystal = nil
-end
-if not minetest.get_modpath("nether") then
-	armor.materials.nether = nil
-end
 
-dofile(modpath.."/armor.lua")
 
 -- Armor Initialization
 
@@ -313,6 +303,10 @@ player_api.register_model("3d_armor_character.b3d", {
 		mine = {x=189, y=198},
 		walk_mine = {x=200, y=219},
 		sit = {x=81, y=160},
+		-- compatibility w/ the emote mod
+		wave = {x = 192, y = 196, override_local = true},
+		point = {x = 196, y = 196, override_local = true},
+		freeze = {x = 205, y = 205, override_local = true},
 	},
 })
 
@@ -420,21 +414,29 @@ if armor.config.punch_damage == true then
 end
 
 minetest.register_on_player_hpchange(function(player, hp_change, reason)
-	if player and reason.type ~= "drown" and reason.hunger == nil
-			and hp_change < 0 then
-		local name = player:get_player_name()
-		if name then
-			local heal = armor.def[name].heal
-			if heal >= math.random(100) then
-				hp_change = 0
-			end
-			-- check if armor damage was handled by fire or on_punchplayer
-			local time = last_punch_time[name] or 0
-			if time == 0 or time + 1 < minetest.get_gametime() then
-				armor:punch(player)
-			end
+	if not minetest.is_player(player) then
+		return hp_change
+	end
+
+	if reason.type == "drown" or reason.hunger or hp_change >= 0 then
+		return hp_change
+	end
+
+	local name = player:get_player_name()
+	local properties = player:get_properties()
+	local hp = player:get_hp()
+	if hp + hp_change < properties.hp_max then
+		local heal = armor.def[name].heal
+		if heal >= math.random(100) then
+			hp_change = 0
+		end
+		-- check if armor damage was handled by fire or on_punchplayer
+		local time = last_punch_time[name] or 0
+		if time == 0 or time + 1 < minetest.get_gametime() then
+			armor:punch(player)
 		end
 	end
+
 	return hp_change
 end, true)
 
@@ -463,7 +465,7 @@ minetest.register_globalstep(function(dtime)
 		local remove = init_player_armor(player) == true
 		pending_players[player] = count + 1
 		if remove == false and count > armor.config.init_times then
-			minetest.log("warning", S("3d_armor: Failed to initialize player"))
+			minetest.log("warning", "3d_armor: Failed to initialize player")
 			remove = true
 		end
 		if remove == true then
@@ -483,22 +485,14 @@ minetest.register_globalstep(function(dtime)
 	end
 end)
 
--- Fire Protection, added by TenPlus1.
 if armor.config.fire_protect == true then
-	-- override any hot nodes that do not already deal damage
-	for _, row in pairs(armor.fire_nodes) do
-		if minetest.registered_nodes[row[1]] then
-			local damage = minetest.registered_nodes[row[1]].damage_per_second
-			if not damage or damage == 0 then
-				minetest.override_item(row[1], {damage_per_second = row[3]})
-			end
-		end
-	end
-else
-	print ("[3d_armor] Fire Nodes disabled")
-end
 
-if armor.config.fire_protect == true then
+	-- make torches hurt
+	minetest.override_item("default:torch", {damage_per_second = 1})
+	minetest.override_item("default:torch_wall", {damage_per_second = 1})
+	minetest.override_item("default:torch_ceiling", {damage_per_second = 1})
+
+	-- check player damage for any hot nodes we may be protected against
 	minetest.register_on_player_hpchange(function(player, hp_change, reason)
 
 		if reason.type == "node_damage" and reason.node then
@@ -507,9 +501,7 @@ if armor.config.fire_protect == true then
 				local name = player:get_player_name()
 				for _,igniter in pairs(armor.fire_nodes) do
 					if reason.node == igniter[1] then
-						if armor.def[name].fire < igniter[2] then
-							armor:punch(player, "fire")
-						else
+						if armor.def[name].fire >= igniter[2] then
 							hp_change = 0
 						end
 					end