Ticket #3748: UnitAI_set_isIdle_on_INDIVIDUAL.IDLE.enter_v1.patch

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

@@ -1452 +1452 @@
                 {
                     this.Guard(this.isGuardOf, false);
                     return true;
                 }
 
-                // The GUI and AI want to know when a unit is idle, but we don't
-                // want to send frequent spurious messages if the unit's only
-                // idle for an instant and will quickly go off and do something else.
-                // So we'll set a timer here and only report the idle event if we
-                // remain idle
-                this.StartTimer(1000);
-
                 // If a unit can heal and attack we first want to heal wounded units,
                 // so check if we are a healer and find whether there's anybody nearby to heal.
                 // (If anyone approaches later it'll be handled via LosHealRangeUpdate.)
                 // If anyone in sight gets hurt that will be handled via LosHealRangeUpdate.
                 if (this.IsHealer() && this.FindNewHealTargets())
@@ -1472 +1465 @@
                 // so immediately check whether there's anybody nearby to attack.
                 // (If anyone approaches later, it'll be handled via LosRangeUpdate.)
                 if (this.FindNewTargets())
                     return true; // (abort the FSM transition since we may have already switched state)
 
+                if (!this.isIdle)
+                {
+                    this.isIdle = true;
+                    Engine.PostMessage(this.entity, MT_UnitIdleChanged, { "idle": true });
+                }
+
                 // Nobody to attack - stay in idle
                 return false;
             },
 
             "leave": function() {
@@ -1488 +1487 @@
                 this.StopTimer();
 
                 if (this.isIdle)
                 {
                     this.isIdle = false;
-                    Engine.PostMessage(this.entity, MT_UnitIdleChanged, { "idle": this.isIdle });
+                    Engine.PostMessage(this.entity, MT_UnitIdleChanged, { "idle": false });
                 }
             },
 
             "LosRangeUpdate": function(msg) {
                 if (this.GetStance().targetVisibleEnemies)
@@ -1513 +1512 @@
             "MoveCompleted": function() {
                 this.SelectAnimation("idle");
             },
 
             "Timer": function(msg) {
-                if (!this.isIdle)
-                {
-                    this.isIdle = true;
-                    Engine.PostMessage(this.entity, MT_UnitIdleChanged, { "idle": this.isIdle });
-                }
             },
         },
 
         "WALKING": {
             "enter": function () {