Ticket #1720: patrol_V3.10.patch

File patrol_V3.10.patch, 9.1 KB (added by Imarok, 3 years ago)

Based on the patch by bb. Fixed a bug introduced in input.js and fixed the handling of multiple selections.

  • binaries/data/config/default.cfg

     
    286286garrison = Ctrl              ; Modifier to garrison when clicking on building
    287287autorallypoint = Ctrl        ; Modifier to set the rally point on the building itself
    288288guard = "G"                  ; Modifier to escort/guard when clicking on unit/building
     289patrol = "P"                 ; Modifier to patrol a unit
    289290repair = "J"                 ; Modifier to repair when clicking on building/mechanical unit
    290291queue = Shift                ; Modifier to queue unit orders instead of replacing
    291292batchtrain = Shift           ; Modifier to train units in batches
  • binaries/data/mods/public/gui/manual/intro.txt

     
    8484[font="sans-bold-14"]Modify mouse action
    8585[font="sans-14"]Ctrl + Right Click on building: Garrison
    8686J + Right Click on building: Repair
     87P + Right Click: Patrol
    8788Shift + Right Click: Queue the move/build/gather/etc order
    8889Shift + Left click when training unit/s: Add units in batches of five
    8990Shift + Left Click or Left Drag over unit on map: Add unit to selection
  • binaries/data/mods/public/gui/session/input.js

     
    1616const ACTION_GARRISON = 1;
    1717const ACTION_REPAIR = 2;
    1818const ACTION_GUARD = 3;
     19const ACTION_PATROL = 4;
    1920var preSelectedAction = ACTION_NONE;
    2021
    2122const INPUT_NORMAL = 0;
     
    207208            }
    208209            return { "possible": true, "data": data, "cursor": cursor };
    209210        }
    210 
    211         return { "possible": (action == "move" || action == "attack-move" || action == "remove-guard") };
     211        return { "possible": ["move", "attack-move", "remove-guard", "patrol"].indexOf(action) > -1 };
    212212    }
    213213
    214214    // Look at the first targeted entity
  • binaries/data/mods/public/gui/session/unit_actions.js

     
    208208        "specificness": 10,
    209209    },
    210210
     211    "patrol":
     212    {
     213        "execute": function(target, action, selection, queued)
     214        {
     215            Engine.PostNetworkCommand({
     216                "type": "patrol",
     217                "entities": selection,
     218                "x": target.x,
     219                "z": target.z,
     220                "target": action.target,
     221                "targetClasses": { "attack": ["Unit"] }, // patrol should only attack units
     222                "queued": queued,
     223                "allowCapture": false
     224            });
     225            Engine.GuiInterfaceCall("PlaySound", { "name": "order_patrol", "entity": selection[0] });
     226            return true;
     227        },
     228        "getActionInfo": function(entState, targetState)
     229        {
     230            return { "possible": true };
     231        },
     232        "hotkeyActionCheck": function(target)
     233        {
     234            if (!Engine.HotkeyIsPressed("session.patrol") || !getActionInfo("patrol", target).possible)
     235                return false;
     236            return {
     237                "type": "patrol",
     238                "cursor": "action-patrol",
     239                "target": target
     240            };
     241        },
     242        "preSelectedActionCheck" : function(target)
     243        {
     244            if (preSelectedAction != ACTION_PATROL || !getActionInfo("patrol", target).possible)
     245                return false;
     246            return {
     247                "type": "patrol",
     248                "cursor": "action-patrol",
     249                "target": target
     250            };
     251        },
     252        "specificness": 37,
     253    },
     254
    211255    "heal":
    212256    {
    213257        "execute": function(target, action, selection, queued)
     
    12181262        },
    12191263    },
    12201264
     1265    "patrol": {
     1266        "getInfo": function(entState)
     1267        {
     1268            if (g_Selection.toList().every(ent => !GetExtendedEntityState(ent).unitAI))
     1269                return false;
     1270            return {
     1271                "tooltip": colorizeHotkey("%(hotkey)s" + " ", "session.patrol") + translate("Patrol"),
     1272                "icon": "patrol.png"
     1273            };
     1274        },
     1275        "execute": function(entState)
     1276        {
     1277            inputState = INPUT_PRESELECTEDACTION;
     1278            preSelectedAction = ACTION_PATROL;
     1279        },
     1280    },
     1281
    12211282    "share-dropsite": {
    12221283        "getInfo": function(entState)
    12231284        {
  • binaries/data/mods/public/simulation/components/UnitAI.js

     
    514514        this.FinishOrder();
    515515    },
    516516
     517    "Order.Patrol": function(msg) {
     518        // Let players move captured domestic animals around
     519        if (this.IsAnimal() || this.IsTurret())
     520        {
     521            this.FinishOrder();
     522            return;
     523        }
     524
     525        if (this.CanPack())
     526        {
     527            this.PushOrderFront("Pack", { "force": true });
     528            return;
     529        }
     530
     531        this.MoveToPoint(this.order.data.x, this.order.data.z);
     532        this.SetNextState("INDIVIDUAL.PATROL");
     533    },
     534
    517535    "Order.Heal": function(msg) {
    518536        // Check the target is alive
    519537        if (!this.TargetIsAlive(this.order.data.target))
     
    829847                this.FinishOrder();
    830848        },
    831849
     850        "Order.Patrol": function(msg) {
     851            this.CallMemberFunction("SetHeldPosition", [msg.data.x, msg.data.z]);
     852
     853            this.MoveToPoint(this.order.data.x, this.order.data.z);
     854            this.SetNextState("PATROL");
     855        },
     856
    832857        "Order.Guard": function(msg) {
    833858            this.CallMemberFunction("Guard", [msg.data.target, false]);
    834859            var cmpFormation = Engine.QueryInterface(this.entity, IID_Formation);
     
    10781103            },
    10791104        },
    10801105
     1106        "PATROL": {
     1107            "enter": function(msg) {
     1108                // Memorize the origin position in case that we want to go back
     1109                let cmpPosition = Engine.QueryInterface(this.entity, IID_Position);
     1110                if (!cmpPosition || !cmpPosition.IsInWorld())
     1111                {
     1112                    this.FinishOrder();
     1113                    return;
     1114                }
     1115                this.patrolStartPos = cmpPosition.GetPosition();
     1116            },
     1117
     1118            "Timer": function(msg) {
     1119                // Check if there are no enemies to attack
     1120                this.FindWalkAndFightTargets();
     1121            },
     1122
     1123            "leave": function(msg) {
     1124                this.StopTimer();
     1125            },
     1126
     1127            "MoveStarted": function(msg) {
     1128                let cmpFormation = Engine.QueryInterface(this.entity, IID_Formation);
     1129                cmpFormation.SetRearrange(true);
     1130                cmpFormation.MoveMembersIntoFormation(true, true);
     1131            },
     1132
     1133            "MoveCompleted": function() {
     1134                /**
     1135                 * A-B-A-B-..:
     1136                 * if the user only commands one patrol order, the patrol will be between
     1137                 * the last position and the defined waypoint
     1138                 * A-B-C-..-A-B-..:
     1139                 * otherwise, the patrol is only between the given patrol commands and the
     1140                 * last position is not included (last position = the position where the unit
     1141                 * is located at the time of the first patrol order)
     1142                 */
     1143
     1144                if (this.orderQueue.length == 1)
     1145                    this.PushOrder("Patrol", this.patrolStartPos);
     1146
     1147                this.PushOrder(this.order.type, this.order.data);
     1148                this.FinishOrder();
     1149            },
     1150        },
     1151
    10811152        "GARRISON":{
    10821153            "enter": function() {
    10831154                // If the garrisonholder should pickup, warn it so it can take needed action
     
    15551626            },
    15561627        },
    15571628
     1629        "PATROL": {
     1630            "enter": function () {
     1631                this.SelectAnimation("move");
     1632
     1633                // Memorize the origin position in case that we want to go back
     1634                let cmpPosition = Engine.QueryInterface(this.entity, IID_Position);
     1635                if (!cmpPosition || !cmpPosition.IsInWorld())
     1636                {
     1637                    this.FinishOrder();
     1638                    return;
     1639                }
     1640                this.patrolStartPos = cmpPosition.GetPosition();
     1641            },
     1642
     1643            "leave": function() {
     1644                this.StopTimer();
     1645            },
     1646
     1647            "Timer": function(msg) {
     1648                this.FindWalkAndFightTargets();
     1649            },
     1650
     1651            "MoveCompleted": function() {
     1652                if (this.orderQueue.length == 1)
     1653                    this.PushOrder("Patrol",this.patrolStartPos);
     1654
     1655                this.PushOrder(this.order.type, this.order.data);
     1656                this.FinishOrder();
     1657            },
     1658        },
     1659
    15581660        "GUARD": {
    15591661            "RemoveGuard": function() {
    15601662                this.StopMoving();
     
    47794881        case "WalkToPointRange":
    47804882        case "MoveIntoFormation":
    47814883        case "GatherNearPosition":
     4884        case "Patrol":
    47824885            targetPositions.push(new Vector2D(order.data.x, order.data.z));
    47834886            break; // and continue the loop
    47844887
     
    49915094    this.AddOrder("WalkAndFight", { "x": x, "z": z, "targetClasses": targetClasses, "force": true }, queued);
    49925095};
    49935096
     5097UnitAI.prototype.Patrol = function(x, z, targetClasses, queued)
     5098{
     5099    this.AddOrder("Patrol", { "x": x, "z": z, "targetClasses": targetClasses, "force": true }, queued);
     5100};
     5101
    49945102/**
    49955103 * Adds leave foundation order to queue, treated as forced.
    49965104 */
  • binaries/data/mods/public/simulation/helpers/Commands.js

     
    176176        });
    177177    },
    178178
     179    "patrol": function(player, cmd, data)
     180    {
     181        GetFormationUnitAIs(data.entities, player).forEach(cmpUnitAI =>
     182            cmpUnitAI.Patrol(cmd.x, cmd.z, cmd.targetClasses, cmd.queued)
     183        );
     184    },
     185
    179186    "heal": function(player, cmd, data)
    180187    {
    181188        if (g_DebugCommands && !(IsOwnedByPlayer(player, cmd.target) || IsOwnedByAllyOfPlayer(player, cmd.target)))