Don't allow setting a new return position if there already is one and we haven't reached it yet

apps/openmw/mwmechanics/aiwander.cpp

@@ -341,26 +341,29 @@ namespace MWMechanics
             mChooseAction = false;
             mIdleNow = false;
 
-            Ogre::Vector3 destNodePos = mReturnPosition;
+            if (!mPathFinder.isPathConstructed())
+            {
+                Ogre::Vector3 destNodePos = mReturnPosition;
 
-            ESM::Pathgrid::Point dest;
+                ESM::Pathgrid::Point dest;
-            dest.mX = destNodePos[0];
+                dest.mX = destNodePos[0];
-            dest.mY = destNodePos[1];
+                dest.mY = destNodePos[1];
-            dest.mZ = destNodePos[2];
+                dest.mZ = destNodePos[2];
 
-            // actor position is already in world co-ordinates
+                // actor position is already in world co-ordinates
-            ESM::Pathgrid::Point start;
+                ESM::Pathgrid::Point start;
-            start.mX = pos.pos[0];
+                start.mX = pos.pos[0];
-            start.mY = pos.pos[1];
+                start.mY = pos.pos[1];
-            start.mZ = pos.pos[2];
+                start.mZ = pos.pos[2];
 
-            // don't take shortcuts for wandering
+                // don't take shortcuts for wandering
-            mPathFinder.buildPath(start, dest, actor.getCell(), false);
+                mPathFinder.buildPath(start, dest, actor.getCell(), false);
 
-            if(mPathFinder.isPathConstructed())
+                if(mPathFinder.isPathConstructed())
-            {
+                {
-                mMoveNow = false;
+                    mMoveNow = false;
-                mWalking = true;
+                    mWalking = true;
+                }
             }
         }
 
@@ -529,6 +532,7 @@ namespace MWMechanics
             mMoveNow = false;
             mWalking = false;
             mChooseAction = true;
+            mHasReturnPosition = false;
         }
 
         return false; // AiWander package not yet completed
@@ -615,7 +619,11 @@ namespace MWMechanics
 
     void AiWander::setReturnPosition(const Ogre::Vector3& position)
     {
-        mHasReturnPosition = true; mReturnPosition = position;
+        if (!mHasReturnPosition)
+        {
+            mHasReturnPosition = true;
+            mReturnPosition = position;
+        }
     }
 
     void AiWander::getRandomIdle()