]> git.lizzy.rs Git - dragonfireclient.git/blobdiff - src/content_sao.cpp
ContentCAO: Fix broken attachments on join (#8701)
[dragonfireclient.git] / src / content_sao.cpp
index 0e0c19fceb5292cb4393d28bbcd2b8e5dca2b40e..41b1aec18794c65db0bea633ccd167368f4cd3e4 100644 (file)
@@ -30,6 +30,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "genericobject.h"
 #include "settings.h"
 #include <algorithm>
+#include <cmath>
 
 std::map<u16, ServerActiveObject::Factory> ServerActiveObject::m_types;
 
@@ -117,15 +118,14 @@ UnitSAO::UnitSAO(ServerEnvironment *env, v3f pos):
        m_armor_groups["fleshy"] = 100;
 }
 
-bool UnitSAO::isAttached() const
+ServerActiveObject *UnitSAO::getParent() const
 {
        if (!m_attachment_parent_id)
-               return false;
+               return nullptr;
        // Check if the parent still exists
        ServerActiveObject *obj = m_env->getActiveObject(m_attachment_parent_id);
-       if (obj)
-               return true;
-       return false;
+
+       return obj;
 }
 
 void UnitSAO::setArmorGroups(const ItemGroupList &armor_groups)
@@ -186,15 +186,21 @@ void UnitSAO::setAttachment(int parent_id, const std::string &bone, v3f position
        // This breaks some things so we also give the server the most accurate representation
        // even if players only see the client changes.
 
+       int old_parent = m_attachment_parent_id;
        m_attachment_parent_id = parent_id;
        m_attachment_bone = bone;
        m_attachment_position = position;
        m_attachment_rotation = rotation;
        m_attachment_sent = false;
+
+       if (parent_id != old_parent) {
+               onDetach(old_parent);
+               onAttach(parent_id);
+       }
 }
 
 void UnitSAO::getAttachment(int *parent_id, std::string *bone, v3f *position,
-       v3f *rotation)
+       v3f *rotation) const
 {
        *parent_id = m_attachment_parent_id;
        *bone = m_attachment_bone;
@@ -202,6 +208,30 @@ void UnitSAO::getAttachment(int *parent_id, std::string *bone, v3f *position,
        *rotation = m_attachment_rotation;
 }
 
+void UnitSAO::clearChildAttachments()
+{
+       for (int child_id : m_attachment_child_ids) {
+               // Child can be NULL if it was deleted earlier
+               if (ServerActiveObject *child = m_env->getActiveObject(child_id))
+                       child->setAttachment(0, "", v3f(0, 0, 0), v3f(0, 0, 0));
+       }
+       m_attachment_child_ids.clear();
+}
+
+void UnitSAO::clearParentAttachment()
+{
+       ServerActiveObject *parent = nullptr;
+       if (m_attachment_parent_id) {
+               parent = m_env->getActiveObject(m_attachment_parent_id);
+               setAttachment(0, "", m_attachment_position, m_attachment_rotation);
+       } else {
+               setAttachment(0, "", v3f(0, 0, 0), v3f(0, 0, 0));
+       }
+       // Do it
+       if (parent)
+               parent->removeAttachmentChild(m_id);
+}
+
 void UnitSAO::addAttachmentChild(int child_id)
 {
        m_attachment_child_ids.insert(child_id);
@@ -212,11 +242,43 @@ void UnitSAO::removeAttachmentChild(int child_id)
        m_attachment_child_ids.erase(child_id);
 }
 
-const std::unordered_set<int> &UnitSAO::getAttachmentChildIds()
+const std::unordered_set<int> &UnitSAO::getAttachmentChildIds() const
 {
        return m_attachment_child_ids;
 }
 
+void UnitSAO::onAttach(int parent_id)
+{
+       if (!parent_id)
+               return;
+
+       ServerActiveObject *parent = m_env->getActiveObject(parent_id);
+
+       if (!parent || parent->isGone())
+               return; // Do not try to notify soon gone parent
+
+       if (parent->getType() == ACTIVEOBJECT_TYPE_LUAENTITY) {
+               // Call parent's on_attach field
+               m_env->getScriptIface()->luaentity_on_attach_child(parent_id, this);
+       }
+}
+
+void UnitSAO::onDetach(int parent_id)
+{
+       if (!parent_id)
+               return;
+
+       ServerActiveObject *parent = m_env->getActiveObject(parent_id);
+       if (getType() == ACTIVEOBJECT_TYPE_LUAENTITY)
+               m_env->getScriptIface()->luaentity_on_detach(m_id, parent);
+
+       if (!parent || parent->isGone())
+               return; // Do not try to notify soon gone parent
+
+       if (parent->getType() == ACTIVEOBJECT_TYPE_LUAENTITY)
+               m_env->getScriptIface()->luaentity_on_detach_child(parent_id, this);
+}
+
 ObjectProperties* UnitSAO::accessObjectProperties()
 {
        return &m_prop;
@@ -224,7 +286,6 @@ ObjectProperties* UnitSAO::accessObjectProperties()
 
 void UnitSAO::notifyObjectPropertiesModified()
 {
-       m_env->updateActiveObject(this);
        m_properties_sent = false;
 }
 
@@ -286,33 +347,50 @@ ServerActiveObject* LuaEntitySAO::create(ServerEnvironment *env, v3f pos,
 {
        std::string name;
        std::string state;
-       s16 hp = 1;
+       u16 hp = 1;
        v3f velocity;
-       float yaw = 0;
-       if (!data.empty()) {
+       v3f rotation;
+
+       while (!data.empty()) { // breakable, run for one iteration
                std::istringstream is(data, std::ios::binary);
-               // read version
+               // 'version' does not allow to incrementally extend the parameter list thus
+               // we need another variable to build on top of 'version=1'. Ugly hack but works™
+               u8 version2 = 0;
                u8 version = readU8(is);
-               // check if version is supported
-               if(version == 0){
-                       name = deSerializeString(is);
-                       state = deSerializeLongString(is);
-               }
-               else if(version == 1){
-                       name = deSerializeString(is);
-                       state = deSerializeLongString(is);
-                       hp = readS16(is);
-                       velocity = readV3F1000(is);
-                       yaw = readF1000(is);
-               }
+
+               name = deSerializeString(is);
+               state = deSerializeLongString(is);
+
+               if (version < 1)
+                       break;
+
+               hp = readU16(is);
+               velocity = readV3F1000(is);
+               // yaw must be yaw to be backwards-compatible
+               rotation.Y = readF1000(is);
+
+               if (is.good()) // EOF for old formats
+                       version2 = readU8(is);
+
+               if (version2 < 1) // PROTOCOL_VERSION < 37
+                       break;
+
+               // version2 >= 1
+               rotation.X = readF1000(is);
+               rotation.Z = readF1000(is);
+
+               // if (version2 < 2)
+               //     break;
+               // <read new values>
+               break;
        }
        // create object
-       infostream<<"LuaEntitySAO::create(name=\""<<name<<"\" state=\""
-                       <<state<<"\")"<<std::endl;
+       infostream << "LuaEntitySAO::create(name=\"" << name << "\" state=\""
+                        << state << "\")" << std::endl;
        LuaEntitySAO *sao = new LuaEntitySAO(env, pos, name, state);
        sao->m_hp = hp;
        sao->m_velocity = velocity;
-       sao->m_yaw = yaw;
+       sao->m_rotation = rotation;
        return sao;
 }
 
@@ -376,18 +454,18 @@ void LuaEntitySAO::step(float dtime, bool send_recommended)
 
                if (m_prop.automatic_face_movement_dir &&
                                (fabs(m_velocity.Z) > 0.001 || fabs(m_velocity.X) > 0.001)) {
-
                        float target_yaw = atan2(m_velocity.Z, m_velocity.X) * 180 / M_PI
                                + m_prop.automatic_face_movement_dir_offset;
-                       float max_rotation_delta =
-                                       dtime * m_prop.automatic_face_movement_max_rotation_per_sec;
-                       float delta = wrapDegrees_0_360(target_yaw - m_yaw);
+                       float max_rotation_per_sec =
+                                       m_prop.automatic_face_movement_max_rotation_per_sec;
 
-                       if (delta > max_rotation_delta && 360 - delta > max_rotation_delta) {
-                               m_yaw += (delta < 180) ? max_rotation_delta : -max_rotation_delta;
-                               m_yaw = wrapDegrees_0_360(m_yaw);
+                       if (max_rotation_per_sec > 0) {
+                               m_rotation.Y = wrapDegrees_0_360(m_rotation.Y);
+                               wrappedApproachShortest(m_rotation.Y, target_yaw,
+                                       dtime * max_rotation_per_sec, 360.f);
                        } else {
-                               m_yaw = target_yaw;
+                               // Negative values of max_rotation_per_sec mean disabled.
+                               m_rotation.Y = target_yaw;
                        }
                }
        }
@@ -411,8 +489,11 @@ void LuaEntitySAO::step(float dtime, bool send_recommended)
                float move_d = m_base_position.getDistanceFrom(m_last_sent_position);
                move_d += m_last_sent_move_precision;
                float vel_d = m_velocity.getDistanceFrom(m_last_sent_velocity);
-               if(move_d > minchange || vel_d > minchange ||
-                               fabs(m_yaw - m_last_sent_yaw) > 1.0){
+               if (move_d > minchange || vel_d > minchange ||
+                               std::fabs(m_rotation.X - m_last_sent_rotation.X) > 1.0f ||
+                               std::fabs(m_rotation.Y - m_last_sent_rotation.Y) > 1.0f ||
+                               std::fabs(m_rotation.Z - m_last_sent_rotation.Z) > 1.0f) {
+
                        sendPosition(true, false);
                }
        }
@@ -468,14 +549,14 @@ std::string LuaEntitySAO::getClientInitializationData(u16 protocol_version)
 {
        std::ostringstream os(std::ios::binary);
 
-       // protocol >= 14
+       // PROTOCOL_VERSION >= 37
        writeU8(os, 1); // version
        os << serializeString(""); // name
        writeU8(os, 0); // is_player
-       writeS16(os, getId()); //id
-       writeV3F1000(os, m_base_position);
-       writeF1000(os, m_yaw);
-       writeS16(os, m_hp);
+       writeU16(os, getId()); //id
+       writeV3F32(os, m_base_position);
+       writeV3F32(os, m_rotation);
+       writeU16(os, m_hp);
 
        std::ostringstream msg_os(std::ios::binary);
        msg_os << serializeLongString(getPropertyPacket()); // message 1
@@ -494,6 +575,8 @@ std::string LuaEntitySAO::getClientInitializationData(u16 protocol_version)
                        (ii != m_attachment_child_ids.end()); ++ii) {
                if (ServerActiveObject *obj = m_env->getActiveObject(*ii)) {
                        message_count++;
+                       // TODO after a protocol bump: only send the object initialization data
+                       // to older clients (superfluous since this message exists)
                        msg_os << serializeLongString(gob_cmd_update_infant(*ii, obj->getSendType(),
                                obj->getClientInitializationData(protocol_version)));
                }
@@ -513,7 +596,7 @@ void LuaEntitySAO::getStaticData(std::string *result) const
 {
        verbosestream<<FUNCTION_NAME<<std::endl;
        std::ostringstream os(std::ios::binary);
-       // version
+       // version must be 1 to keep backwards-compatibility. See version2
        writeU8(os, 1);
        // name
        os<<serializeString(m_init_name);
@@ -525,12 +608,19 @@ void LuaEntitySAO::getStaticData(std::string *result) const
        } else {
                os<<serializeLongString(m_init_state);
        }
-       // hp
-       writeS16(os, m_hp);
-       // velocity
+       writeU16(os, m_hp);
        writeV3F1000(os, m_velocity);
        // yaw
-       writeF1000(os, m_yaw);
+       writeF1000(os, m_rotation.Y);
+
+       // version2. Increase this variable for new values
+       writeU8(os, 1); // PROTOCOL_VERSION >= 37
+
+       writeF1000(os, m_rotation.X);
+       writeF1000(os, m_rotation.Z);
+
+       // <write new values>
+
        *result = os.str();
 }
 
@@ -545,21 +635,15 @@ int LuaEntitySAO::punch(v3f dir,
                return 0;
        }
 
-       // It's best that attachments cannot be punched
-       if (isAttached())
-               return 0;
+       FATAL_ERROR_IF(!puncher, "Punch action called without SAO");
 
-       ItemStack *punchitem = NULL;
-       ItemStack punchitem_static;
-       if (puncher) {
-               punchitem_static = puncher->getWieldedItem();
-               punchitem = &punchitem_static;
-       }
+       s32 old_hp = getHP();
+       const ItemStack &punchitem = puncher->getWieldedItem();
 
        PunchDamageResult result = getPunchDamage(
                        m_armor_groups,
                        toolcap,
-                       punchitem,
+                       &punchitem,
                        time_from_last_punch);
 
        bool damage_handled = m_env->getScriptIface()->luaentity_Punch(m_id, puncher,
@@ -567,29 +651,29 @@ int LuaEntitySAO::punch(v3f dir,
 
        if (!damage_handled) {
                if (result.did_punch) {
-                       setHP(getHP() - result.damage,
+                       setHP((s32)getHP() - result.damage,
                                PlayerHPChangeReason(PlayerHPChangeReason::SET_HP));
 
-                       if (result.damage > 0) {
-                               std::string punchername = puncher ? puncher->getDescription() : "nil";
-
-                               actionstream << getDescription() << " punched by "
-                                               << punchername << ", damage " << result.damage
-                                               << " hp, health now " << getHP() << " hp" << std::endl;
-                       }
-
-                       std::string str = gob_cmd_punched(result.damage, getHP());
+                       std::string str = gob_cmd_punched(getHP());
                        // create message and add to list
                        ActiveObjectMessage aom(getId(), true, str);
                        m_messages_out.push(aom);
                }
        }
 
-       if (getHP() == 0) {
+       if (getHP() == 0 && !isGone()) {
                m_pending_removal = true;
+               clearParentAttachment();
+               clearChildAttachments();
                m_env->getScriptIface()->luaentity_on_death(m_id, puncher);
        }
 
+       actionstream << puncher->getDescription() << " (id=" << puncher->getId() <<
+                       ", hp=" << puncher->getHP() << ") punched " <<
+                       getDescription() << " (id=" << m_id << ", hp=" << m_hp <<
+                       "), damage=" << (old_hp - (s32)getHP()) <<
+                       (damage_handled ? " (handled by Lua)" : "") << std::endl;
+
        return result.wear;
 }
 
@@ -597,9 +681,7 @@ void LuaEntitySAO::rightClick(ServerActiveObject *clicker)
 {
        if (!m_registered)
                return;
-       // It's best that attachments cannot be clicked
-       if (isAttached())
-               return;
+
        m_env->getScriptIface()->luaentity_Rightclick(m_id, clicker);
 }
 
@@ -608,7 +690,6 @@ void LuaEntitySAO::setPos(const v3f &pos)
        if(isAttached())
                return;
        m_base_position = pos;
-       m_env->updateActiveObject(this);
        sendPosition(false, true);
 }
 
@@ -617,7 +698,6 @@ void LuaEntitySAO::moveTo(v3f pos, bool continuous)
        if(isAttached())
                return;
        m_base_position = pos;
-       m_env->updateActiveObject(this);
        if(!continuous)
                sendPosition(true, true);
 }
@@ -638,14 +718,12 @@ std::string LuaEntitySAO::getDescription()
        return os.str();
 }
 
-void LuaEntitySAO::setHP(s16 hp, const PlayerHPChangeReason &reason)
+void LuaEntitySAO::setHP(s32 hp, const PlayerHPChangeReason &reason)
 {
-       if (hp < 0)
-               hp = 0;
-       m_hp = hp;
+       m_hp = rangelim(hp, 0, U16_MAX);
 }
 
-s16 LuaEntitySAO::getHP() const
+u16 LuaEntitySAO::getHP() const
 {
        return m_hp;
 }
@@ -717,10 +795,10 @@ void LuaEntitySAO::sendPosition(bool do_interpolate, bool is_movement_end)
        m_last_sent_move_precision = m_base_position.getDistanceFrom(
                        m_last_sent_position);
        m_last_sent_position_timer = 0;
-       m_last_sent_yaw = m_yaw;
        m_last_sent_position = m_base_position;
        m_last_sent_velocity = m_velocity;
        //m_last_sent_acceleration = m_acceleration;
+       m_last_sent_rotation = m_rotation;
 
        float update_interval = m_env->getSendRecommendedInterval();
 
@@ -728,7 +806,7 @@ void LuaEntitySAO::sendPosition(bool do_interpolate, bool is_movement_end)
                m_base_position,
                m_velocity,
                m_acceleration,
-               m_yaw,
+               m_rotation,
                do_interpolate,
                is_movement_end,
                update_interval
@@ -796,7 +874,7 @@ PlayerSAO::PlayerSAO(ServerEnvironment *env_, RemotePlayer *player_, session_t p
        m_prop.pointable = true;
        // Start of default appearance, this should be overwritten by Lua
        m_prop.visual = "upright_sprite";
-       m_prop.visual_size = v2f(1, 2);
+       m_prop.visual_size = v3f(1, 2, 1);
        m_prop.textures.clear();
        m_prop.textures.emplace_back("player.png");
        m_prop.textures.emplace_back("player_back.png");
@@ -813,6 +891,9 @@ PlayerSAO::PlayerSAO(ServerEnvironment *env_, RemotePlayer *player_, session_t p
        m_breath = m_prop.breath_max;
        // Disable zoom in survival mode using a value of 0
        m_prop.zoom_fov = g_settings->getBool("creative_mode") ? 15.0f : 0.0f;
+
+       if (!g_settings->getBool("enable_damage"))
+               m_armor_groups["immortal"] = 1;
 }
 
 PlayerSAO::~PlayerSAO()
@@ -869,10 +950,10 @@ std::string PlayerSAO::getClientInitializationData(u16 protocol_version)
        writeU8(os, 1); // version
        os << serializeString(m_player->getName()); // name
        writeU8(os, 1); // is_player
-       writeS16(os, getId()); //id
-       writeV3F1000(os, m_base_position);
-       writeF1000(os, m_yaw);
-       writeS16(os, getHP());
+       writeS16(os, getId()); // id
+       writeV3F32(os, m_base_position);
+       writeV3F32(os, m_rotation);
+       writeU16(os, getHP());
 
        std::ostringstream msg_os(std::ios::binary);
        msg_os << serializeLongString(getPropertyPacket()); // message 1
@@ -915,7 +996,7 @@ void PlayerSAO::getStaticData(std::string * result) const
 
 void PlayerSAO::step(float dtime, bool send_recommended)
 {
-       if (m_drowning_interval.step(dtime, 2.0f)) {
+       if (!isImmortal() && m_drowning_interval.step(dtime, 2.0f)) {
                // Get nose/mouth position, approximate with eye position
                v3s16 p = floatToInt(getEyePosition(), BS);
                MapNode n = m_env->getMap().getNodeNoEx(p);
@@ -939,13 +1020,15 @@ void PlayerSAO::step(float dtime, bool send_recommended)
                v3s16 p = floatToInt(getEyePosition(), BS);
                MapNode n = m_env->getMap().getNodeNoEx(p);
                const ContentFeatures &c = m_env->getGameDef()->ndef()->get(n);
-               // If player is alive & no drowning, breathe
-               if (m_hp > 0 && m_breath < m_prop.breath_max && c.drowning == 0)
+               // If player is alive & no drowning & not in ignore, breathe
+               if (m_breath < m_prop.breath_max &&
+                               c.drowning == 0 && n.getContent() != CONTENT_IGNORE && m_hp > 0)
                        setBreath(m_breath + 1);
        }
 
-       if (m_node_hurt_interval.step(dtime, 1.0f)) {
+       if (!isImmortal() && m_node_hurt_interval.step(dtime, 1.0f)) {
                u32 damage_per_second = 0;
+               std::string nodename;
                // Lowest and highest damage points are 0.1 within collisionbox
                float dam_top = m_prop.collisionbox.MaxEdge.Y - 0.1f;
 
@@ -955,20 +1038,26 @@ void PlayerSAO::step(float dtime, bool send_recommended)
                        v3s16 p = floatToInt(m_base_position +
                                v3f(0.0f, dam_height * BS, 0.0f), BS);
                        MapNode n = m_env->getMap().getNodeNoEx(p);
-                       damage_per_second = std::max(damage_per_second,
-                               m_env->getGameDef()->ndef()->get(n).damage_per_second);
+                       const ContentFeatures &c = m_env->getGameDef()->ndef()->get(n);
+                       if (c.damage_per_second > damage_per_second) {
+                               damage_per_second = c.damage_per_second;
+                               nodename = c.name;
+                       }
                }
 
                // Top damage point
                v3s16 ptop = floatToInt(m_base_position +
                        v3f(0.0f, dam_top * BS, 0.0f), BS);
                MapNode ntop = m_env->getMap().getNodeNoEx(ptop);
-               damage_per_second = std::max(damage_per_second,
-                       m_env->getGameDef()->ndef()->get(ntop).damage_per_second);
+               const ContentFeatures &c = m_env->getGameDef()->ndef()->get(ntop);
+               if (c.damage_per_second > damage_per_second) {
+                       damage_per_second = c.damage_per_second;
+                       nodename = c.name;
+               }
 
                if (damage_per_second != 0 && m_hp > 0) {
-                       s16 newhp = ((s32) damage_per_second > m_hp ? 0 : m_hp - damage_per_second);
-                       PlayerHPChangeReason reason(PlayerHPChangeReason::NODE_DAMAGE);
+                       s32 newhp = (s32)m_hp - (s32)damage_per_second;
+                       PlayerHPChangeReason reason(PlayerHPChangeReason::NODE_DAMAGE, nodename);
                        setHP(newhp, reason);
                        m_env->getGameDef()->SendPlayerHPOrDie(this, reason);
                }
@@ -1023,7 +1112,7 @@ void PlayerSAO::step(float dtime, bool send_recommended)
                return;
 
        // If the object is attached client-side, don't waste bandwidth sending its
-       // position to clients.
+       // position or rotation to clients.
        if (m_position_not_sent && !isAttached()) {
                m_position_not_sent = false;
                float update_interval = m_env->getSendRecommendedInterval();
@@ -1037,7 +1126,7 @@ void PlayerSAO::step(float dtime, bool send_recommended)
                        pos,
                        v3f(0.0f, 0.0f, 0.0f),
                        v3f(0.0f, 0.0f, 0.0f),
-                       m_yaw,
+                       m_rotation,
                        true,
                        false,
                        update_interval
@@ -1105,8 +1194,11 @@ void PlayerSAO::setBasePosition(const v3f &position)
 
        // This needs to be ran for attachments too
        ServerActiveObject::setBasePosition(position);
-       m_env->updateActiveObject(this);
-       m_position_not_sent = true;
+
+       // Updating is not wanted/required for player migration
+       if (m_env) {
+               m_position_not_sent = true;
+       }
 }
 
 void PlayerSAO::setPos(const v3f &pos)
@@ -1114,6 +1206,10 @@ void PlayerSAO::setPos(const v3f &pos)
        if(isAttached())
                return;
 
+       // Send mapblock of target location
+       v3s16 blockpos = v3s16(pos.X / MAP_BLOCKSIZE, pos.Y / MAP_BLOCKSIZE, pos.Z / MAP_BLOCKSIZE);
+       m_env->getGameDef()->SendBlock(m_peer_id, blockpos);
+
        setBasePosition(pos);
        // Movement caused by this command is always valid
        m_last_good_position = pos;
@@ -1135,12 +1231,14 @@ void PlayerSAO::moveTo(v3f pos, bool continuous)
        m_env->getGameDef()->SendMovePlayer(m_peer_id);
 }
 
-void PlayerSAO::setYaw(const float yaw)
+void PlayerSAO::setPlayerYaw(const float yaw)
 {
-       if (m_player && yaw != m_yaw)
+       v3f rotation(0, yaw, 0);
+       if (m_player && yaw != m_rotation.Y)
                m_player->setDirty(true);
 
-       UnitSAO::setYaw(yaw);
+       // Set player model yaw, not look view
+       UnitSAO::setRotation(rotation);
 }
 
 void PlayerSAO::setFov(const float fov)
@@ -1159,13 +1257,13 @@ void PlayerSAO::setWantedRange(const s16 range)
        m_wanted_range = range;
 }
 
-void PlayerSAO::setYawAndSend(const float yaw)
+void PlayerSAO::setPlayerYawAndSend(const float yaw)
 {
-       setYaw(yaw);
+       setPlayerYaw(yaw);
        m_env->getGameDef()->SendMovePlayer(m_peer_id);
 }
 
-void PlayerSAO::setPitch(const float pitch)
+void PlayerSAO::setLookPitch(const float pitch)
 {
        if (m_player && pitch != m_pitch)
                m_player->setDirty(true);
@@ -1173,9 +1271,9 @@ void PlayerSAO::setPitch(const float pitch)
        m_pitch = pitch;
 }
 
-void PlayerSAO::setPitchAndSend(const float pitch)
+void PlayerSAO::setLookPitchAndSend(const float pitch)
 {
-       setPitch(pitch);
+       setLookPitch(pitch);
        m_env->getGameDef()->SendMovePlayer(m_peer_id);
 }
 
@@ -1184,17 +1282,15 @@ int PlayerSAO::punch(v3f dir,
        ServerActiveObject *puncher,
        float time_from_last_punch)
 {
-       // It's best that attachments cannot be punched
-       if (isAttached())
-               return 0;
-
        if (!toolcap)
                return 0;
 
+       FATAL_ERROR_IF(!puncher, "Punch action called without SAO");
+
        // No effect if PvP disabled
        if (!g_settings->getBool("enable_pvp")) {
                if (puncher->getType() == ACTIVEOBJECT_TYPE_PLAYER) {
-                       std::string str = gob_cmd_punched(0, getHP());
+                       std::string str = gob_cmd_punched(getHP());
                        // create message and add to list
                        ActiveObjectMessage aom(getId(), true, str);
                        m_messages_out.push(aom);
@@ -1202,14 +1298,10 @@ int PlayerSAO::punch(v3f dir,
                }
        }
 
+       s32 old_hp = getHP();
        HitParams hitparams = getHitParams(m_armor_groups, toolcap,
                        time_from_last_punch);
 
-       std::string punchername = "nil";
-
-       if (puncher != 0)
-               punchername = puncher->getDescription();
-
        PlayerSAO *playersao = m_player->getPlayerSAO();
 
        bool damage_handled = m_env->getScriptIface()->on_punchplayer(playersao,
@@ -1217,60 +1309,45 @@ int PlayerSAO::punch(v3f dir,
                                hitparams.hp);
 
        if (!damage_handled) {
-               setHP(getHP() - hitparams.hp,
+               setHP((s32)getHP() - (s32)hitparams.hp,
                                PlayerHPChangeReason(PlayerHPChangeReason::PLAYER_PUNCH, puncher));
        } else { // override client prediction
                if (puncher->getType() == ACTIVEOBJECT_TYPE_PLAYER) {
-                       std::string str = gob_cmd_punched(0, getHP());
+                       std::string str = gob_cmd_punched(getHP());
                        // create message and add to list
                        ActiveObjectMessage aom(getId(), true, str);
                        m_messages_out.push(aom);
                }
        }
 
-
-       actionstream << "Player " << m_player->getName() << " punched by "
-                       << punchername;
-       if (!damage_handled) {
-               actionstream << ", damage " << hitparams.hp << " HP";
-       } else {
-               actionstream << ", damage handled by lua";
-       }
-       actionstream << std::endl;
+       actionstream << puncher->getDescription() << " (id=" << puncher->getId() <<
+               ", hp=" << puncher->getHP() << ") punched " <<
+               getDescription() << " (id=" << m_id << ", hp=" << m_hp <<
+               "), damage=" << (old_hp - (s32)getHP()) <<
+               (damage_handled ? " (handled by Lua)" : "") << std::endl;
 
        return hitparams.wear;
 }
 
-s16 PlayerSAO::readDamage()
+void PlayerSAO::setHP(s32 hp, const PlayerHPChangeReason &reason)
 {
-       s16 damage = m_damage;
-       m_damage = 0;
-       return damage;
-}
+       s32 oldhp = m_hp;
 
-void PlayerSAO::setHP(s16 hp, const PlayerHPChangeReason &reason)
-{
-       s16 oldhp = m_hp;
+       hp = rangelim(hp, 0, m_prop.hp_max);
 
-       s16 hp_change = m_env->getScriptIface()->on_player_hpchange(this, hp - oldhp, reason);
-       if (hp_change == 0)
-               return;
-       hp = oldhp + hp_change;
+       if (oldhp != hp) {
+               s32 hp_change = m_env->getScriptIface()->on_player_hpchange(this, hp - oldhp, reason);
+               if (hp_change == 0)
+                       return;
 
-       if (hp < 0)
-               hp = 0;
-       else if (hp > m_prop.hp_max)
-               hp = m_prop.hp_max;
+               hp = rangelim(oldhp + hp_change, 0, m_prop.hp_max);
+       }
 
-       if (hp < oldhp && !g_settings->getBool("enable_damage")) {
+       if (hp < oldhp && isImmortal())
                return;
-       }
 
        m_hp = hp;
 
-       if (oldhp > hp)
-               m_damage += (oldhp - hp);
-
        // Update properties on death
        if ((hp == 0) != (oldhp == 0))
                m_properties_sent = false;
@@ -1281,7 +1358,7 @@ void PlayerSAO::setBreath(const u16 breath, bool send)
        if (m_player && breath != m_breath)
                m_player->setDirty(true);
 
-       m_breath = MYMIN(breath, m_prop.breath_max);
+       m_breath = rangelim(breath, 0, m_prop.breath_max);
 
        if (send)
                m_env->getGameDef()->SendPlayerBreath(this);
@@ -1470,3 +1547,8 @@ bool PlayerSAO::getSelectionBox(aabb3f *toset) const
 
        return true;
 }
+
+float PlayerSAO::getZoomFOV() const
+{
+       return m_prop.zoom_fov;
+}