[Client] Prevent permanent deletion of player markers on cell changes

apps/openmw/mwgui/mapwindow.cpp

@@ -458,15 +458,6 @@ namespace MWGui
 
         updateMagicMarkers();
         updateCustomMarkers();
-        /*
-            Start of tes3mp addition
-
-            Update player markers when cell changes to fix their locations
-        */
-        updatePlayerMarkers();
-        /*
-            End of tes3mp addition
-        */
     }
 
     void LocalMapBase::requestMapRender(const MWWorld::CellStore *cell)

apps/openmw/mwmp/DedicatedPlayer.cpp

@@ -384,7 +384,9 @@ void DedicatedPlayer::setCell()
         removeMarker();
     // Otherwise, update their marker so the player shows up in the right cell on the world map
     else
-        updateMarker();
+    {
+        enableMarker();
+    }
 
     // If this player is now in a cell that we are the local authority over, we should send them all
     // NPC data in that cell
@@ -408,7 +410,9 @@ void DedicatedPlayer::setCell()
 void DedicatedPlayer::updateMarker()
 {
     if (!markerEnabled)
+    {
         return;
+    }
 
     GUIController *gui = Main::get().getGUIController();
 
@@ -419,7 +423,15 @@ void DedicatedPlayer::updateMarker()
         gui->mPlayerMarkers.addMarker(marker);
     }
     else
+    {
         gui->mPlayerMarkers.addMarker(marker, true);
+    }
+}
+
+void DedicatedPlayer::enableMarker()
+{
+    markerEnabled = true;
+    updateMarker();
 }
 
 void DedicatedPlayer::removeMarker()
@@ -431,18 +443,9 @@ void DedicatedPlayer::removeMarker()
     GUIController *gui = Main::get().getGUIController();
 
     if (gui->mPlayerMarkers.contains(marker))
-        Main::get().getGUIController()->mPlayerMarkers.deleteMarker(marker);
-}
-
-void DedicatedPlayer::setMarkerState(bool state)
-{
-    if (state)
     {
-        markerEnabled = true;
+        Main::get().getGUIController()->mPlayerMarkers.deleteMarker(marker);
-        updateMarker();
     }
-    else
-        removeMarker();
 }
 
 void DedicatedPlayer::playAnimation()
@@ -472,7 +475,7 @@ void DedicatedPlayer::createReference(const std::string& recId)
 
     ESM::CustomMarker mEditingMarker = Main::get().getGUIController()->createMarker(guid);
     marker = mEditingMarker;
-    setMarkerState(true);
+    enableMarker();
 }
 
 void DedicatedPlayer::deleteReference()

apps/openmw/mwmp/DedicatedPlayer.hpp

@@ -47,7 +47,7 @@ namespace mwmp
 
         void updateMarker();
         void removeMarker();
-        void setMarkerState(bool state);
+        void enableMarker();
 
         void playAnimation();
         void playSpeech();

apps/openmw/mwmp/LocalPlayer.cpp

@@ -36,6 +36,7 @@
 #include "LocalPlayer.hpp"
 #include "Main.hpp"
 #include "Networking.hpp"
+#include "PlayerList.hpp"
 #include "CellController.hpp"
 #include "GUIController.hpp"
 #include "MechanicsHelper.hpp"
@@ -463,6 +464,13 @@ void LocalPlayer::updateCell(bool forceUpdate)
         getNetworking()->getPlayerPacket(ID_PLAYER_CELL_CHANGE)->Send();
 
         isChangingRegion = false;
+
+        // If this is an interior cell, are there any other players in it? If so,
+        // enable their markers
+        if (!ptrCell->isExterior())
+        {
+            mwmp::PlayerList::enableMarkers(*ptrCell);
+        }
     }
 }
 

apps/openmw/mwmp/PlayerList.cpp

@@ -92,6 +92,17 @@ bool PlayerList::isDedicatedPlayer(const MWWorld::Ptr &ptr)
     return (getPlayer(ptr) != 0);
 }
 
+void PlayerList::enableMarkers(const ESM::Cell& cell)
+{
+    for (auto &playerEntry : players)
+    {
+        if (Main::get().getCellController()->isSameCell(cell, playerEntry.second->cell))
+        {
+            playerEntry.second->enableMarker();
+        }
+    }
+}
+
 /*
     Go through all DedicatedPlayers checking if their mHitAttemptActorId matches this one
     and set it to -1 if it does

apps/openmw/mwmp/PlayerList.hpp

@@ -37,6 +37,8 @@ namespace mwmp
 
         static bool isDedicatedPlayer(const MWWorld::Ptr &ptr);
 
+        static void enableMarkers(const ESM::Cell& cell);
+
         static void clearHitAttemptActorId(int actorId);
 
     private: