Ticket #3143: 3143_lobby_data_v2.6.03.patch

binaries/data/mods/public/gui/common/functions_utility.js

@@ -67 +67 @@
     return text.substr(0, 255).replace(/\\/g, "\\\\").replace(/\[/g, "\\[");
 }
 
+function unescapeText(text)
+{
+    if (!text)
+        return text;
+    return text.replace(/\\\\/g, "\\").replace(/\\\[/g, "\[");
+}
+
 function translateMapTitle(mapTitle)
 {
     return mapTitle == "random" ? translateWithContext("map selection", "Random") : translate(mapTitle);
@@ -226 +233 @@
 
     for (let playerData of playerDataArray)
     {
-        if (playerData == null || playerData.Civ == "gaia")
+        if (playerData == null || playerData.Civ && playerData.Civ == "gaia")
             continue;
 
         ++playerIdx;
         let teamIdx = playerData.Team;
         let isAI = playerData.AI && playerData.AI != "";
-        let playerState = playerStates && playerStates[playerIdx];
+        let playerState = playerStates && playerStates[playerIdx] || playerData.State;
         let isActive = !playerState || playerState == "active";
+        let isOffline = playerData.Offline;
 
         let playerDescription;
         if (isAI)
@@ -247 +255 @@
         }
         else
         {
-            if (isActive)
-                // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
-                playerDescription = translate("%(playerName)s (%(civ)s)");
+            if (isOffline)
+            {
+                if (playerData.Civ)
+                    if (isActive)
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (OFFLINE, %(civ)s)");
+                    else
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (OFFLINE, %(civ)s, %(state)s)");
+                else
+                    if (isActive)
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (OFFLINE)");
+                    else
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (OFFLINE, %(state)s)");
+            }
             else
-                // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
-                playerDescription = translate("%(playerName)s (%(civ)s, %(state)s)");
+            {
+                if (playerData.Civ)
+                    if (isActive)
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (%(civ)s)");
+                    else
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (%(civ)s, %(state)s)");
+                else
+                    if (isActive)
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s");
+                    else
+                        // Translation: Describe a player in a selected game, f.e. in the replay- or savegame menu
+                        playerDescription = translate("%(playerName)s (%(state)s)");
+            }
         }
 
         // Sort player descriptions by team
@@ -283 +319 @@
     }
 
     let teams = Object.keys(playerDescriptions);
+    if (teams.indexOf("observer") > -1)
+        teams.splice(teams.indexOf("observer"), 1);
 
+    let teamDescription = [];
+
     // If there are no teams, merge all playersDescriptions
     if (teams.length == 1)
-        return playerDescriptions[teams[0]].join("\n") + "\n";
+        teamDescription.push(playerDescriptions[teams[0]].join("\n"));
 
     // If there are teams, merge "Team N:" + playerDescriptions
-    return teams.map(team => {
+    else
+        teamDescription = teamDescription.concat(teams.map(team => {
 
-        let teamCaption = team == -1 ?
-            translate("No Team") :
-            sprintf(translate("Team %(team)s"), { "team": +team + 1 });
+            let teamCaption = team == -1 ?
+                translate("No Team") :
+                sprintf(translate("Team %(team)s"), { "team": +team + 1 });
 
-        // Translation: Describe players of one team in a selected game, f.e. in the replay- or savegame menu or lobby
-        return sprintf(translate("%(team)s:\n%(playerDescriptions)s"), {
-            "team": '[font="sans-bold-14"]' + teamCaption + "[/font]",
-            "playerDescriptions": playerDescriptions[team].join("\n")
-        });
-    }).join("\n\n");
+            // Translation: Describe players of one team in a selected game, f.e. in the replay- or savegame menu or lobby
+            return sprintf(translate("%(team)s:\n%(playerDescriptions)s"), {
+                "team": '[font="sans-bold-14"]' + teamCaption + "[/font]",
+                "playerDescriptions": playerDescriptions[team].join("\n")
+            });
+        }));
+
+    if (playerDescriptions.observer)
+        teamDescription.push(sprintf(translatePlural(
+            '[font="sans-bold-14"]Observer:[/font]\n%(observerDescription)s',
+            '[font="sans-bold-14"]Observers:[/font]\n%(observerDescription)s',
+            playerDescriptions.observer.length),
+            {"observerDescription": playerDescriptions.observer.join("\n")}
+        ));
+
+    return teamDescription.join("\n\n");
 }

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

