Ticket #2447: readyv4.patch

binaries/data/mods/public/gui/common/modern/sprites.xml

@@ -296 +296 @@
             size="0 0 22 22"
         />
     </sprite>
+    <sprite name="ModernCheckOn">
+        <image  texture="global/modern/checkbox-on.png"
+            real_texture_placement="0 0 32 32"
+        />
+    </sprite>
+    <sprite name="ModernCheckOff">
+        <image  texture="global/modern/checkbox-off.png"
+            real_texture_placement="0 0 32 32"
+        />
+    </sprite>
 </sprites>

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

@@ -31 +31 @@
 // (and therefore shouldn't send further messages to the network)
 var g_IsInGuiUpdate;
 
+// Is this user ready
+var g_IsReady;
+
+// If no one has changed ready status, we have no need to spam the settings changed message.
+// 2 - Host's initial ready, suppressed settings message, 1 - Will show settings message, <=0 - Suppressed settings message
+var g_ReadyChanged = 2;
+
 var g_PlayerAssignments = {};
 
 // Default game setup attributes
@@ -97 +104 @@
     {
         cancelButton.tooltip = "Return to the lobby."
     }
-
 }
 
 // Called after the map data is loaded and cached
@@ -412 +418 @@
         break;
 
     case "players":
+        var resetReady = false;
+        var newPlayer = "";
         // Find and report all joinings/leavings
         for (var host in message.hosts)
+        {
             if (! g_PlayerAssignments[host])
+            {
                 addChatMessage({ "type": "connect", "username": message.hosts[host].name });
+                newPlayer = host;
+            }
+        }
 
         for (var host in g_PlayerAssignments)
+        {
             if (! message.hosts[host])
+            {
                 addChatMessage({ "type": "disconnect", "guid": host });
-
+                playersChanged = "leave";
+                if (g_PlayerAssignments[host] != -1)
+                    resetReady = true; // Observers shouldn't reset ready.
+            }
+        }
+        
         // Update the player list
         g_PlayerAssignments = message.hosts;
         updatePlayerList();
-
+        if (g_PlayerAssignments[newPlayer])
+        {
+            if (g_PlayerAssignments[newPlayer].player != -1)
+                resetReady = true;
+        }
+        
+        if (resetReady)
+            resetReadyData(); // Observers shouldn't reset ready.
+        updateReadyUI();        
         if (g_IsController)
             sendRegisterGameStanza();
         break;
@@ -447 +475 @@
         addChatMessage({ "type": "message", "guid": message.guid, "text": message.text });
         break;
 
+    // Singular client to host message
+    case "ready":
+        g_ReadyChanged -= 1;
+        warn(String(g_ReadyChanged));
+        if (g_ReadyChanged < 1)
+        {
+            if (parseInt(message.text) == 1)
+                addChatMessage({ "type": "ready", "guid": message.guid, "text": "is ready!"});
+            else
+                addChatMessage({ "type": "ready", "guid": message.guid, "text": "is not ready."});
+        }
+        if (!g_IsController)
+            break;
+        g_PlayerAssignments[message.guid].status = (parseInt(message.text) == 1);
+        Engine.AssignNetworkPlayerStatus(parseInt(message.text), message.guid);
+        updateReadyUI();
+        break;
+
     default:
         error("Unrecognised net message type "+message.type);
     }
@@ -705 +751 @@
             if (g_IsNetworked)
                 Engine.AssignNetworkPlayer(player, "");
             else
-                g_PlayerAssignments = { "local": { "name": "You", "player": 1, "civ": "", "team": -1} };
+                g_PlayerAssignments = { "local": { "name": "You", "player": 1, "civ": "", "team": -1, "ready": 0} };
         }
     }
 
