From e8bc0d9fd1be148f111f2c560e563e0552fc9f50 Mon Sep 17 00:00:00 2001
From: Andrei Karas <akaras@inbox.ru>
Date: Mon, 14 Mar 2011 04:35:05 +0200
Subject: Fixing direction bug in moving.

While player moving server dont send player direction.
This autofixing in nextTile method, but if this method was not
called, then player have incorrect direction.

Also move checking direction 8 from beinghandler.cpp
to messagein.cpp like was done in mana client.
---
 src/net/tmwa/beinghandler.cpp | 28 ++++++++++++++++++----------
 1 file changed, 18 insertions(+), 10 deletions(-)

(limited to 'src/net/tmwa/beinghandler.cpp')

diff --git a/src/net/tmwa/beinghandler.cpp b/src/net/tmwa/beinghandler.cpp
index 6196afb09..2d0ae3c70 100644
--- a/src/net/tmwa/beinghandler.cpp
+++ b/src/net/tmwa/beinghandler.cpp
@@ -348,7 +348,7 @@ void BeingHandler::handleMessage(Net::MessageIn &msg)
                     }
                 }
 
-                dstBeing->setDirection(getDirection(dir));
+                dstBeing->setDirection(dir);
             }
 
             msg.readInt8();   // unknown
@@ -825,7 +825,7 @@ void BeingHandler::handleMessage(Net::MessageIn &msg)
 
             msg.readInt16(); // unused
 
-            unsigned char dir = getDirection(msg.readInt8());
+            unsigned char dir = msg.readInt8();
             dstBeing->setDirection(dir);
             if (player_node)
                 player_node->imitateDirection(dstBeing, dir);
@@ -938,6 +938,22 @@ void BeingHandler::handleMessage(Net::MessageIn &msg)
                 dstBeing->setTileCoords(srcX, srcY);
                 dstBeing->setDestination(dstX, dstY);
 
+                // because server dont send direction in move packet,
+                // we fixing it
+
+                int dir = 0;
+                if (dstX > srcX)
+                    dir |= Being::RIGHT;
+                else if (dstX < srcX)
+                    dir |= Being::LEFT;
+                if (dstY > srcY)
+                    dir |= Being::DOWN;
+                else if (dstY < srcY)
+                    dir |= Being::UP;
+
+                if (dir)
+                    dstBeing->setDirection(dir);
+
                 if (player_node->getCurrentAction() != Being::STAND)
                     player_node->imitateAction(dstBeing, Being::STAND);
                 if (player_node->getDirection() != dstBeing->getDirection())
@@ -952,7 +968,6 @@ void BeingHandler::handleMessage(Net::MessageIn &msg)
                 Uint16 x, y;
                 msg.readCoordinates(x, y, dir);
                 dstBeing->setTileCoords(x, y);
-                dir = getDirection(dir);
                 dstBeing->setDirection(dir);
 
                 player_node->imitateDirection(dstBeing, dir);
@@ -1142,13 +1157,6 @@ void BeingHandler::handleMessage(Net::MessageIn &msg)
     }
 }
 
-Uint8 BeingHandler::getDirection(Uint8 dir)
-{
-    if (dir == 0)
-        dir = 8;
-    return dir;
-}
-
 void BeingHandler::undress(Being *being)
 {
     being->setSprite(SPRITE_BOTTOMCLOTHES, 0);
-- 
cgit v1.2.3-70-g09d2