@@ -676 +676 @@
 {
     if (g_IsController && Engine.HasXmppClient())
     {
-        let playerNames = Object.keys(g_PlayerAssignments).map(guid => g_PlayerAssignments[guid].name);
-        Engine.SendChangeStateGame(playerNames.length, playerNames.join(", "));
+        let clients = formatClients();
+        Engine.SendChangeStateGame(clients.nbp, clients.list);
     }
 
     Engine.SwitchGuiPage("page_loading.xml", {
@@ -1926 +1926 @@
         setReady(false, false);
 }
 
+function formatClients()
+{
+    let numberOfPlayers = 0;
+    let list = [];
+    for (let guid in g_PlayerAssignments)
+    {
+        if (g_GameAttributes.settings.PlayerData[g_PlayerAssignments[guid].player - 1])
+        {
+            ++numberOfPlayers;
+            list.push({
+                "Name": g_PlayerAssignments[guid].name
+            });
+        }
+        else
+            list.push({
+                "Name": g_PlayerAssignments[guid].name,
+                "Team": "observer"
+            });
+    }
+    return {
+        "list": escapeText(JSON.stringify(list)),
+        "nbp": numberOfPlayers
+    };
+}
+
 /**
  * Send the relevant gamesettings to the lobbybot.
  */
@@ -1939 +1964 @@
 
     let mapSize = g_GameAttributes.mapType == "random" ? Engine.GetGUIObjectByName("mapSize").list_data[selectedMapSize] : "Default";
     let victoryCondition = Engine.GetGUIObjectByName("victoryCondition").list[selectedVictoryCondition];
-    let playerNames = Object.keys(g_PlayerAssignments).map(guid => g_PlayerAssignments[guid].name).sort();
+    let clients = formatClients();
 
     let stanza = {
         "name": g_ServerName,
@@ -1949 +1974 @@
         "mapSize": mapSize,
         "mapType": g_GameAttributes.mapType,
         "victoryCondition": victoryCondition,
-        "nbp": Object.keys(g_PlayerAssignments).length || 1,
-        "tnbp": g_GameAttributes.settings.PlayerData.length,
-        "players": playerNames.join(", ")
+        "nbp": clients.nbp,
+        "maxnbp": g_GameAttributes.settings.PlayerData.length,
+        "clients": clients.list,
     };
 
     // Only send the stanza if the relevant settings actually changed

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

@@ -285 +285 @@
         return true;
 
     if (playersNumberFilter.selected != 0 &&
-        game.tnbp != playersNumberFilter.list_data[playersNumberFilter.selected])
+        game.maxnbp != playersNumberFilter.list_data[playersNumberFilter.selected])
         return true;
 
     if (mapTypeFilter.selected != 0 &&
@@ -292 +292 @@
         game.mapType != mapTypeFilter.list_data[mapTypeFilter.selected])
         return true;
 
-    if (!showFullFilter.checked && game.tnbp <= game.nbp)
+    if (!showFullFilter.checked && game.maxnbp <= game.nbp)
         return true;
 
     return false;
@@ -555 +555 @@
             break;
         case 'nPlayers':
             // Compare playercount ratio
-            sortA = a.nbp * b.tnbp;
-            sortB = b.nbp * a.tnbp;
+            sortA = a.nbp * b.maxnbp;
+            sortB = b.nbp * a.maxnbp;
             break;
         case 'status':
         default:
@@ -591 +591 @@
         list_mapName.push(translateMapTitle(game.niceMapName));
         list_mapSize.push(translateMapSize(game.mapSize));
         list_mapType.push(g_MapTypes.Title[mapTypeIdx] || "");
-        list_nPlayers.push(game.nbp + "/" + game.tnbp);
+        list_nPlayers.push(game.nbp + "/" + game.maxnbp);
         list.push(gameName);
         list_data.push(i);
     }