@@ -822 +868 @@
     // Reset player assignments on map change
     if (!g_IsNetworked)
     {   // Slot 1
-        g_PlayerAssignments = { "local": { "name": "You", "player": 1, "civ": "", "team": -1} };
+        g_PlayerAssignments = { "local": { "name": "You", "player": 1, "civ": "", "team": -1, "ready": 0} };
     }
     else
     {
@@ -842 +888 @@
 
 function launchGame()
 {
+    if (g_IsNetworked && !g_IsReady)
+    {
+        for (var guid in g_PlayerAssignments) 
+        {
+            if (g_PlayerAssignments[guid].name.split(" (")[0] == Engine.ConfigDB_GetValue("user", "playername") && g_PlayerAssignments[guid].player == -1) 
+                break; // Observers don't need to be ready.
+            else
+                return; // Host may have changed their own code.
+        }
+    }
     if (g_IsNetworked && !g_IsController)
     {
         error("Only host can start game");
@@ -1184 +1240 @@
         if (i >= numPlayers)
             continue;
 
-        var pName = Engine.GetGUIObjectByName("playerName["+i+"]");
+        var pName = Engine.GetGUIObjectByName("playerName["+i+"]"); 
         var pCiv = Engine.GetGUIObjectByName("playerCiv["+i+"]");
         var pCivText = Engine.GetGUIObjectByName("playerCivText["+i+"]");
         var pTeam = Engine.GetGUIObjectByName("playerTeam["+i+"]");
@@ -1198 +1254 @@
         // Common to all game types
         var color = iColorToString(getSetting(pData, pDefs, "Colour"));
         pColor.sprite = "colour:"+color+" 100";
-        pName.caption = getSetting(pData, pDefs, "Name");
+        pName.caption = getSetting(pData, pDefs, "Name"); 
 
         var team = getSetting(pData, pDefs, "Team");
         var civ = getSetting(pData, pDefs, "Civ");
@@ -1235 +1291 @@
 
     // Game attributes include AI settings, so update the player list
     updatePlayerList();
+
+    // We should have everyone confirm that the new settings are acceptable.
+    resetReadyData();   
 }
 
 function updateGameAttributes()
@@ -1404 +1463 @@
                         swapPlayers(guid, playerSlot);
 
                     Engine.SetNetworkGameAttributes(g_GameAttributes);
+                    updateReadyUI();
                 }
             };
         }
