Ticket #2206: correct_backtowork_mod.diff

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

@@ -2108 +2108 @@
     // Filter out all entities that can't go back to work.
     var workers = g_Selection.toList().filter(function(e) {
         var state = GetEntityState(e);
-        return (state && state.unitAI && state.unitAI.lastWorkOrder);
+        return (state && state.unitAI && state.unitAI.hasWorkOrders);
     });
     
     Engine.PostNetworkCommand({"type": "back-to-work", "workers": workers});

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

@@ -294 +294 @@
         });
     }
     
-    if (entState.unitAI && entState.unitAI.lastWorkOrder)
+    if (entState.unitAI && entState.unitAI.hasWorkOrders)
     {
         commands.push({
             "name": "back-to-work",

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

@@ -347 +347 @@
     {
         ret.unitAI = {
             "state": cmpUnitAI.GetCurrentState(),
-            "orders": cmpUnitAI.GetOrders(),
-            "lastWorkOrder": cmpUnitAI.GetLastWorkOrder(),
+            "orders": cmpUnitAI.GetWorkOrders(),
+            "hasWorkOrders": cmpUnitAI.HasWorkOrders(),
         };
         // Add some information needed for ungarrisoning
         if (cmpUnitAI.isGarrisoned && ret.player !== undefined)

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

@@ -78 +78 @@
     cmpPromotedUnitAI.Cheer();
     var orders = cmpCurrentUnitAI.GetOrders();
     cmpPromotedUnitAI.AddOrders(orders);
-    var lastWorkOrder = cmpCurrentUnitAI.GetLastWorkOrder();
-    cmpPromotedUnitAI.SetLastWorkOrder(lastWorkOrder);
+    var workOrders = cmpCurrentUnitAI.GetWorkOrders();
+    cmpPromotedUnitAI.SetWorkOrders(workOrders);
 
     Engine.BroadcastMessage(MT_EntityRenamed, { entity: this.entity, newentity: promotedUnitEntity });
 

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

@@ -2526 +2526 @@
     this.lastFormationName = "";
     this.finishedOrder = false; // used to find if all formation members finished the order
     
-    // To go back to work later
-    this.lastWorkOrder = undefined;
+    // Queue of remembered works
+    this.workOrders = [];
 
     // For preventing increased action rate due to Stop orders or target death.
     this.lastAttacked = undefined;
@@ -2806 +2806 @@
 
 UnitAI.prototype.BackToWork = function()
 {
-    if(this.lastWorkOrder)
+    if(this.workOrders.length)
     {
-        this.ReplaceOrder(this.lastWorkOrder.type, this.lastWorkOrder.data);
+        this.AddOrders(this.workOrders);
+        this.workOrders = [];
         return true;
     }
     else
@@ -2831 +2832 @@
         error("FinishOrder called for entity " + this.entity + " (" + template + ") when order queue is empty\n" + stack);
     }
 
-    // Remove the order from the queue, then forget it if it was a work to avoid trying to go back to it later.
+    // Remove the order from the queue
     var finishedOrder = this.orderQueue.shift();
-    if(finishedOrder.type == "Gather" || finishedOrder.type == "Trade" || finishedOrder.type == "Repair")
-        this.lastWorkOrder = undefined;
-    
     this.order = this.orderQueue[0];
-    // If the worker was returning a resource, we shall 
-    //  -> (if the worker is gathering) keep remembering the "Gather" order which is this.order
-    //  -> (if the worker was directly ordered to return a resource) forget the "ReturnResource" order which is finished
-    if(finishedOrder.type == "ReturnResource" && (!this.order || this.order.type != "Gather"))
-        this.lastWorkOrder = undefined;
 
     if (this.orderQueue.length)
     {
@@ -2892 +2885 @@
 {
     var order = { "type": type, "data": data };
     this.orderQueue.push(order);
-    
-    if(type == "Gather" || type == "Trade" || type == "Repair" || type == "ReturnResource")
-        this.lastWorkOrder = order;
 
     // If we didn't already have an order, then process this new one
     if (this.orderQueue.length == 1)
@@ -2952 +2942 @@
 
 UnitAI.prototype.ReplaceOrder = function(type, data)
 {
+    // Keep the previous orders if needed
+    if (data && data.force)
+    {
+        if (type == "Gather" || type == "Trade" || type == "Repair" || type == "ReturnResource")
+            this.workOrders = [];
+        else
+        {
+            var i0 = 0;
+            if (this.order && this.order.type == "Cheering")
+                i0 = 1;
+            for (var i = i0; i < this.orderQueue.length; ++i)
+            {
+                var ordertype = this.orderQueue[i].type;
+                if (ordertype == "Gather" || ordertype == "Trade" || ordertype == "Repair" || ordertype == "ReturnResource")
+                {
+                    this.workOrders = this.orderQueue.slice(i0);
+                    break;
+                }
+            }
+        }
+    }
+
     // Special cases of orders that shouldn't be replaced:
     // 1. Cheering - we're invulnerable, add order after we finish
     // 2. Packing/unpacking - we're immobile, add order after we finish (unless it's cancel)
@@ -2998 +3010 @@
     return orders;
 };
 
-UnitAI.prototype.GetLastWorkOrder = function()
+UnitAI.prototype.HasWorkOrders = function()
 {
-    return this.lastWorkOrder;
+    return (this.workOrders.length > 0);
 };
 
-UnitAI.prototype.SetLastWorkOrder = function(order)
+UnitAI.prototype.GetWorkOrders = function()
 {
-    this.lastWorkOrder = order;
+    return this.workOrders;
 };
 
+UnitAI.prototype.SetWorkOrders = function(orders)
+{
+    this.workOrders = orders;
+};
+
 UnitAI.prototype.TimerHandler = function(data, lateness)
 {
     // Reset the timer