@@ -625 +625 @@
 
     Engine.GetGUIObjectByName("sgMapName").caption = translateMapTitle(game.niceMapName);
     Engine.GetGUIObjectByName("sgNbPlayers").caption = sprintf(
-        translate("Players: %(current)s/%(total)s"), {
+        translate("Players: %(current)s/%(max)s"), {
             "current": game.nbp,
-            "total": game.tnbp
+            "max": game.maxnbp
         });
 
-    Engine.GetGUIObjectByName("sgPlayersNames").caption = game.players;
+    Engine.GetGUIObjectByName("sgPlayersNames").caption = formatPlayerInfo(JSON.parse(unescapeText(game.clients)));
     Engine.GetGUIObjectByName("sgMapSize").caption = translateMapSize(game.mapSize);
 
     let mapTypeIdx = g_MapTypes.Name.indexOf(game.mapType);
@@ -662 +662 @@
 
     let username = g_UserRating ? g_Username + " (" + g_UserRating + ")" : g_Username;
 
-    if (game.state == "init" || game.players.split(", ").indexOf(username) > -1)
+    if (game.state == "init" || JSON.parse(unescapeText(game.clients)).some(player => player.Name == username))
         joinSelectedGame();
     else
         messageBox(

binaries/data/mods/public/gui/session/messages.js

@@ -544 +544 @@
 
     // Update lobby gamestatus
     if (g_IsController && Engine.HasXmppClient())
-    {
-        let players = Object.keys(g_PlayerAssignments).map(guid => g_PlayerAssignments[guid].name);
-        Engine.SendChangeStateGame(Object.keys(g_PlayerAssignments).length, players.join(", "));
-    }
+        sendLobbyReportUpdate();
 }
 
 function onClientJoin(guid)

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

@@ -288 +288 @@
     if (hotloadData)
         g_Selection.selected = hotloadData.selection;
 
+    if (g_IsController && Engine.HasXmppClient())
+        sendLobbyReportUpdate();
+
     onSimulationUpdate();
 
     setTimeout(displayGamestateNotifications, 1000);

binaries/data/mods/public/gui/session/utility_functions.js

@@ -131 +131 @@
     alpha = alpha > 125 ? 125 : alpha;
     return "color:255 0 0 " + alpha;
 }
+
+function sendLobbyReportUpdate()
+{
+    let playerDataArray = JSON.parse(JSON.stringify(g_GameAttributes.settings.PlayerData));
+    let numberOfObservers = 0;
+    let playerStates = {};
+
+    for (let player of g_Players)
+    {
+        playerStates[player].State = player.state;
+        playerStates[player].Offline = player.offline;
+    }
+
+    for (let pAssign of g_PlayerAssignments)
+    {
+        let pData = g_GameAttributes.settings.PlayerData[pAssign.player - 1];
+
+        //if client is observer
+        if (!pData)
+        {
+            playerDataArray.push({
+                "Name": pAssign.name,
+                "Team": "observer"
+            });
+            ++numberOfObservers;
+        }
+    }
+
+    for (let playerData of playerDataArray)
+    {
+        delete playerData.Civ;
+        delete playerData.Color;
+
+        if (!g_GameAttributes.settings.LockTeams && playerData.Team != "observer")
+            delete playerData.Team;
+
+        let pState = playerStates[playerData.Name];
+        if (pState)
+        {
+            playerData.State = pState.State;
+            playerData.Offline = pState.Offline;
+        }
+    }
+
+    Engine.SendChangeStateGame(g_PlayerAssignments.length - numberOfObservers, escapeText(JSON.stringify(playerDataArray)));
+}

source/lobby/IXmppClient.h