@@ -1485 +1545 @@
 
 function addChatMessage(msg)
 {
-    var username = escapeText(msg.username || g_PlayerAssignments[msg.guid].name);
-    var message = escapeText(msg.text);
+    var username = "";
+    if (msg.username || g_PlayerAssignments[msg.guid])
+        username = escapeText(msg.username || g_PlayerAssignments[msg.guid].name);
 
+    var message = "";
+    if (msg.text)
+        message = escapeText(msg.text);
+
     // TODO: Maybe host should have distinct font/color?
     var color = "white";
 
-    if (g_PlayerAssignments[msg.guid] && g_PlayerAssignments[msg.guid].player != -1)
-    {   // Valid player who has been assigned - get player colour
-        var player = g_PlayerAssignments[msg.guid].player - 1;
-        var mapName = g_GameAttributes.map;
-        var mapData = loadMapData(mapName);
-        var mapSettings = (mapData && mapData.settings ? mapData.settings : {});
-        var pData = mapSettings.PlayerData ? mapSettings.PlayerData[player] : {};
-        var pDefs = g_DefaultPlayerData ? g_DefaultPlayerData[player] : {};
+    if (msg.guid)
+    {
+        if (g_PlayerAssignments[msg.guid] && g_PlayerAssignments[msg.guid].player != -1)
+        {   // Valid player who has been assigned - get player colour
+            var player = g_PlayerAssignments[msg.guid].player - 1;
+            var mapName = g_GameAttributes.map;
+            var mapData = loadMapData(mapName);
+            var mapSettings = (mapData && mapData.settings ? mapData.settings : {});
+            var pData = mapSettings.PlayerData ? mapSettings.PlayerData[player] : {};
+            var pDefs = g_DefaultPlayerData ? g_DefaultPlayerData[player] : {};
 
-        color = iColorToString(getSetting(pData, pDefs, "Colour"));
+            color = iColorToString(getSetting(pData, pDefs, "Colour"));
+        }
     }
 
     var formatted;
@@ -1518 +1586 @@
         formatted = '[font="serif-bold-13"]<[color="'+ color +'"]' + username + '[/color]>[/font] ' + message;
         break;
 
+    case "ready":
+        formatted = '[font="serif-bold-13"][color="'+ color +'"]*' + username + " " + message + '[/color][/font]';
+        break;
+
+    case "settings":
+        formatted = '[font="serif-bold-13"]*The host has changed the settings.[/font]';
+        break;
+
     default:
         error("Invalid chat message '" + uneval(msg) + "'");
         return;
@@ -1534 +1610 @@
     Engine.GetGUIObjectByName("moreOptions").hidden = !Engine.GetGUIObjectByName("moreOptions").hidden;
 }
 
+function toggleReady()
+{
+    g_IsReady = !g_IsReady;
+    if (g_IsReady)
+    {
+        Engine.SendNetworkReady(1);
+        Engine.GetGUIObjectByName("startButton").caption = "I'm not ready.";
+    }
+    else
+    {
+        Engine.SendNetworkReady(0);
+        Engine.GetGUIObjectByName("startButton").caption = "I'm ready!";
+    }
+}
+
+function updateReadyUI()
+{   
+    var allReady = true;
+    for (var guid in g_PlayerAssignments)
+    {
+        var pData = g_GameAttributes.settings.PlayerData ? g_GameAttributes.settings.PlayerData[g_PlayerAssignments[guid].player - 1] : {};
+        var pDefs = g_DefaultPlayerData ? g_DefaultPlayerData[g_PlayerAssignments[guid].player - 1] : {};
+        // We don't really care whether observers are ready.
+        if (g_PlayerAssignments[guid].player == -1)
+            return;     
+        if (g_PlayerAssignments[guid].status || !g_IsNetworked)
+        {
+            // Engine.GetGUIObjectByName("playerReadySprite[" + String(g_PlayerAssignments[guid].player - 1) + "]").sprite = "ModernCheckOn";
+            Engine.GetGUIObjectByName("playerName[" + String(g_PlayerAssignments[guid].player - 1)+"]").caption = '[color="0 255 0"]' + getSetting(pData, pDefs, "Name") + '[/color]';
+        }
+        else
+        {
+            // Engine.GetGUIObjectByName("playerReadySprite[" + String(g_PlayerAssignments[guid].player - 1) + "]").sprite = "ModernCheckOff";
+            Engine.GetGUIObjectByName("playerName[" + String(g_PlayerAssignments[guid].player - 1)+"]").caption = getSetting(pData, pDefs, "Name");
+            allReady = false;
+        }
+    }
+    // AIs are always ready.
+    for (var playerid = 0; playerid < MAX_PLAYERS; playerid++)
+    {       
+        if (g_GameAttributes.settings.PlayerData[playerid])
+        {
+            var pData = g_GameAttributes.settings.PlayerData ? g_GameAttributes.settings.PlayerData[playerid] : {};
+            var pDefs = g_DefaultPlayerData ? g_DefaultPlayerData[playerid] : {};
+            if (g_GameAttributes.settings.PlayerData[playerid].AI != "" || g_GameAttributes.settings.PlayerData[playerid].Name == "Unassigned")
+            {
+                // Engine.GetGUIObjectByName("playerReadySprite[" + String(playerid) + "]").sprite = "ModernCheckOn";
+                Engine.GetGUIObjectByName("playerName[" + String(playerid)+"]").caption = '[color="0 255 0"]' + getSetting(pData, pDefs, "Name") + '[/color]';
+            }
+        }
+    }
+
+    // The host is not allowed to start until everyone is ready.
+    if (g_IsNetworked && g_IsController)
+        Engine.GetGUIObjectByName("startGame").enabled = allReady;
+}
+
+function resetReadyData()
+{
+    if (g_ReadyChanged < 1)
+    {
+        addChatMessage({ "type": "settings"});
+    }
+    g_ReadyChanged = 2;
+    if (g_IsNetworked && g_IsController)
+    {
+        Engine.ClearAllPlayerReady();
+        g_IsReady = true;
+        Engine.SendNetworkReady(1);
+    }
+    else
+        g_IsReady = false;
+}
 ////////////////////////////////////////////////////////////////////////////////////////////////
 // Basic map filters API
 

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

@@ -14 +14 @@
         <object style="TitleText" type="text" size="50%-128 4 50%+128 36">
             Match Setup
         </object>
-    
+
         <object type="image" style="ModernDialog" size="50%-190 50%-80 50%+190 50%+80" name="loadingWindow">
 
             <object type="text" style="TitleText" size="50%-128 0%-16 50%+128 16">
@@ -32 +32 @@
             <action on="Tick">
                 onTick();
             </action>
-            
+
             <!-- Number of Players-->
             <object size="24 26 224 54">
 
@@ -40 +40 @@
                 <object size="0 0 150 28">
                     <object size="0 0 100% 100%" type="text" style="ModernRightLabelText">Number of players:</object>
                 </object>
-                    
+
                 <!-- Number of Players-->
                 <object size="150 0 200 28">
                     <object name="numPlayersText" size="0 0 100% 100%" type="text" style="ModernLeftLabelText"/>
@@ -53 +53 @@
                         <action on="SelectionChange">selectNumPlayers(this.list_data[this.selected]);</action>
                     </object>
                 </object>
+            </object>
 
-            </object>
-            
             <!-- Player assignments -->
             <object size="24 59 100%-440 358" type="image" sprite="ModernDarkBoxGold" name="playerAssignmentsPannel">
                 <object size="0 6 100% 30">
                     <object name="playerNameHeading" type="text" style="ModernLabelText" size="0 0 20% 100%">Player Name</object>
-                    <object name="playerPlacementHeading" type="text" style="ModernLabelText" size="20%+5 0 50% 100%">Player Placement</object>
-                    <object name="playerCivHeading" type="text" style="ModernLabelText" size="50%+65 0 85% 100%">Civilization</object>
+                    <object name="playerPlacementHeading" type="text" style="ModernLabelText" size="20% 0 50%+35 100%">Player Placement</object>
+                    <object name="playerCivHeading" type="text" style="ModernLabelText" size="50%+65 0 85%-26 100%">Civilization</object>
                     <object name="civInfoButton"
                         type="button"
                         sprite="iconInfoGold"
                         sprite_over="iconInfoWhite"
-                        size="82%-8 0 82%+8 16"
+                        size="67.5%+68 0 67.5%+84 16"
                         tooltip_style="onscreenToolTip"
                         tooltip="View civilization info"
                     >
@@ -75 +74 @@
                         ]]></action>
                     </object>
                     <object name="playerTeamHeading" type="text" style="ModernLabelText" size="85%+5 0 100%-5 100%">Team</object>
