Ticket #1915: garrisonCapacity-v2.diff

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

@@ -311 +311 @@
         case "garrison":
             if (hasClass(entState, "Unit") && targetState.garrisonHolder && (playerOwned || mutualAllyOwned))
             {
+                var tooltip = "Current garrison: " + targetState.garrisonHolder.entities.length
+                    + "/" + targetState.garrisonHolder.capacity;
+                if (targetState.garrisonHolder.entities.length >= targetState.garrisonHolder.capacity)
+                    tooltip = "[color=\"orange\"]" + tooltip + "[/color]";
                 var allowedClasses = targetState.garrisonHolder.allowedClasses;
                 for each (var unitClass in entState.identity.classes)
                 {
                     if (allowedClasses.indexOf(unitClass) != -1)
                     {
-                        return {"possible": true};
+                        return {"possible": true, "tooltip": tooltip};
                     }
                 }
             }
@@ -460 +464 @@
         target = targets[0];
     }
 
+    var actionInfo = undefined;
     if (preSelectedAction != ACTION_NONE)
     {
         switch (preSelectedAction)
@@ -482 +487 @@
     {
         return {"type": "attack", "cursor": "action-attack", "target": target};
     }
-    else if (Engine.HotkeyIsPressed("session.garrison") && getActionInfo("garrison", target).possible)
+    else if (Engine.HotkeyIsPressed("session.garrison") && (actionInfo = getActionInfo("garrison", target)).possible)
     {
-        return {"type": "garrison", "cursor": "action-garrison", "target": target};
+        return {"type": "garrison", "cursor": "action-garrison", "tooltip": actionInfo.tooltip, "target": target};
     }
     else if (Engine.HotkeyIsPressed("session.attackmove") && getActionInfo("attack-move", target).possible)
     {
@@ -492 +497 @@
     }
     else
     {
-        var actionInfo = undefined;
         if ((actionInfo = getActionInfo("setup-trade-route", target)).possible)
             return {"type": "setup-trade-route", "cursor": "action-setup-trade-route", "tooltip": actionInfo.tooltip, "target": target};
         else if ((actionInfo = getActionInfo("gather", target)).possible)

binaries/data/mods/public/simulation/components/GuiInterface.js

@@ -297 +297 @@
     {
         ret.garrisonHolder = {
             "entities": cmpGarrisonHolder.GetEntities(),
-            "allowedClasses": cmpGarrisonHolder.GetAllowedClassesList()
+            "allowedClasses": cmpGarrisonHolder.GetAllowedClassesList(),
+            "capacity": cmpGarrisonHolder.GetCapacity()
         };
     }