@@ -1 @@
-/* Copyright (C) 2015 Wildfire Games.
+/* Copyright (C) 2016 Wildfire Games.
  * This file is part of 0 A.D.
  *
  * 0 A.D. is free software: you can redistribute it and/or modify
@@ -38 +38 @@
     virtual void SendIqGameReport(ScriptInterface& scriptInterface, JS::HandleValue data) = 0;
     virtual void SendIqRegisterGame(ScriptInterface& scriptInterface, JS::HandleValue data) = 0;
     virtual void SendIqUnregisterGame() = 0;
-    virtual void SendIqChangeStateGame(const std::string& nbp, const std::string& players) = 0;
+    virtual void SendIqChangeStateGame(const std::string& nbp, const std::string& clients) = 0;
     virtual void SetNick(const std::string& nick) = 0;
     virtual void GetNick(std::string& nick) = 0;
     virtual void kick(const std::string& nick, const std::string& reason) = 0;

source/lobby/scripting/JSInterface_Lobby.cpp

@@ -1 @@
-/* Copyright (C) 2014 Wildfire Games.
+/* Copyright (C) 2016 Wildfire Games.
  * This file is part of 0 A.D.
  *
  * 0 A.D. is free software: you can redistribute it and/or modify
@@ -171 +171 @@
     g_XmppClient->SendIqUnregisterGame();
 }
 
-void JSI_Lobby::SendChangeStateGame(ScriptInterface::CxPrivate* UNUSED(pCxPrivate), const std::wstring& nbp, const std::wstring& players)
+void JSI_Lobby::SendChangeStateGame(ScriptInterface::CxPrivate* UNUSED(pCxPrivate), const std::wstring& nbp, const std::wstring& clients)
 {
     if (!g_XmppClient)
         return;
-    g_XmppClient->SendIqChangeStateGame(utf8_from_wstring(nbp), utf8_from_wstring(players));
+    g_XmppClient->SendIqChangeStateGame(utf8_from_wstring(nbp), utf8_from_wstring(clients));
 }
 
 JS::Value JSI_Lobby::GetPlayerList(ScriptInterface::CxPrivate* pCxPrivate)

source/lobby/scripting/JSInterface_Lobby.h

@@ -1 @@
-/* Copyright (C) 2014 Wildfire Games.
+/* Copyright (C) 2016 Wildfire Games.
  * This file is part of 0 A.D.
  *
  * 0 A.D. is free software: you can redistribute it and/or modify
@@ -43 +43 @@
     void SendGameReport(ScriptInterface::CxPrivate* pCxPrivate, JS::HandleValue data);
     void SendRegisterGame(ScriptInterface::CxPrivate* pCxPrivate, JS::HandleValue data);
     void SendUnregisterGame(ScriptInterface::CxPrivate* pCxPrivate);
-    void SendChangeStateGame(ScriptInterface::CxPrivate* pCxPrivate, const std::wstring& nbp, const std::wstring& players);
+    void SendChangeStateGame(ScriptInterface::CxPrivate* pCxPrivate, const std::wstring& nbp, const std::wstring& clients);
     JS::Value GetPlayerList(ScriptInterface::CxPrivate* pCxPrivate);
     void LobbyClearPresenceUpdates(ScriptInterface::CxPrivate* pCxPrivate);
     int LobbyGetMucMessageCount(ScriptInterface::CxPrivate* pCxPrivate);

source/lobby/XmppClient.cpp

@@ -408 +408 @@
  * decides which - but we need to update the current players that are
  * in-game so the server can make the calculation.
  */
-void XmppClient::SendIqChangeStateGame(const std::string& nbp, const std::string& players)
+void XmppClient::SendIqChangeStateGame(const std::string& nbp, const std::string& clients)
 {
     glooxwrapper::JID xpartamuppJid(m_xpartamuppId);
 
@@ -417 +417 @@
     g->m_Command = "changestate";
     glooxwrapper::Tag* game = glooxwrapper::Tag::allocate("game");
     game->addAttribute("nbp", nbp);
-    game->addAttribute("players", players);
+    game->addAttribute("clients", clients);
     g->m_GameList.emplace_back(game);
 
     glooxwrapper::IQ iq(gloox::IQ::Set, xpartamuppJid);
@@ -502 +502 @@
     JSAutoRequest rq(cx);
 
     scriptInterface.Eval("([])", ret);
-    const char* stats[] = { "name", "ip", "port", "state", "nbp", "tnbp", "players", "mapName", "niceMapName", "mapSize", "mapType", "victoryCondition" };
+    const char* stats[] = { "name", "ip", "port", "state", "nbp", "maxnbp", "clients", "mapName", "niceMapName", "mapSize", "mapType", "victoryCondition" };
     for(const glooxwrapper::Tag* const& t : m_GameList)
     {
         JS::RootedValue game(cx);

source/lobby/XmppClient.h

@@ -68 +68 @@
     void SendIqGameReport(ScriptInterface& scriptInterface, JS::HandleValue data);
     void SendIqRegisterGame(ScriptInterface& scriptInterface, JS::HandleValue data);
     void SendIqUnregisterGame();
-    void SendIqChangeStateGame(const std::string& nbp, const std::string& players);
+    void SendIqChangeStateGame(const std::string& nbp, const std::string& clients);
     void SetNick(const std::string& nick);
     void GetNick(std::string& nick);
     void kick(const std::string& nick, const std::string& reason);