+                    <!--<object name="playerReadyHeading" type="text" style="ModernLabelText" size="90%+5 0 100%-5 100%">Ready</object>-->
                 </object>
                 <object size="1 36 100%-1 100%">
                     <repeat count="8">
                         <object name="playerBox[n]" size="0 0 100% 32" hidden="true">
                             <object name="playerColour[n]" type="image" size="0 0 100% 100%"/>
-                            <object name="playerName[n]" type="text" style="ModernLabelText" size="0 2 20% 30"/>
-                            <object name="playerAssignment[n]" type="dropdown" style="ModernDropDown" size="20%+5 2 50% 30" tooltip_style="onscreenToolTip" tooltip="Select player."/>
-                            <object name="playerConfig[n]" type="button" style="StoneButton" size="50%+5 6 50%+60 26"
+                            <object name="playerName[n]" type="text" style="ModernLabelText" size="0 2 20% 30"/> 
+                            <object name="playerAssignment[n]" type="dropdown" style="ModernDropDown" size="20% 2 50%+35 30" tooltip_style="onscreenToolTip" tooltip="Select player."/>
+                            <object name="playerConfig[n]" type="button" style="StoneButton" size="50%+40 6 50%+60 26"
                                 tooltip_style="onscreenToolTip"
                                 tooltip="Configure AI settings."
                                 font="serif-bold-stroke-12"
-                            >Settings</object>
+                            >O</object>
                             <object name="playerCiv[n]" type="dropdown" style="ModernDropDown" size="50%+65 2 85% 30" tooltip_style="onscreenToolTip" tooltip="Select player's civilization."/>
                             <object name="playerCivText[n]" type="text" style="ModernLabelText" size="50%+65 0 85% 30"/>
                             <object name="playerTeam[n]" type="dropdown" style="ModernDropDown" size="85%+5 2 100%-5 30" tooltip_style="onscreenToolTip" tooltip="Select player's team."/>
                             <object name="playerTeamText[n]" type="text" style="ModernLabelText" size="85%+5 0 100%-5 100%"/>
+                            <!--<object name="playerReadySprite[n]" type="image" sprite="ModernCheckOff" size="95%-16 0 95%+16 32"/> -->
                         </object>
                     </repeat>
                 </object>
             </object>
-            <object size="24 64 100%-460 358" type="image" sprite="CoverFillDark" name="playerAssignmentsPannelCover" hidden="true"/>
+            <object size="24 64 100%-460 358" type="image" sprite="CoverFillDark" name="playerAssignmentsPanelCover" hidden="true"/>
             <!-- Map selection -->
             
             
@@ -112 +113 @@
                 <object name="mapSelectionText" type="text" style="ModernLeftLabelText" size="0 64 100% 94"  hidden="true"/>
                 <object name="mapSizeText" type="text" style="ModernLeftLabelText" size="0 96 100% 126" hidden="true"/>
             </object>
