Move shared parameters sending to UnitSAO (#9968)
[oweals/minetest.git] / src / client / content_cao.cpp
index e76cacdd47799b5f67fb2a067d0c2393cb64a8ec..702d089afe582a824929d9269c64c849a7dfb0e6 100644 (file)
@@ -46,6 +46,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "wieldmesh.h"
 #include <algorithm>
 #include <cmath>
+#include "client/shader.h"
 
 class Settings;
 struct ToolCapabilities;
@@ -180,7 +181,7 @@ public:
 
        void addToScene(ITextureSource *tsrc);
        void removeFromScene(bool permanent);
-       void updateLight(u8 light_at_pos);
+       void updateLight(u32 day_night_ratio);
        v3s16 getLightPosition();
        void updateNodePos();
 
@@ -253,7 +254,7 @@ void TestCAO::removeFromScene(bool permanent)
        m_node = NULL;
 }
 
-void TestCAO::updateLight(u8 light_at_pos)
+void TestCAO::updateLight(u32 day_night_ratio)
 {
 }
 
@@ -303,7 +304,6 @@ void TestCAO::processMessage(const std::string &data)
        GenericCAO
 */
 
-#include "genericobject.h"
 #include "clientobject.h"
 
 GenericCAO::GenericCAO(Client *client, ClientEnvironment *env):
@@ -352,6 +352,8 @@ void GenericCAO::initialize(const std::string &data)
                        player->setCAO(this);
                }
        }
+
+       m_enable_shaders = g_settings->getBool("enable_shaders");
 }
 
 void GenericCAO::processInitData(const std::string &data)
@@ -401,15 +403,19 @@ bool GenericCAO::getSelectionBox(aabb3f *toset) const
        return true;
 }
 
-v3f GenericCAO::getPosition()
+const v3f GenericCAO::getPosition() const
 {
-       if (getParent() != nullptr) {
-               if (m_matrixnode)
-                       return m_matrixnode->getAbsolutePosition();
+       if (!getParent())
+               return pos_translator.val_current;
 
-               return m_position;
+       // Calculate real position in world based on MatrixNode
+       if (m_matrixnode) {
+               v3s16 camera_offset = m_env->getCameraOffset();
+               return m_matrixnode->getAbsolutePosition() +
+                               intToFloat(camera_offset, BS);
        }
-       return pos_translator.val_current;
+
+       return m_position;
 }
 
 const bool GenericCAO::isImmortal()
@@ -417,7 +423,7 @@ const bool GenericCAO::isImmortal()
        return itemgroup_get(getGroups(), "immortal");
 }
 
-scene::ISceneNode* GenericCAO::getSceneNode()
+scene::ISceneNode *GenericCAO::getSceneNode() const
 {
        if (m_meshnode) {
                return m_meshnode;
@@ -437,7 +443,7 @@ scene::ISceneNode* GenericCAO::getSceneNode()
        return NULL;
 }
 
-scene::IAnimatedMeshSceneNode* GenericCAO::getAnimatedMeshSceneNode()
+scene::IAnimatedMeshSceneNode *GenericCAO::getAnimatedMeshSceneNode() const
 {
        return m_animated_meshnode;
 }
@@ -469,6 +475,7 @@ void GenericCAO::setAttachment(int parent_id, const std::string &bone, v3f posit
                        parent->addAttachmentChild(m_id);
        }
 
+
        updateAttachments();
 }
 
@@ -523,7 +530,9 @@ void GenericCAO::removeFromScene(bool permanent)
        // Should be true when removing the object permanently
        // and false when refreshing (eg: updating visuals)
        if (m_env && permanent) {
-               clearChildAttachments();
+               // The client does not know whether this object does re-appear to
+               // a later time, thus do not clear child attachments.
+
                clearParentAttachment();
        }
 
@@ -567,29 +576,51 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
 
        m_visuals_expired = false;
 
-       if (!m_prop.is_visible) {
+       if (!m_prop.is_visible)
                return;
-       }
 
-       video::E_MATERIAL_TYPE material_type = (m_prop.use_texture_alpha) ?
-               video::EMT_TRANSPARENT_ALPHA_CHANNEL : video::EMT_TRANSPARENT_ALPHA_CHANNEL_REF;
+       infostream << "GenericCAO::addToScene(): " << m_prop.visual << std::endl;
 
