Ticket #4078: all_ai_in_sp.patch

binaries/data/mods/public/gui/credits/texts/programming.json

@@ -92 +92 @@
             {"nick": "infyquest", "name": "Vijay Kiran Kamuju"},
             {"nick": "IronNerd", "name": "Matthew McMullan"},
             {"nick": "Itms", "name": "Nicolas Auvray"},
+            {"nick": "ixly"},
             {"nick": "Jaison", "name": "Marco tom Suden"},
             {"nick": "jammus", "name": "James Scott"},
             {"nick": "Jgwman"},

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

@@ -1601 +1601 @@
             ++assignedCount;
     }
 
-    // Only enable start button if we have enough assigned players
+    // Only enable start button if we are the controller
     if (g_IsController)
-        Engine.GetGUIObjectByName("startGame").enabled = assignedCount > 0;
+        Engine.GetGUIObjectByName("startGame").enabled = true;
 
     for (let ai of g_Settings.AIDescriptions)
     {
@@ -1683 +1683 @@
                     return;
 
                 let guid = hostGuidList[this.selected];
-                if (!guid)
+
+                if (!guid || guid.substr(0, 3) == "ai:")
                 {
+                    // Unassign any host from this player slot
                     if (g_IsNetworked)
-                        // Unassign any host from this player slot
                         Engine.AssignNetworkPlayer(playerID, "");
-                    // Remove AI from this player slot
-                    g_GameAttributes.settings.PlayerData[playerSlot].AI = "";
+
+                    // If a human was removed from the slot, make sure they are not reassigned
+                    let changeTarget = this.list_data[selection];
+                    if(g_PlayerAssignments[changeTarget])
+                        g_PlayerAssignments[changeTarget].player = -1;
+
+                    // Set the AI, unless we are unassigning
+                    g_GameAttributes.settings.PlayerData[playerSlot].AI = guid ? guid.substr(3) : "";
                 }
-                else if (guid.substr(0, 3) == "ai:")
-                {
-                    if (g_IsNetworked)
-                        // Unassign any host from this player slot
-                        Engine.AssignNetworkPlayer(playerID, "");
-                    // Set the AI for this player slot
-                    g_GameAttributes.settings.PlayerData[playerSlot].AI = guid.substr(3);
-                }
                 else
                     swapPlayers(guid, playerSlot);