-            
+
             <object name="mapTypeSelection"
                 type="dropdown"
                 style="ModernDropDown"
@@ -159 +160 @@
             </object>
 
             <!-- Chat window -->
-            <object name="chatPanel" size="24 370 100%-440 100%-60" type="image" sprite="ModernDarkBoxGold">
+            <object name="chatPanel" size="24 370 100%-440 100%-58" type="image" sprite="ModernDarkBoxGold">
                 <object name="chatText" size="2 2 100%-2 100%-26" type="text" style="ChatPanel"/>
 
                 <object name="chatInput" size="4 100%-24 100%-76 100%-4" type="input" style="ModernInput">
@@ -179 +180 @@
                 textcolor="white"
                 sprite="BackgroundTranslucent"
                 hidden="true"
-                size="100%-700 100%-56 100%-312 100%-24"
+                size="25 100%-56 100%-445 100%-24"
             >[Tooltip text]</object>
 
-            <!-- Start Button -->
+            <!-- Start/Ready Button -->
             <object
                 name="startGame"
                 type="button"
                 style="StoneButton"
-                size="100%-308 100%-52 100%-168 100%-24"
+                size="100%-285 100%-52 100%-165 100%-24"
                 tooltip_style="onscreenToolTip"
                 tooltip="Start a new game with the current settings."
                 enabled="false"
             >
                 Start game!
-                <action on="Press">launchGame();</action>
+                <action on="Press">
+                    if (g_IsController)
+                        launchGame();
+                    else
+                        toggleReady();</action>
             </object>
 
             <!-- Cancel Button -->
@@ -202 +207 @@
                 caption="Back"
                 type="button"
                 style="StoneButton"
-                size="100%-164 100%-52 100%-24 100%-24"
+                size="100%-145 100%-52 100%-25 100%-24"
                 tooltip_style="onscreenToolTip"
             >
                 <action on="Press">
@@ -215 +220 @@
                     ]]>
                 </action>
             </object>
-            
+
             <!-- Options -->
             <object name="gameOptionsBox" size="100%-425 497 100%-25 525">
                 <!-- More Options Button -->

source/gui/scripting/ScriptFunctions.cpp

@@ -343 +343 @@
     g_NetServer->AssignPlayer(playerID, guid);
 }
 
+void AssignNetworkPlayerStatus(ScriptInterface::CxPrivate* UNUSED(pCxPrivate), int ready, std::string guid)
+{
+    ENSURE(g_NetServer);
+
+    g_NetServer->ReadyPlayer(ready, guid);
+}
+
+void ClearAllPlayerReady (ScriptInterface::CxPrivate* UNUSED(pCxPrivate))
+{
+    ENSURE(g_NetServer);
+
+    g_NetServer->ClearAllPlayerReady();
+}
+
 void SendNetworkChat(ScriptInterface::CxPrivate* UNUSED(pCxPrivate), std::wstring message)
 {
     ENSURE(g_NetClient);
@@ -350 +364 @@
     g_NetClient->SendChatMessage(message);
 }
 