-       if (m_prop.visual == "sprite") {
-               infostream<<"GenericCAO::addToScene(): single_sprite"<<std::endl;
+       if (m_enable_shaders) {
+               IShaderSource *shader_source = m_client->getShaderSource();
+               u32 shader_id = shader_source->getShader(
+                               "object_shader",
+                               (m_prop.use_texture_alpha) ? TILE_MATERIAL_ALPHA : TILE_MATERIAL_BASIC,
+                               NDT_NORMAL);
+               m_material_type = shader_source->getShaderInfo(shader_id).material;
+       } else {
+               m_material_type = (m_prop.use_texture_alpha) ?
+                       video::EMT_TRANSPARENT_ALPHA_CHANNEL : video::EMT_TRANSPARENT_ALPHA_CHANNEL_REF;
+       }
+
+       auto grabMatrixNode = [this] {
                m_matrixnode = RenderingEngine::get_scene_manager()->
                                addDummyTransformationSceneNode();
                m_matrixnode->grab();
+       };
+
+       auto setSceneNodeMaterial = [this] (scene::ISceneNode *node) {
+               node->setMaterialFlag(video::EMF_LIGHTING, false);
+               node->setMaterialFlag(video::EMF_BILINEAR_FILTER, false);
+               node->setMaterialFlag(video::EMF_FOG_ENABLE, true);
+               node->setMaterialType(m_material_type);
+
+               if (m_enable_shaders) {
+                       node->setMaterialFlag(video::EMF_GOURAUD_SHADING, false);
+                       node->setMaterialFlag(video::EMF_NORMALIZE_NORMALS, true);
+               }
+       };
+
+       if (m_prop.visual == "sprite") {
+               grabMatrixNode();
                m_spritenode = RenderingEngine::get_scene_manager()->addBillboardSceneNode(
                                m_matrixnode, v2f(1, 1), v3f(0,0,0), -1);
                m_spritenode->grab();
                m_spritenode->setMaterialTexture(0,
                                tsrc->getTextureForMesh("unknown_node.png"));
-               m_spritenode->setMaterialFlag(video::EMF_LIGHTING, false);
-               m_spritenode->setMaterialFlag(video::EMF_BILINEAR_FILTER, false);
-               m_spritenode->setMaterialType(material_type);
-               m_spritenode->setMaterialFlag(video::EMF_FOG_ENABLE, true);
-               u8 li = m_last_light;
-               m_spritenode->setColor(video::SColor(255,li,li,li));
+
+               setSceneNodeMaterial(m_spritenode);
+
                m_spritenode->setSize(v2f(m_prop.visual_size.X,
                                m_prop.visual_size.Y) * BS);
                {
@@ -599,19 +630,19 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
                                        txs, tys, 0, 0);
                }
        } else if (m_prop.visual == "upright_sprite") {
+               grabMatrixNode();
                scene::SMesh *mesh = new scene::SMesh();
                double dx = BS * m_prop.visual_size.X / 2;
                double dy = BS * m_prop.visual_size.Y / 2;
-               u8 li = m_last_light;
-               video::SColor c(255, li, li, li);
+               video::SColor c(0xFFFFFFFF);
 
                { // Front
                        scene::IMeshBuffer *buf = new scene::SMeshBuffer();
                        video::S3DVertex vertices[4] = {
-                               video::S3DVertex(-dx, -dy, 0, 0,0,0, c, 1,1),
-                               video::S3DVertex( dx, -dy, 0, 0,0,0, c, 0,1),
-                               video::S3DVertex( dx,  dy, 0, 0,0,0, c, 0,0),
-                               video::S3DVertex(-dx,  dy, 0, 0,0,0, c, 1,0),
+                               video::S3DVertex(-dx, -dy, 0, 0,0,1, c, 1,1),
+                               video::S3DVertex( dx, -dy, 0, 0,0,1, c, 0,1),
+                               video::S3DVertex( dx,  dy, 0, 0,0,1, c, 0,0),
+                               video::S3DVertex(-dx,  dy, 0, 0,0,1, c, 1,0),
                        };
                        if (m_is_player) {
                                // Move minimal Y position to 0 (feet position)
@@ -624,7 +655,14 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
                        buf->getMaterial().setFlag(video::EMF_LIGHTING, false);
                        buf->getMaterial().setFlag(video::EMF_BILINEAR_FILTER, false);
                        buf->getMaterial().setFlag(video::EMF_FOG_ENABLE, true);
-                       buf->getMaterial().MaterialType = video::EMT_TRANSPARENT_ALPHA_CHANNEL;
+                       buf->getMaterial().MaterialType = m_material_type;
+
+                       if (m_enable_shaders) {
+                               buf->getMaterial().EmissiveColor = c;
+                               buf->getMaterial().setFlag(video::EMF_GOURAUD_SHADING, false);
+                               buf->getMaterial().setFlag(video::EMF_NORMALIZE_NORMALS, true);
+                       }
+
                        // Add to mesh
                        mesh->addMeshBuffer(buf);
                        buf->drop();
@@ -632,10 +670,10 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
                { // Back
                        scene::IMeshBuffer *buf = new scene::SMeshBuffer();
                        video::S3DVertex vertices[4] = {
-                               video::S3DVertex( dx,-dy, 0, 0,0,0, c, 1,1),
-                               video::S3DVertex(-dx,-dy, 0, 0,0,0, c, 0,1),
-                               video::S3DVertex(-dx, dy, 0, 0,0,0, c, 0,0),
-                               video::S3DVertex( dx, dy, 0, 0,0,0, c, 1,0),
+                               video::S3DVertex( dx,-dy, 0, 0,0,-1, c, 1,1),
+                               video::S3DVertex(-dx,-dy, 0, 0,0,-1, c, 0,1),
+                               video::S3DVertex(-dx, dy, 0, 0,0,-1, c, 0,0),
+                               video::S3DVertex( dx, dy, 0, 0,0,-1, c, 1,0),
                        };
                        if (m_is_player) {
                                // Move minimal Y position to 0 (feet position)
@@ -648,14 +686,18 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
                        buf->getMaterial().setFlag(video::EMF_LIGHTING, false);
                        buf->getMaterial().setFlag(video::EMF_BILINEAR_FILTER, false);
                        buf->getMaterial().setFlag(video::EMF_FOG_ENABLE, true);
-                       buf->getMaterial().MaterialType = video::EMT_TRANSPARENT_ALPHA_CHANNEL_REF;
+                       buf->getMaterial().MaterialType = m_material_type;
+
+                       if (m_enable_shaders) {
+                               buf->getMaterial().EmissiveColor = c;
+                               buf->getMaterial().setFlag(video::EMF_GOURAUD_SHADING, false);
+                               buf->getMaterial().setFlag(video::EMF_NORMALIZE_NORMALS, true);
+                       }
+
                        // Add to mesh
                        mesh->addMeshBuffer(buf);
                        buf->drop();
                }
-               m_matrixnode = RenderingEngine::get_scene_manager()->
-                       addDummyTransformationSceneNode();
-               m_matrixnode->grab();
                m_meshnode = RenderingEngine::get_scene_manager()->
                        addMeshSceneNode(mesh, m_matrixnode);
                m_meshnode->grab();
@@ -664,55 +706,43 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
                // This is needed for changing the texture in the future
                m_meshnode->setReadOnlyMaterials(true);
        } else if (m_prop.visual == "cube") {
-               infostream<<"GenericCAO::addToScene(): cube"<<std::endl;
+               grabMatrixNode();
                scene::IMesh *mesh = createCubeMesh(v3f(BS,BS,BS));
-               m_matrixnode = RenderingEngine::get_scene_manager()->
-                       addDummyTransformationSceneNode(nullptr);
-               m_matrixnode->grab();
                m_meshnode = RenderingEngine::get_scene_manager()->
                        addMeshSceneNode(mesh, m_matrixnode);
                m_meshnode->grab();
                mesh->drop();
 
                m_meshnode->setScale(m_prop.visual_size);
-               u8 li = m_last_light;
-               setMeshColor(m_meshnode->getMesh(), video::SColor(255,li,li,li));
+               m_meshnode->setMaterialFlag(video::EMF_BACK_FACE_CULLING,
+                       m_prop.backface_culling);
 
-               m_meshnode->setMaterialFlag(video::EMF_LIGHTING, false);
-               m_meshnode->setMaterialFlag(video::EMF_BILINEAR_FILTER, false);
-               m_meshnode->setMaterialType(material_type);
-               m_meshnode->setMaterialFlag(video::EMF_FOG_ENABLE, true);
+               setSceneNodeMaterial(m_meshnode);
        } else if (m_prop.visual == "mesh") {
-               infostream<<"GenericCAO::addToScene(): mesh"<<std::endl;
+               grabMatrixNode();
                scene::IAnimatedMesh *mesh = m_client->getMesh(m_prop.mesh, true);
                if (mesh) {
-                       m_matrixnode = RenderingEngine::get_scene_manager()->
-                               addDummyTransformationSceneNode(nullptr);
-                       m_matrixnode->grab();
                        m_animated_meshnode = RenderingEngine::get_scene_manager()->
                                addAnimatedMeshSceneNode(mesh, m_matrixnode);
                        m_animated_meshnode->grab();
                        mesh->drop(); // The scene node took hold of it
                        m_animated_meshnode->animateJoints(); // Needed for some animations
                        m_animated_meshnode->setScale(m_prop.visual_size);
-                       u8 li = m_last_light;
 
                        // set vertex colors to ensure alpha is set
-                       setMeshColor(m_animated_meshnode->getMesh(), video::SColor(255,li,li,li));
+                       setMeshColor(m_animated_meshnode->getMesh(), video::SColor(0xFFFFFFFF));
+
+                       setAnimatedMeshColor(m_animated_meshnode, video::SColor(0xFFFFFFFF));
 
-                       setAnimatedMeshColor(m_animated_meshnode, video::SColor(255,li,li,li));
+                       setSceneNodeMaterial(m_animated_meshnode);
 
-                       m_animated_meshnode->setMaterialFlag(video::EMF_LIGHTING, true);
-                       m_animated_meshnode->setMaterialFlag(video::EMF_BILINEAR_FILTER, false);
-                       m_animated_meshnode->setMaterialType(material_type);
-                       m_animated_meshnode->setMaterialFlag(video::EMF_FOG_ENABLE, true);
                        m_animated_meshnode->setMaterialFlag(video::EMF_BACK_FACE_CULLING,
                                m_prop.backface_culling);
                } else
                        errorstream<<"GenericCAO::addToScene(): Could not load mesh "<<m_prop.mesh<<std::endl;
        } else if (m_prop.visual == "wielditem" || m_prop.visual == "item") {
+               grabMatrixNode();
                ItemStack item;
-               infostream << "GenericCAO::addToScene(): wielditem" << std::endl;
                if (m_prop.wield_item.empty()) {
                        // Old format, only textures are specified.
                        infostream << "textures: " << m_prop.textures.size() << std::endl;
@@ -726,18 +756,13 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
                        infostream << "serialized form: " << m_prop.wield_item << std::endl;
                        item.deSerialize(m_prop.wield_item, m_client->idef());
                }
-               m_matrixnode = RenderingEngine::get_scene_manager()->
-                       addDummyTransformationSceneNode(nullptr);
-               m_matrixnode->grab();
                m_wield_meshnode = new WieldMeshSceneNode(
                        RenderingEngine::get_scene_manager(), -1);
-               m_wield_meshnode->setParent(m_matrixnode);
                m_wield_meshnode->setItem(item, m_client,
                        (m_prop.visual == "wielditem"));
 
                m_wield_meshnode->setScale(m_prop.visual_size / 2.0f);
-               u8 li = m_last_light;
-               m_wield_meshnode->setColor(video::SColor(255, li, li, li));
+               m_wield_meshnode->setColor(video::SColor(0xFFFFFFFF));
        } else {
                infostream<<"GenericCAO::addToScene(): \""<<m_prop.visual
                                <<"\" not supported"<<std::endl;
@@ -749,54 +774,71 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
 
        scene::ISceneNode *node = getSceneNode();
 
-       if (node && !m_prop.nametag.empty() && !m_is_local_player) {
-               // Add nametag
-               v3f pos;
-               pos.Y = m_prop.selectionbox.MaxEdge.Y + 0.3f;
-               m_nametag = m_client->getCamera()->addNametag(node,
-                       m_prop.nametag, m_prop.nametag_color,
-                       pos);
-       }
+       if (node && m_matrixnode)
+               node->setParent(m_matrixnode);
 
+       updateNametag();
        updateNodePos();
        updateAnimation();
        updateBonePosition();
        updateAttachments();
+       setNodeLight(m_last_light);
 }
 
-void GenericCAO::updateLight(u8 light_at_pos)
+void GenericCAO::updateLight(u32 day_night_ratio)
 {
-       // Don't update light of attached one
-       if (getParent() != NULL) {
-               return;
-       }
+       u8 light_at_pos = 0;
+       bool pos_ok;
 
-       updateLightNoCheck(light_at_pos);
+       v3s16 p = getLightPosition();
+       MapNode n = m_env->getMap().getNode(p, &pos_ok);
+       if (pos_ok)
+               light_at_pos = n.getLightBlend(day_night_ratio, m_client->ndef());
+       else
+               light_at_pos = blend_light(day_night_ratio, LIGHT_SUN, 0);
 
-       // Update light of all children
-       for (u16 i : m_attachment_child_ids) {
-               ClientActiveObject *obj = m_env->getActiveObject(i);
-               if (obj) {
-                       obj->updateLightNoCheck(light_at_pos);
-               }
+       u8 light = decode_light(light_at_pos);
+       if (light != m_last_light) {
+               m_last_light = light;
+               setNodeLight(light);
        }
 }
 
-void GenericCAO::updateLightNoCheck(u8 light_at_pos)
+void GenericCAO::setNodeLight(u8 light)
 {
-       if (m_glow < 0)
+       video::SColor color(255, light, light, light);
+
+       if (m_prop.visual == "wielditem" || m_prop.visual == "item") {
+               if (m_wield_meshnode)
+                       m_wield_meshnode->setNodeLightColor(color);
                return;
+       }
+
+       if (m_enable_shaders) {
+               if (m_prop.visual == "upright_sprite") {
+                       if (!m_meshnode)
+                               return;
+
+                       scene::IMesh *mesh = m_meshnode->getMesh();
+                       for (u32 i = 0; i < mesh->getMeshBufferCount(); ++i) {
+                               scene::IMeshBuffer *buf = mesh->getMeshBuffer(i);
+                               buf->getMaterial().EmissiveColor = color;
+                       }
+               } else {
+                       scene::ISceneNode *node = getSceneNode();
+                       if (!node)
+                               return;
 
-       u8 li = decode_light(light_at_pos + m_glow);
-       if (li != m_last_light) {
-               m_last_light = li;
-               video::SColor color(255,li,li,li);
+                       for (u32 i = 0; i < node->getMaterialCount(); ++i) {
+                               video::SMaterial &material = node->getMaterial(i);
+                               material.EmissiveColor = color;
+                       }
+               }
+       } else {
                if (m_meshnode) {
                        setMeshColor(m_meshnode->getMesh(), color);
                } else if (m_animated_meshnode) {
                        setAnimatedMeshColor(m_animated_meshnode, color);
-               } else if (m_wield_meshnode) {
-                       m_wield_meshnode->setColor(color);
                } else if (m_spritenode) {
                        m_spritenode->setColor(color);
                }
@@ -811,6 +853,38 @@ v3s16 GenericCAO::getLightPosition()
        return floatToInt(m_position, BS);
 }
 
+void GenericCAO::updateNametag()
+{
+       if (m_is_local_player) // No nametag for local player
+               return;
+
+       if (m_prop.nametag.empty()) {
+               // Delete nametag
+               if (m_nametag) {
+                       m_client->getCamera()->removeNametag(m_nametag);
+                       m_nametag = nullptr;
+               }
+               return;
+       }
+
+       scene::ISceneNode *node = getSceneNode();
+       if (!node)
+               return;
+
+       v3f pos;
+       pos.Y = m_prop.selectionbox.MaxEdge.Y + 0.3f;
+       if (!m_nametag) {
+               // Add nametag
+               m_nametag = m_client->getCamera()->addNametag(node,
+                       m_prop.nametag, m_prop.nametag_color, pos);
+       } else {
+               // Update nametag
+               m_nametag->nametag_text = m_prop.nametag;
+               m_nametag->nametag_color = m_prop.nametag_color;
+               m_nametag->nametag_pos = pos;
+       }
+}
+
 void GenericCAO::updateNodePos()
 {
        if (getParent() != NULL)
@@ -832,18 +906,24 @@ void GenericCAO::updateNodePos()
 
 void GenericCAO::step(float dtime, ClientEnvironment *env)
 {
-       // Handel model of local player instantly to prevent lags
+       if (m_animated_meshnode) {
+               m_animated_meshnode->animateJoints();
+               updateBonePosition();
+       }
+       
+       // Handle model animations and update positions instantly to prevent lags
        if (m_is_local_player) {
                LocalPlayer *player = m_env->getLocalPlayer();
+               m_position = player->getPosition();
+               pos_translator.val_current = m_position;
+               m_rotation.Y = wrapDegrees_0_360(player->getYaw());
+               rot_translator.val_current = m_rotation;
+
                if (m_is_visible) {
                        int old_anim = player->last_animation;
                        float old_anim_speed = player->last_animation_speed;
-                       m_position = player->getPosition();
-                       m_rotation.Y = wrapDegrees_0_360(player->getYaw());
                        m_velocity = v3f(0,0,0);
                        m_acceleration = v3f(0,0,0);
-                       pos_translator.val_current = m_position;
-                       rot_translator.val_current = m_rotation;
                        const PlayerControl &controls = player->getPlayerControl();
 
                        bool walking = false;
@@ -881,7 +961,7 @@ void GenericCAO::step(float dtime, ClientEnvironment *env)
 
                        // Apply animations if input detected and not attached
                        // or set idle animation
-                       if ((new_anim.X + new_anim.Y) > 0 && !player->isAttached) {
+                       if ((new_anim.X + new_anim.Y) > 0 && !getParent()) {
                                allow_update = true;
                                m_animation_range = new_anim;
                                m_animation_speed = new_speed;
@@ -943,12 +1023,7 @@ void GenericCAO::step(float dtime, ClientEnvironment *env)
                m_velocity = v3f(0,0,0);
                m_acceleration = v3f(0,0,0);
                pos_translator.val_current = m_position;
-
-               if(m_is_local_player) // Update local player attachment position
-               {
-                       LocalPlayer *player = m_env->getLocalPlayer();
-                       player->overridePosition = getParent()->getPosition();
-               }
+               pos_translator.val_target = m_position;
        } else {
                rot_translator.translate(dtime);
                v3f lastpos = pos_translator.val_current;
@@ -972,16 +1047,14 @@ void GenericCAO::step(float dtime, ClientEnvironment *env)
 
                        bool is_end_position = moveresult.collides;
                        pos_translator.update(m_position, is_end_position, dtime);
-                       pos_translator.translate(dtime);
-                       updateNodePos();
                } else {
                        m_position += dtime * m_velocity + 0.5 * dtime * dtime * m_acceleration;
                        m_velocity += dtime * m_acceleration;
                        pos_translator.update(m_position, pos_translator.aim_is_end,
                                        pos_translator.anim_time);
-                       pos_translator.translate(dtime);
-                       updateNodePos();
                }
+               pos_translator.translate(dtime);
+               updateNodePos();
 
                float moved = lastpos.getDistanceFrom(pos_translator.val_current);
                m_step_distance_counter += moved;
@@ -1020,10 +1093,13 @@ void GenericCAO::step(float dtime, ClientEnvironment *env)
                        updateTextures(m_previous_texture_modifier);
                }
        }
+
        if (!getParent() && std::fabs(m_prop.automatic_rotate) > 0.001) {
-               m_rotation.Y += dtime * m_prop.automatic_rotate * 180 / M_PI;
-               rot_translator.val_current = m_rotation;
-               updateNodePos();
+               // This is the child node's rotation. It is only used for automatic_rotate.
+               v3f local_rot = node->getRotation();
+               local_rot.Y = modulo360f(local_rot.Y - dtime * core::RADTODEG *
+                               m_prop.automatic_rotate);
+               node->setRotation(local_rot);
        }
 
        if (!getParent() && m_prop.automatic_face_movement_dir &&
@@ -1106,16 +1182,13 @@ void GenericCAO::updateTextures(std::string mod)
        m_current_texture_modifier = mod;
        m_glow = m_prop.glow;
 
-       video::E_MATERIAL_TYPE material_type = (m_prop.use_texture_alpha) ?
-               video::EMT_TRANSPARENT_ALPHA_CHANNEL : video::EMT_TRANSPARENT_ALPHA_CHANNEL_REF;
-
        if (m_spritenode) {
                if (m_prop.visual == "sprite") {
                        std::string texturestring = "unknown_node.png";
                        if (!m_prop.textures.empty())
                                texturestring = m_prop.textures[0];
                        texturestring += mod;
-                       m_spritenode->getMaterial(0).MaterialType = material_type;
+                       m_spritenode->getMaterial(0).MaterialType = m_material_type;
                        m_spritenode->getMaterial(0).MaterialTypeParam = 0.5f;
                        m_spritenode->setMaterialTexture(0,
                                        tsrc->getTextureForMesh(texturestring));
@@ -1151,7 +1224,7 @@ void GenericCAO::updateTextures(std::string mod)
 
                                // Set material flags and texture
                                video::SMaterial& material = m_animated_meshnode->getMaterial(i);
-                               material.MaterialType = material_type;
+                               material.MaterialType = m_material_type;
                                material.MaterialTypeParam = 0.5f;
                                material.TextureLayer[0].Texture = texture;
                                material.setFlag(video::EMF_LIGHTING, true);
@@ -1198,7 +1271,7 @@ void GenericCAO::updateTextures(std::string mod)
 
                                // Set material flags and texture
                                video::SMaterial& material = m_meshnode->getMaterial(i);
-                               material.MaterialType = material_type;
+                               material.MaterialType = m_material_type;
                                material.MaterialTypeParam = 0.5f;
                                material.setFlag(video::EMF_LIGHTING, false);
                                material.setFlag(video::EMF_BILINEAR_FILTER, false);
@@ -1262,18 +1335,19 @@ void GenericCAO::updateTextures(std::string mod)
                                        buf->getMaterial().AmbientColor = m_prop.colors[1];
                                        buf->getMaterial().DiffuseColor = m_prop.colors[1];
                                        buf->getMaterial().SpecularColor = m_prop.colors[1];
-                                       setMeshColor(mesh, m_prop.colors[1]);
                                } else if (!m_prop.colors.empty()) {
                                        buf->getMaterial().AmbientColor = m_prop.colors[0];
                                        buf->getMaterial().DiffuseColor = m_prop.colors[0];
                                        buf->getMaterial().SpecularColor = m_prop.colors[0];
-                                       setMeshColor(mesh, m_prop.colors[0]);
                                }
 
                                buf->getMaterial().setFlag(video::EMF_TRILINEAR_FILTER, use_trilinear_filter);
                                buf->getMaterial().setFlag(video::EMF_BILINEAR_FILTER, use_bilinear_filter);
                                buf->getMaterial().setFlag(video::EMF_ANISOTROPIC_FILTER, use_anisotropic_filter);
                        }
+                       // Set mesh color (only if lighting is disabled)
+                       if (!m_prop.colors.empty() && m_glow < 0)
+                               setMeshColor(mesh, m_prop.colors[0]);
                }
        }
 }
@@ -1310,16 +1384,41 @@ void GenericCAO::updateBonePosition()
                return;
 
        m_animated_meshnode->setJointMode(irr::scene::EJUOR_CONTROL); // To write positions to the mesh on render
-       for(std::unordered_map<std::string, core::vector2d<v3f>>::const_iterator
-                       ii = m_bone_position.begin(); ii != m_bone_position.end(); ++ii) {
-               std::string bone_name = (*ii).first;
-               v3f bone_pos = (*ii).second.X;
-               v3f bone_rot = (*ii).second.Y;
+       for (auto &it : m_bone_position) {
+               std::string bone_name = it.first;
                irr::scene::IBoneSceneNode* bone = m_animated_meshnode->getJointNode(bone_name.c_str());
-               if(bone)
-               {
-                       bone->setPosition(bone_pos);
+               if (bone) {
+                       bone->setPosition(it.second.X);
+                       bone->setRotation(it.second.Y);
+               }
+       }
+       
+       // search through bones to find mistakenly rotated bones due to bug in Irrlicht
+       for (u32 i = 0; i < m_animated_meshnode->getJointCount(); ++i) {
+               irr::scene::IBoneSceneNode *bone = m_animated_meshnode->getJointNode(i);
+               if (!bone)
+                       continue;
+
+               //If bone is manually positioned there is no need to perform the bug check
+               bool skip = false;
+               for (auto &it : m_bone_position) {
+                       if (it.first == bone->getName()) {
+                               skip = true;
+                               break;
+                       }
+               }
+               if (skip)
+                       continue;
+
+               // Workaround for Irrlicht bug
+               // We check each bone to see if it has been rotated ~180deg from its expected position due to a bug in Irricht
+               // when using EJUOR_CONTROL joint control. If the bug is detected we update the bone to the proper position
+               // and update the bones transformation.
+               v3f bone_rot = bone->getRelativeTransformation().getRotationDegrees();
+               float offset = fabsf(bone_rot.X - bone->getRotation().X);
+               if (offset > 179.9f && offset < 180.1f) { 
                        bone->setRotation(bone_rot);
+                       bone->updateAbsolutePosition();
                }
        }
 }
@@ -1330,14 +1429,22 @@ void GenericCAO::updateAttachments()
 
        m_attached_to_local = parent && parent->isLocalPlayer();
 
-       if (!parent && m_attachment_parent_id) {
-               //m_is_visible = false; maybe later. needs better handling
-               return;
-       }
+       /*
+       Following cases exist:
+               m_attachment_parent_id == 0 && !parent
+                       This object is not attached
+               m_attachment_parent_id != 0 && parent
+                       This object is attached
+               m_attachment_parent_id != 0 && !parent
+                       This object will be attached as soon the parent is known
+               m_attachment_parent_id == 0 && parent
+                       Impossible case
+       */
 
        if (!parent) { // Detach or don't attach
                if (m_matrixnode) {
-                       v3f old_pos = m_matrixnode->getAbsolutePosition();
+                       v3f old_pos = getPosition();
+
                        m_matrixnode->setParent(m_smgr->getRootSceneNode());
                        getPosRotMatrix().setTranslation(old_pos);
                        m_matrixnode->updateAbsolutePosition();
@@ -1361,11 +1468,23 @@ void GenericCAO::updateAttachments()
                        m_matrixnode->updateAbsolutePosition();
                }
        }
-       if (m_is_local_player) {
-               LocalPlayer *player = m_env->getLocalPlayer();
-               player->isAttached = parent;
-               player->parent = parent;
-       }
+}
+
+bool GenericCAO::visualExpiryRequired(const ObjectProperties &new_) const
+{
+       const ObjectProperties &old = m_prop;
+       // Ordered to compare primitive types before std::vectors
+       return old.backface_culling != new_.backface_culling ||
+               old.initial_sprite_basepos != new_.initial_sprite_basepos ||
+               old.is_visible != new_.is_visible ||
+               old.mesh != new_.mesh ||
+               old.spritediv != new_.spritediv ||
+               old.use_texture_alpha != new_.use_texture_alpha ||
+               old.visual != new_.visual ||
+               old.visual_size != new_.visual_size ||
+               old.wield_item != new_.wield_item ||
+               old.colors != new_.colors ||
+               old.textures != new_.textures;
 }
 
 void GenericCAO::processMessage(const std::string &data)
@@ -1374,15 +1493,22 @@ void GenericCAO::processMessage(const std::string &data)
        std::istringstream is(data, std::ios::binary);
        // command
        u8 cmd = readU8(is);
-       if (cmd == GENERIC_CMD_SET_PROPERTIES) {
-               m_prop = gob_read_set_properties(is);
+       if (cmd == AO_CMD_SET_PROPERTIES) {
+               ObjectProperties newprops;
+               newprops.deSerialize(is);
+
+               // Check what exactly changed
+               bool expire_visuals = visualExpiryRequired(newprops);
+
+               // Apply changes
+               m_prop = std::move(newprops);
 
                m_selection_box = m_prop.selectionbox;
                m_selection_box.MinEdge *= BS;
                m_selection_box.MaxEdge *= BS;
 
-               m_tx_size.X = 1.0 / m_prop.spritediv.X;
-               m_tx_size.Y = 1.0 / m_prop.spritediv.Y;
+               m_tx_size.X = 1.0f / m_prop.spritediv.X;
+               m_tx_size.Y = 1.0f / m_prop.spritediv.Y;
 
                if(!m_initial_tx_basepos_set){
                        m_initial_tx_basepos_set = true;
@@ -1401,19 +1527,21 @@ void GenericCAO::processMessage(const std::string &data)
 
                if ((m_is_player && !m_is_local_player) && m_prop.nametag.empty())
                        m_prop.nametag = m_name;
+               updateNametag();
 
-               expireVisuals();
-       } else if (cmd == GENERIC_CMD_UPDATE_POSITION) {
+               if (expire_visuals) {
+                       expireVisuals();
+               } else {
+                       infostream << "GenericCAO: properties updated but expiring visuals"
+                               << " not necessary" << std::endl;
+               }
+       } else if (cmd == AO_CMD_UPDATE_POSITION) {
                // Not sent by the server if this object is an attachment.
                // We might however get here if the server notices the object being detached before the client.
                m_position = readV3F32(is);
                m_velocity = readV3F32(is);
                m_acceleration = readV3F32(is);
-
-               if (std::fabs(m_prop.automatic_rotate) < 0.001f)
-                       m_rotation = readV3F32(is);
-               else
-                       readV3F32(is);
+               m_rotation = readV3F32(is);
 
                m_rotation = wrapDegrees_0_360_v3f(m_rotation);
                bool do_interpolate = readU8(is);
@@ -1437,16 +1565,16 @@ void GenericCAO::processMessage(const std::string &data)
                }
                rot_translator.update(m_rotation, false, update_interval);
                updateNodePos();
-       } else if (cmd == GENERIC_CMD_SET_TEXTURE_MOD) {
+       } else if (cmd == AO_CMD_SET_TEXTURE_MOD) {
                std::string mod = deSerializeString(is);
 
-               // immediatly reset a engine issued texture modifier if a mod sends a different one
+               // immediately reset a engine issued texture modifier if a mod sends a different one
                if (m_reset_textures_timer > 0) {
                        m_reset_textures_timer = -1;
                        updateTextures(m_previous_texture_modifier);
                }
                updateTextures(mod);
-       } else if (cmd == GENERIC_CMD_SET_SPRITE) {
+       } else if (cmd == AO_CMD_SET_SPRITE) {
                v2s16 p = readV2S16(is);
                int num_frames = readU16(is);
                float framelength = readF32(is);
@@ -1458,7 +1586,7 @@ void GenericCAO::processMessage(const std::string &data)
                m_tx_select_horiz_by_yawpitch = select_horiz_by_yawpitch;
 
                updateTexturePos();
-       } else if (cmd == GENERIC_CMD_SET_PHYSICS_OVERRIDE) {
+       } else if (cmd == AO_CMD_SET_PHYSICS_OVERRIDE) {
                float override_speed = readF32(is);
                float override_jump = readF32(is);
                float override_gravity = readF32(is);
@@ -1478,7 +1606,7 @@ void GenericCAO::processMessage(const std::string &data)
                        player->physics_override_sneak_glitch = sneak_glitch;
                        player->physics_override_new_move = new_move;
                }
-       } else if (cmd == GENERIC_CMD_SET_ANIMATION) {
+       } else if (cmd == AO_CMD_SET_ANIMATION) {
                // TODO: change frames send as v2s32 value
                v2f range = readV2F32(is);
                if (!m_is_local_player) {
@@ -1512,17 +1640,17 @@ void GenericCAO::processMessage(const std::string &data)
                                        updateAnimation();
                        }
                }
-       } else if (cmd == GENERIC_CMD_SET_ANIMATION_SPEED) {
+       } else if (cmd == AO_CMD_SET_ANIMATION_SPEED) {
                m_animation_speed = readF32(is);
                updateAnimationSpeed();
-       } else if (cmd == GENERIC_CMD_SET_BONE_POSITION) {
+       } else if (cmd == AO_CMD_SET_BONE_POSITION) {
                std::string bone = deSerializeString(is);
                v3f position = readV3F32(is);
                v3f rotation = readV3F32(is);
                m_bone_position[bone] = core::vector2d<v3f>(position, rotation);
 
-               updateBonePosition();
-       } else if (cmd == GENERIC_CMD_ATTACH_TO) {
+               // updateBonePosition(); now called every step
+       } else if (cmd == AO_CMD_ATTACH_TO) {
                u16 parent_id = readS16(is);
                std::string bone = deSerializeString(is);
                v3f position = readV3F32(is);
@@ -1533,7 +1661,7 @@ void GenericCAO::processMessage(const std::string &data)
                // localplayer itself can't be attached to localplayer
                if (!m_is_local_player)
                        m_is_visible = !m_attached_to_local;
-       } else if (cmd == GENERIC_CMD_PUNCHED) {
+       } else if (cmd == AO_CMD_PUNCHED) {
                u16 result_hp = readU16(is);
 
                // Use this instead of the send damage to not interfere with prediction
@@ -1546,7 +1674,7 @@ void GenericCAO::processMessage(const std::string &data)
 
                if (damage > 0)
                {
-                       if (m_hp <= 0)
+                       if (m_hp == 0)
                        {
                                // TODO: Execute defined fast response
                                // As there is no definition, make a smoke puff
@@ -1554,22 +1682,22 @@ void GenericCAO::processMessage(const std::string &data)
                                                m_smgr, m_env, m_position,
                                                v2f(m_prop.visual_size.X, m_prop.visual_size.Y) * BS);
                                m_env->addSimpleObject(simple);
-                       } else if (m_reset_textures_timer < 0) {
-                               // TODO: Execute defined fast response
-                               // Flashing shall suffice as there is no definition
+                       } else if (m_reset_textures_timer < 0 && !m_prop.damage_texture_modifier.empty()) {
                                m_reset_textures_timer = 0.05;
                                if(damage >= 2)
                                        m_reset_textures_timer += 0.05 * damage;
-                               updateTextures(m_current_texture_modifier + "^[brighten");
+                               updateTextures(m_current_texture_modifier + m_prop.damage_texture_modifier);
                        }
-               } else {
+               }
+
+               if (m_hp == 0) {
                        // Same as 'Server::DiePlayer'
                        clearParentAttachment();
                        // Same as 'ObjectRef::l_remove'
                        if (!m_is_player)
                                clearChildAttachments();
                }
-       } else if (cmd == GENERIC_CMD_UPDATE_ARMOR_GROUPS) {
+       } else if (cmd == AO_CMD_UPDATE_ARMOR_GROUPS) {
                m_armor_groups.clear();
                int armor_groups_size = readU16(is);
                for(int i=0; i<armor_groups_size; i++)
@@ -1578,22 +1706,14 @@ void GenericCAO::processMessage(const std::string &data)
                        int rating = readS16(is);
                        m_armor_groups[name] = rating;
                }
-       } else if (cmd == GENERIC_CMD_UPDATE_NAMETAG_ATTRIBUTES) {
-               // Deprecated, for backwards compatibility only.
-               readU8(is); // version
-               m_prop.nametag_color = readARGB8(is);
-               if (m_nametag != NULL) {
-                       m_nametag->nametag_color = m_prop.nametag_color;
-                       v3f pos;
-                       pos.Y = m_prop.collisionbox.MaxEdge.Y + 0.3f;
-                       m_nametag->nametag_pos = pos;
-               }
-       } else if (cmd == GENERIC_CMD_SPAWN_INFANT) {
+       } else if (cmd == AO_CMD_SPAWN_INFANT) {
                u16 child_id = readU16(is);
                u8 type = readU8(is); // maybe this will be useful later
                (void)type;
 
                addAttachmentChild(child_id);
+       } else if (cmd == AO_CMD_OBSOLETE1) {
+               // Don't do anything and also don't log a warning
        } else {
                warningstream << FUNCTION_NAME
                        << ": unknown command or outdated client \""
@@ -1629,13 +1749,11 @@ bool GenericCAO::directReportPunch(v3f dir, const ItemStack *punchitem,
                                        v2f(m_prop.visual_size.X, m_prop.visual_size.Y) * BS);
                        m_env->addSimpleObject(simple);
                }
-               // TODO: Execute defined fast response
-               // Flashing shall suffice as there is no definition
-               if (m_reset_textures_timer < 0) {
+               if (m_reset_textures_timer < 0 && !m_prop.damage_texture_modifier.empty()) {
                        m_reset_textures_timer = 0.05;
                        if (result.damage >= 2)
                                m_reset_textures_timer += 0.05 * result.damage;
-                       updateTextures(m_current_texture_modifier + "^[brighten");
+                       updateTextures(m_current_texture_modifier + m_prop.damage_texture_modifier);
                }
        }