Ticket #3171: t3171_clear_muc_messages_v5_16789.patch

binaries/data/mods/public/gui/lobby/lobby.js

@@ -12 +12 @@
 var g_mapSizes = {};
 const g_mapTypesText = [translateWithContext("map", "Skirmish"), translateWithContext("map", "Random"), translate("Scenario")];
 const g_mapTypes = ["skirmish", "random", "scenario"];
 var g_userRating = ""; // Rating of user, defaults to Unrated
 var g_modPrefix = "@";
-var g_joined = false;
 // Block spammers for 30 seconds.
 var SPAM_BLOCK_LENGTH = 30;
 
 ////////////////////////////////////////////////////////////////////////////////////////////////
 
@@ -701 +700 @@
             case "join":
                 if (nick == g_Name)
                 {
                     // We just joined, we need to get the full player list
                     [playerList, presenceList, nickList, ratingList] = updatePlayerList();
-                    // Don't display any joins until our join request bounces back
-                    // Our join message should be the last one as we just got added to the stack
-                    g_joined = true;
-                    break;
                 }
-                else if (g_joined)
+                else
                 {
                     var [name, status, rating] = formatPlayerListEntry(nick, presence, "-");
                     playerList.push(name);
                     presenceList.push(status);
                     nickList.push(nick);

source/lobby/XmppClient.cpp

@@ -492 +492 @@
  *
  * @return A JS array containing all known players and their presences
  */
 void XmppClient::GUIGetPlayerList(ScriptInterface& scriptInterface, JS::MutableHandleValue ret)
 {
+    // Clear outdated presence updates
+    std::remove_if(m_GuiMessageQueue.begin(), m_GuiMessageQueue.end(),
+        [](XmppClient::GUIMessage& message)
+        {
+            return message.type == L"muc";
+        }
+    );
+
     JSContext* cx = scriptInterface.GetContext();
     JSAutoRequest rq(cx);
     
     scriptInterface.Eval("([])", ret);