+void SendNetworkReady(ScriptInterface::CxPrivate* UNUSED(pCxPrivate), int message)
+{
+    ENSURE(g_NetClient);
+
+    g_NetClient->SendReadyMessage(message);
+}
+
 std::vector<CScriptValRooted> GetAIs(ScriptInterface::CxPrivate* pCxPrivate)
 {
     return ICmpAIManager::GetAIs(*(pCxPrivate->pScriptInterface));
@@ -838 +859 @@
     scriptInterface.RegisterFunction<CScriptVal, &PollNetworkClient>("PollNetworkClient");
     scriptInterface.RegisterFunction<void, CScriptVal, &SetNetworkGameAttributes>("SetNetworkGameAttributes");
     scriptInterface.RegisterFunction<void, int, std::string, &AssignNetworkPlayer>("AssignNetworkPlayer");
+    scriptInterface.RegisterFunction<void, int, std::string, &AssignNetworkPlayerStatus>("AssignNetworkPlayerStatus");
+    scriptInterface.RegisterFunction<void, &ClearAllPlayerReady>("ClearAllPlayerReady");
     scriptInterface.RegisterFunction<void, std::wstring, &SendNetworkChat>("SendNetworkChat");
+    scriptInterface.RegisterFunction<void, int, &SendNetworkReady>("SendNetworkReady");
     scriptInterface.RegisterFunction<std::vector<CScriptValRooted>, &GetAIs>("GetAIs");
     scriptInterface.RegisterFunction<CScriptValRooted, &GetEngineInfo>("GetEngineInfo");
 

source/network/NetClient.cpp

@@ -87 +87 @@
     AddTransition(NCS_INITIAL_GAMESETUP, (uint)NMT_GAME_SETUP, NCS_PREGAME, (void*)&OnGameSetup, context);
 
     AddTransition(NCS_PREGAME, (uint)NMT_CHAT, NCS_PREGAME, (void*)&OnChat, context);
+    AddTransition(NCS_PREGAME, (uint)NMT_READY, NCS_PREGAME, (void*)&OnReady, context);
     AddTransition(NCS_PREGAME, (uint)NMT_GAME_SETUP, NCS_PREGAME, (void*)&OnGameSetup, context);
     AddTransition(NCS_PREGAME, (uint)NMT_PLAYER_ASSIGNMENT, NCS_PREGAME, (void*)&OnPlayerAssignment, context);
     AddTransition(NCS_PREGAME, (uint)NMT_GAME_START, NCS_LOADING, (void*)&OnGameStart, context);
@@ -214 +215 @@
         GetScriptInterface().Eval("({})", host);
         GetScriptInterface().SetProperty(host.get(), "name", std::wstring(it->second.m_Name), false);
         GetScriptInterface().SetProperty(host.get(), "player", it->second.m_PlayerID, false);
+        GetScriptInterface().SetProperty(host.get(), "status", it->second.m_Status, false);
         GetScriptInterface().SetProperty(hosts.get(), it->first.c_str(), host, false);
     }
 
@@ -254 +256 @@
     SendMessage(&chat);
 }
 
+void CNetClient::SendReadyMessage(const int status)
+{
+    CReadyMessage readyStatus;
+    readyStatus.m_Status = status;
+    SendMessage(&readyStatus);
+}
+
 bool CNetClient::HandleMessage(CNetMessage* message)
 {
     // Handle non-FSM messages first
@@ -419 +428 @@
     return true;
 }
 
+bool CNetClient::OnReady(void* context, CFsmEvent* event)
+{
+    ENSURE(event->GetType() == (uint)NMT_READY);
+
+    CNetClient* client = (CNetClient*)context;
+
+    CReadyMessage* message = (CReadyMessage*)event->GetParamRef();
+
+    CScriptValRooted msg;
+    client->GetScriptInterface().Eval("({'type':'ready'})", msg);
+    client->GetScriptInterface().SetProperty(msg.get(), "guid", std::string(message->m_GUID), false);
+    client->GetScriptInterface().SetProperty(msg.get(), "text", int (message->m_Status), false);
+    client->PushGuiMessage(msg);
+
+    return true;
+}
+
 bool CNetClient::OnGameSetup(void* context, CFsmEvent* event)
 {
     ENSURE(event->GetType() == (uint)NMT_GAME_SETUP);
@@ -453 +479 @@
         assignment.m_Enabled = true;
         assignment.m_Name = message->m_Hosts[i].m_Name;
         assignment.m_PlayerID = message->m_Hosts[i].m_PlayerID;
+        assignment.m_Status = message->m_Hosts[i].m_Status;
         newPlayerAssignments[message->m_Hosts[i].m_GUID] = assignment;
     }
 

source/network/NetClient.h

@@ -164 +164 @@
     void LoadFinished();
 
     void SendChatMessage(const std::wstring& text);
+    
+    void SendReadyMessage(const int status);
 
 private:
     // Net message / FSM transition handlers
@@ -172 +174 @@
     static bool OnHandshakeResponse(void* context, CFsmEvent* event);
     static bool OnAuthenticate(void* context, CFsmEvent* event);
     static bool OnChat(void* context, CFsmEvent* event);
+    static bool OnReady(void* context, CFsmEvent* event);
     static bool OnGameSetup(void* context, CFsmEvent* event);
     static bool OnPlayerAssignment(void* context, CFsmEvent* event);
     static bool OnInGame(void* context, CFsmEvent* event);

source/network/NetHost.h

@@ -46 +46 @@
 
     /// The player that the given host controls, or -1 if none (observer)
     i32 m_PlayerID;
+
+    /// Status - Ready or not: 0 for not ready, 1 for ready
+    i32 m_Status;
 };
 
 typedef std::map<CStr, PlayerAssignment> PlayerAssignmentMap; // map from GUID -> assignment

