diff --git a/builtin/game/constants.lua b/builtin/game/constants.lua index 50c515b24..29eeb8330 100644 --- a/builtin/game/constants.lua +++ b/builtin/game/constants.lua @@ -21,6 +21,10 @@ core.EMERGE_GENERATED = 4 -- constants.h -- Size of mapblocks in nodes core.MAP_BLOCKSIZE = 16 +-- Default maximal HP of a player +core.PLAYER_MAX_HP_DEFAULT = 20 +-- Maximal breath of a player +core.PLAYER_MAX_BREATH = 11 -- light.h -- Maximum value for node 'light_source' parameter diff --git a/builtin/game/statbars.lua b/builtin/game/statbars.lua index 6aa106140..1a59a89e4 100644 --- a/builtin/game/statbars.lua +++ b/builtin/game/statbars.lua @@ -6,7 +6,7 @@ local health_bar_definition = hud_elem_type = "statbar", position = { x=0.5, y=1 }, text = "heart.png", - number = 20, + number = core.PLAYER_MAX_HP_DEFAULT, direction = 0, size = { x=24, y=24 }, offset = { x=(-10*24)-25, y=-(48+24+16)}, @@ -17,7 +17,7 @@ local breath_bar_definition = hud_elem_type = "statbar", position = { x=0.5, y=1 }, text = "bubble.png", - number = 20, + number = core.PLAYER_MAX_BREATH, direction = 0, size = { x=24, y=24 }, offset = {x=25,y=-(48+24+16)}, @@ -37,39 +37,45 @@ local function initialize_builtin_statbars(player) return end - if (hud_ids[name] == nil) then + if not hud_ids[name] then hud_ids[name] = {} -- flags are not transmitted to client on connect, we need to make sure -- our current flags are transmitted by sending them actively player:hud_set_flags(player:hud_get_flags()) end + local hud = hud_ids[name] if player:hud_get_flags().healthbar and enable_damage then - if hud_ids[name].id_healthbar == nil then - health_bar_definition.number = player:get_hp() - hud_ids[name].id_healthbar = player:hud_add(health_bar_definition) + if hud.id_healthbar == nil then + local hp = player:get_hp() + local max_display_hp = math.max(core.PLAYER_MAX_HP_DEFAULT, + math.max(player:get_properties().hp_max, hp)) + -- Limit width of health bar: Scale to the default maximal HP + health_bar_definition.number = + hp / max_display_hp * core.PLAYER_MAX_HP_DEFAULT + hud.id_healthbar = player:hud_add(health_bar_definition) end else - if hud_ids[name].id_healthbar ~= nil then - player:hud_remove(hud_ids[name].id_healthbar) - hud_ids[name].id_healthbar = nil + if hud.id_healthbar ~= nil then + player:hud_remove(hud.id_healthbar) + hud.id_healthbar = nil end end - if (player:get_breath() < 11) then + if player:get_breath() < core.PLAYER_MAX_BREATH then if player:hud_get_flags().breathbar and enable_damage then - if hud_ids[name].id_breathbar == nil then - hud_ids[name].id_breathbar = player:hud_add(breath_bar_definition) + if hud.id_breathbar == nil then + hud.id_breathbar = player:hud_add(breath_bar_definition) end else - if hud_ids[name].id_breathbar ~= nil then - player:hud_remove(hud_ids[name].id_breathbar) - hud_ids[name].id_breathbar = nil + if hud.id_breathbar ~= nil then + player:hud_remove(hud.id_breathbar) + hud.id_breathbar = nil end end - elseif hud_ids[name].id_breathbar ~= nil then - player:hud_remove(hud_ids[name].id_breathbar) - hud_ids[name].id_breathbar = nil + elseif hud.id_breathbar ~= nil then + player:hud_remove(hud.id_breathbar) + hud.id_breathbar = nil end end @@ -101,7 +107,12 @@ local function player_event_handler(player,eventname) initialize_builtin_statbars(player) if hud_ids[name].id_healthbar ~= nil then - player:hud_change(hud_ids[name].id_healthbar,"number",player:get_hp()) + local hp = player:get_hp() + local max_display_hp = math.max(core.PLAYER_MAX_HP_DEFAULT, + math.max(player:get_properties().hp_max, hp)) + -- Limit width of health bar: Scale to the default maximal HP + local hp_count = hp / max_display_hp * core.PLAYER_MAX_HP_DEFAULT + player:hud_change(hud_ids[name].id_healthbar, "number", hp_count) return true end end @@ -110,7 +121,7 @@ local function player_event_handler(player,eventname) initialize_builtin_statbars(player) if hud_ids[name].id_breathbar ~= nil then - player:hud_change(hud_ids[name].id_breathbar,"number",player:get_breath()*2) + player:hud_change(hud_ids[name].id_breathbar, "number", player:get_breath() * 2) return true end end diff --git a/doc/lua_api.txt b/doc/lua_api.txt index 694afd1f6..303d85560 100644 --- a/doc/lua_api.txt +++ b/doc/lua_api.txt @@ -3345,6 +3345,7 @@ This is basically a reference to a C++ `ServerActiveObject` * `0`: player is drowning, * `1`-`10`: remaining number of bubbles * `11`: bubbles bar is not shown + * See constant: `minetest.PLAYER_MAX_BREATH` * `set_attribute(attribute, value)`: * Sets an extra attribute with value on player. * `value` must be a string, or a number which will be converted to a string. @@ -4021,7 +4022,7 @@ Definition tables ### Object Properties { - hp_max = 1, + hp_max = 1, -- For players, the maximal HP defaults to `minetest.PLAYER_MAX_HP_DEFAULT` physical = true, collide_with_objects = true, -- collide with other objects if physical = true weight = 5, diff --git a/src/constants.h b/src/constants.h index fb9e97cb3..6c3e21f47 100644 --- a/src/constants.h +++ b/src/constants.h @@ -88,7 +88,7 @@ with this program; if not, write to the Free Software Foundation, Inc., #define PLAYER_INVENTORY_SIZE (8 * 4) // Maximum hit points of a player -#define PLAYER_MAX_HP 20 +#define PLAYER_MAX_HP_DEFAULT 20 // Maximal breath of a player #define PLAYER_MAX_BREATH 11 diff --git a/src/content_sao.cpp b/src/content_sao.cpp index c22e341a9..a48ad3f7a 100644 --- a/src/content_sao.cpp +++ b/src/content_sao.cpp @@ -795,7 +795,7 @@ PlayerSAO::PlayerSAO(ServerEnvironment *env_, RemotePlayer *player_, u16 peer_id { assert(m_peer_id != 0); // pre-condition - m_prop.hp_max = PLAYER_MAX_HP; + m_prop.hp_max = PLAYER_MAX_HP_DEFAULT; m_prop.physical = false; m_prop.weight = 75; m_prop.collisionbox = aabb3f(-0.3f, -1.0f, -0.3f, 0.3f, 0.75f, 0.3f); @@ -811,7 +811,7 @@ PlayerSAO::PlayerSAO(ServerEnvironment *env_, RemotePlayer *player_, u16 peer_id // end of default appearance m_prop.is_visible = true; m_prop.makes_footstep_sound = true; - m_hp = PLAYER_MAX_HP; + m_hp = m_prop.hp_max; } PlayerSAO::~PlayerSAO() @@ -1248,8 +1248,8 @@ void PlayerSAO::setHP(s16 hp) if (hp < 0) hp = 0; - else if (hp > PLAYER_MAX_HP) - hp = PLAYER_MAX_HP; + else if (hp > m_prop.hp_max) + hp = m_prop.hp_max; if (hp < oldhp && !g_settings->getBool("enable_damage")) { return; diff --git a/src/remoteplayer.cpp b/src/remoteplayer.cpp index df9542630..0c4819853 100644 --- a/src/remoteplayer.cpp +++ b/src/remoteplayer.cpp @@ -109,7 +109,7 @@ void RemotePlayer::deSerialize(std::istream &is, const std::string &playername, try { sao->setHPRaw(args.getS32("hp")); } catch(SettingNotFoundException &e) { - sao->setHPRaw(PLAYER_MAX_HP); + sao->setHPRaw(PLAYER_MAX_HP_DEFAULT); } try { diff --git a/src/script/common/c_content.cpp b/src/script/common/c_content.cpp index c0e29abf0..d40138bea 100644 --- a/src/script/common/c_content.cpp +++ b/src/script/common/c_content.cpp @@ -183,7 +183,9 @@ void read_object_properties(lua_State *L, int index, if(!lua_istable(L, index)) return; - prop->hp_max = getintfield_default(L, -1, "hp_max", 10); + int hp_max = 0; + if (getintfield(L, -1, "hp_max", hp_max)) + prop->hp_max = (s16)rangelim(hp_max, 0, S16_MAX); getboolfield(L, -1, "physical", prop->physical); getboolfield(L, -1, "collide_with_objects", prop->collideWithObjects); diff --git a/src/script/lua_api/l_object.cpp b/src/script/lua_api/l_object.cpp index 8905f2d0c..483c10971 100644 --- a/src/script/lua_api/l_object.cpp +++ b/src/script/lua_api/l_object.cpp @@ -750,6 +750,11 @@ int ObjectRef::l_set_properties(lua_State *L) if (!prop) return 0; read_object_properties(L, 2, prop, getServer(L)->idef()); + if (prop->hp_max < co->getHP()) { + co->setHP(prop->hp_max); + if (co->getType() == ACTIVEOBJECT_TYPE_PLAYER) + getServer(L)->SendPlayerHPOrDie((PlayerSAO *)co); + } co->notifyObjectPropertiesModified(); return 0; } diff --git a/src/server.cpp b/src/server.cpp index b07b72d2e..ad7d11b63 100644 --- a/src/server.cpp +++ b/src/server.cpp @@ -2683,7 +2683,7 @@ void Server::RespawnPlayer(u16 peer_id) << playersao->getPlayer()->getName() << " respawns" << std::endl; - playersao->setHP(PLAYER_MAX_HP); + playersao->setHP(playersao->accessObjectProperties()->hp_max); playersao->setBreath(PLAYER_MAX_BREATH); bool repositioned = m_script->on_respawnplayer(playersao);