source/network/NetMessage.cpp

@@ -174 +174 @@
     case NMT_CHAT:
         pNewMessage = new CChatMessage;
         break;
+    
+    case NMT_READY:
+        pNewMessage = new CReadyMessage;
+        break;
 
     case NMT_SIMULATION_COMMAND:
         pNewMessage = new CSimulationMessage(scriptInterface);

source/network/NetMessages.h

@@ -46 +46 @@
     NMT_AUTHENTICATE,       // Authentication stage
     NMT_AUTHENTICATE_RESULT,
     NMT_CHAT,       // Common chat message
+    NMT_READY,
     NMT_GAME_SETUP,
     NMT_PLAYER_ASSIGNMENT,
 
@@ -118 +119 @@
     NMT_FIELD(CStrW, m_Message)
 END_NMT_CLASS()
 
+START_NMT_CLASS_(Ready, NMT_READY)
+    NMT_FIELD(CStr8, m_GUID)
+    NMT_FIELD_INT(m_Status, u8, 1)
+END_NMT_CLASS()
+
 START_NMT_CLASS_(PlayerAssignment, NMT_PLAYER_ASSIGNMENT)
     NMT_START_ARRAY(m_Hosts)
         NMT_FIELD(CStr8, m_GUID)
         NMT_FIELD(CStrW, m_Name)
         NMT_FIELD_INT(m_PlayerID, i8, 1)
+        NMT_FIELD_INT(m_Status, u8, 1)
     NMT_END_ARRAY()
 END_NMT_CLASS()
 

source/network/NetServer.cpp

@@ -376 +376 @@
 
     std::vector<std::pair<int, CStr> > newAssignPlayer;
     std::vector<bool> newStartGame;
+    std::vector<std::pair<int, CStr> > newPlayerReady;
+    std::vector<bool> newPlayerResetReady;
     std::vector<std::string> newGameAttributes;
     std::vector<u32> newTurnLength;
 
@@ -386 +388 @@
             return false;
 
         newStartGame.swap(m_StartGameQueue);
+        newPlayerReady.swap(m_PlayerReadyQueue);
+        newPlayerResetReady.swap(m_PlayerResetReadyQueue);
         newAssignPlayer.swap(m_AssignPlayerQueue);
         newGameAttributes.swap(m_GameAttributesQueue);
         newTurnLength.swap(m_TurnLengthQueue);
@@ -394 +398 @@
     for (size_t i = 0; i < newAssignPlayer.size(); ++i)
         AssignPlayer(newAssignPlayer[i].first, newAssignPlayer[i].second);
 
+    for (size_t i = 0; i < newPlayerReady.size(); ++i)
+        ReadyPlayer(newPlayerReady[i].first, newPlayerReady[i].second);
+
+    if (!newPlayerResetReady.empty())
+        ClearAllPlayerReady();
+
     if (!newGameAttributes.empty())
         UpdateGameAttributes(GetScriptInterface().ParseJSON(newGameAttributes.back()));
 
@@ -548 +558 @@
 
     session->AddTransition(NSS_PREGAME, (uint)NMT_CONNECTION_LOST, NSS_UNCONNECTED, (void*)&OnDisconnect, context);
     session->AddTransition(NSS_PREGAME, (uint)NMT_CHAT, NSS_PREGAME, (void*)&OnChat, context);
+    session->AddTransition(NSS_PREGAME, (uint)NMT_READY, NSS_PREGAME, (void*)&OnReady, context);
     session->AddTransition(NSS_PREGAME, (uint)NMT_LOADED_GAME, NSS_INGAME, (void*)&OnLoadedGame, context);
 
     session->AddTransition(NSS_JOIN_SYNCING, (uint)NMT_LOADED_GAME, NSS_INGAME, (void*)&OnJoinSyncingLoadedGame, context);
@@ -649 +660 @@
     assignment.m_Enabled = true;
     assignment.m_Name = name;
     assignment.m_PlayerID = playerID;
+    assignment.m_Status = 0;
     m_PlayerAssignments[guid] = assignment;
 
     // Send the new assignments to all currently active players
@@ -663 +675 @@
     SendPlayerAssignments();
 }
 
+void CNetServerWorker::ReadyPlayer(const int ready, const CStr& guid)
+{
+    m_PlayerAssignments[guid].m_Status = ready;
+
+    SendPlayerAssignments();
+}
+
+void CNetServerWorker::ClearAllPlayerReady()
+{
+    for (PlayerAssignmentMap::iterator it = m_PlayerAssignments.begin(); it != m_PlayerAssignments.end(); ++it)
+    {
+        it->second.m_Status = 0;
+    }
+
+    SendPlayerAssignments();
+}
+
 void CNetServerWorker::AssignPlayer(int playerID, const CStr& guid)
 {
     // Remove anyone who's already assigned to this player
@@ -690 +719 @@
         h.m_GUID = it->first;
         h.m_Name = it->second.m_Name;
         h.m_PlayerID = it->second.m_PlayerID;
+        h.m_Status = it->second.m_Status;
         message.m_Hosts.push_back(h);
     }
 }
@@ -860 +890 @@
     return true;
 }
 
+bool CNetServerWorker::OnReady(void* context, CFsmEvent* event)
+{
+    ENSURE(event->GetType() == (uint)NMT_READY);
+
+    CNetServerSession* session = (CNetServerSession*)context;
+    CNetServerWorker& server = session->GetServer();
+
+    CReadyMessage* message = (CReadyMessage*)event->GetParamRef();
+
+    message->m_GUID = session->GetGUID();
+
+    server.Broadcast(message);
+
+    return true;
+}
+
 bool CNetServerWorker::OnLoadedGame(void* context, CFsmEvent* event)
 {
     ENSURE(event->GetType() == (uint)NMT_LOADED_GAME);
@@ -1053 +1099 @@
     m_Worker->m_AssignPlayerQueue.push_back(std::make_pair(playerID, guid));
 }
 
+void CNetServer::ReadyPlayer(int ready, const CStr& guid)
+{
+    CScopeLock lock(m_Worker->m_WorkerMutex);
+    m_Worker->m_PlayerReadyQueue.push_back(std::make_pair(ready, guid));
+}
+
+void CNetServer::ClearAllPlayerReady()
+{
+    CScopeLock lock(m_Worker->m_WorkerMutex);
+    m_Worker->m_PlayerResetReadyQueue.push_back(false);
+}
+
 void CNetServer::StartGame()
 {
     CScopeLock lock(m_Worker->m_WorkerMutex);

source/network/NetServer.h

@@ -122 +122 @@
      * The changes will be asynchronously propagated to all clients.
      */
     void AssignPlayer(int playerID, const CStr& guid);
+    
+    /**
+     * Call from the GUI to update the player readiness.
+     * The changes will be asynchronously propagated to all clients.
+     */
+    void ReadyPlayer(int ready, const CStr& guid);
 
     /**
+     * Call from the GUI to set the all player readiness to 0.
+     * The changes will be asynchronously propagated to all clients.
+     */
+    void ClearAllPlayerReady();
+    
+    /**
      * Call from the GUI to asynchronously notify all clients that they should start loading the game.
      */
     void StartGame();
@@ -233 +245 @@
 
     void AddPlayer(const CStr& guid, const CStrW& name);
     void RemovePlayer(const CStr& guid);
+    void ReadyPlayer(const int ready, const CStr& guid); 
     void SendPlayerAssignments();
+    void ClearAllPlayerReady();
 
     void SetupSession(CNetServerSession* session);
     bool HandleConnect(CNetServerSession* session);
@@ -245 +259 @@
     static bool OnAuthenticate(void* context, CFsmEvent* event);
     static bool OnInGame(void* context, CFsmEvent* event);
     static bool OnChat(void* context, CFsmEvent* event);
+    static bool OnReady(void* context, CFsmEvent* event);
     static bool OnLoadedGame(void* context, CFsmEvent* event);
     static bool OnJoinSyncingLoadedGame(void* context, CFsmEvent* event);
     static bool OnDisconnect(void* context, CFsmEvent* event);
@@ -320 +335 @@
     // Queues for messages sent by the game thread:
     std::vector<std::pair<int, CStr> > m_AssignPlayerQueue; // protected by m_WorkerMutex
     std::vector<bool> m_StartGameQueue; // protected by m_WorkerMutex
+    std::vector<std::pair<int, CStr> > m_PlayerReadyQueue; // protected by m_WorkerMutex
+    std::vector<bool> m_PlayerResetReadyQueue; // protected by m_WorkerMutex
     std::vector<std::string> m_GameAttributesQueue; // protected by m_WorkerMutex
     std::vector<u32> m_TurnLengthQueue; // protected by m_WorkerMutex
 };