Ticket #999: #999-2012-02-24.patch

0ad/binaries/data/mods/public/simulation/helpers/Commands.js

@@ -66 +66 @@
         });
         break;
 
+    case "heal":
+        if (g_DebugCommands && !(IsOwnedByPlayer(player, cmd.target) || IsOwnedByAllyOfPlayer(player, cmd.target)))
+        {
+            // This check is for debugging only!
+            warn("Invalid command: heal target is not owned by an ally of or player "+player+" itself: "+uneval(cmd));
+        }
+
+        // See UnitAI.CanHeal for target checks
+        var entities = FilterEntityList(cmd.entities, player, controlAllUnits);
+        GetFormationUnitAIs(entities).forEach(function(cmpUnitAI) {
+            cmpUnitAI.Heal(cmd.target, cmd.queued);
+        });
+        break;
+
     case "repair":
         // This covers both repairing damaged buildings, and constructing unfinished foundations
         if (g_DebugCommands && !IsOwnedByAllyOfPlayer(player, cmd.target))

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

@@ -264 +264 @@
         ret.barterMarket = { "prices": cmpBarter.GetPrices() };
     }
 
+    // Abilities
+    var cmpHeal = Engine.QueryInterface(ent, IID_Heal);
+    // Check if we have abilities
+    if (cmpHeal || (cmpHeal && cmpPromotion))
+        ret.Ability = [];
+    if (cmpHeal)
+    {
+        ret.Ability.Healer = { "healableClasses": cmpHeal.GetHealableClasses() };
+    }
+
+    // TODO remove this; This is just used to test/demonstrate the extensibility
+    // of the Ability system
+    // promoteAbility (just for healers)
+    if (cmpPromotion && cmpHeal)
+    {
+        ret.Ability.Promote = true ;
+    }
+
     var cmpRangeManager = Engine.QueryInterface(SYSTEM_ENTITY, IID_RangeManager);
     ret.visibility = cmpRangeManager.GetLosVisibility(ent, player, false);
 

0ad/binaries/data/mods/public/simulation/components/interfaces/Heal.js

@@ +1 @@
+Engine.RegisterInterface("Heal");
+
+//TODO register message type healed?

0ad/binaries/data/mods/public/simulation/components/Health.js

@@ -131 +131 @@
 {
     // If we're already dead, don't allow resurrection
     if (this.hitpoints == 0)
-        return;
+        return false;
 
     var old = this.hitpoints;
     this.hitpoints = Math.min(this.hitpoints + amount, this.GetMaxHitpoints());
 
     Engine.PostMessage(this.entity, MT_HealthChanged, { "from": old, "to": this.hitpoints });
+    // We return the old and the actual hp
+    return { "old": old, "new": this.hitpoints};
 };
 
 //// Private functions ////

0ad/binaries/data/mods/public/simulation/components/Loot.js

@@ -21 +21 @@
 
 Loot.prototype.GetXp = function()
 {
-    return this.template.xp;
+    return +(this.template.xp || 0);
 };
 
 Loot.prototype.GetResources = function()

0ad/binaries/data/mods/public/simulation/components/tests/test_GuiInterface.js

@@ -5 +5 @@
 Engine.LoadComponentScript("interfaces/DamageReceiver.js");
 Engine.LoadComponentScript("interfaces/Foundation.js");
 Engine.LoadComponentScript("interfaces/GarrisonHolder.js");
+Engine.LoadComponentScript("interfaces/Heal.js");
 Engine.LoadComponentScript("interfaces/Health.js");
 Engine.LoadComponentScript("interfaces/Promotion.js");
 Engine.LoadComponentScript("interfaces/RallyPoint.js");

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

@@ -282 +282 @@
         // so abandon this attack order
         this.FinishOrder();
     },
+
+    "Order.Heal": function(msg) {
+        // Check the target is alive
+        if (!this.TargetIsAlive(this.order.data.target))
+        {
+            this.FinishOrder();
+            return;
+        }
+
+        // Check if the target is in range
+        if (this.CheckTargetRange(this.order.data.target, IID_Heal))
+        {
+            this.StopMoving();
+            this.SetNextState("INDIVIDUAL.HEAL.HEALING");
+            return;
+        }
+
+        // If we can't reach the target, but are standing ground,
+        // then abandon this heal order
+        if (this.GetStance().respondStandGround && !this.order.data.force)
+        {
+            this.FinishOrder();
+            return;
+        }
+
+        // Try to move within heal range
+        if (this.MoveToTargetRange(this.order.data.target, IID_Heal))
+        {
+            // We've started walking to the given point
+            this.SetNextState("INDIVIDUAL.HEAL.APPROACHING");
+            return;
+        }
+
+        // We can't reach the target, and can't move towards it,
+        // so abandon this heal order
+        this.FinishOrder();
+    },
+
     "Order.Gather": function(msg) {
         
         // If the target is still alive, we need to kill it first
@@ -402 +439 @@
             cmpFormation.Disband();
         },
 
+        "Order.Heal": function(msg) {
+            // TODO: see notes in Order.Attack
+            var cmpFormation = Engine.QueryInterface(this.entity, IID_Formation);
+            cmpFormation.CallMemberFunction("Heal", [msg.data.target, false]);
+            cmpFormation.Disband();
+        },
+
         "Order.Repair": function(msg) {
             // TODO: see notes in Order.Attack
             var cmpFormation = Engine.QueryInterface(this.entity, IID_Formation);
@@ -995 +1039 @@
             },
         },
 
+        "HEAL": {
+            "EntityRenamed": function(msg) {
+                if (this.order.data.target == msg.entity)
+                    this.order.data.target = msg.newentity;
+            },
+
+            "Attacked": function(msg) {
+                // If we stand ground we will rather die than flee
+                if (!this.GetStance().respondStandGround)
+                    this.Flee(msg.data.attacker, false);
+            },
+
+            "APPROACHING": {
+                "enter": function () {
+                    this.SelectAnimation("move");
+                    this.StartTimer(1000, 1000);
+                },
+
+                "leave": function() {
+                    this.StopTimer();
+                },
+
+                "Timer": function(msg) {
+                    if (this.ShouldAbandonChase(this.order.data.target, this.order.data.force))
+                    {
+                        this.StopMoving();
+                        this.FinishOrder();
+
+                        // Return to our original position
+                        if (this.GetStance().respondHoldGround)
+                            this.WalkToHeldPosition();
+                    }
+                },
+
+                "MoveCompleted": function() {
+                    this.SetNextState("HEALING");
+                },
+
+                "Attacked": function(msg) {
+                    // If we stand ground we will rather die than flee
+                    if (!this.GetStance().respondStandGround)
+                        this.Flee(msg.data.attacker, false);
+                },
+            },
+
+            "HEALING": {
+                "enter": function() {
+                    var cmpHeal = Engine.QueryInterface(this.entity, IID_Heal);
+                    this.healTimers = cmpHeal.GetTimers();
+//                  this.SelectAnimation("heal", false, 1.0, "heal"); // TODO needs animation
+//                  this.SetAnimationSync(this.healTimers.prepare, this.healTimers.repeat);
+                    this.StartTimer(this.healTimers.prepare, this.healTimers.repeat);
+                    // TODO if .prepare is short, players can cheat by cycling heal/stop/heal
+                    // to beat the .repeat time; should enforce a minimum time
+                    // see comment in ATTACKING.enter
+                    this.FaceTowardsTarget(this.order.data.target);
+                },
+
+                "leave": function() {
+                    this.StopTimer();
+                },
+
+                "Timer": function(msg) {
+                    var target = this.order.data.target;
+                    // Check the target is still alive and healable
+                    if (this.TargetIsAlive(target) && this.CanHeal(target))
+                    {
+                        // Check if we can still reach the target
+                        if (this.CheckTargetRange(target, IID_Heal))
+                        {
+                            this.FaceTowardsTarget(target);
+                            var cmpHeal = Engine.QueryInterface(this.entity, IID_Heal);
+                            cmpHeal.PerformHeal(target);
+                            return;
+                        }
+                        // Can't reach it - try to chase after it
+                        if (this.ShouldChaseTargetedEntity(target, this.order.data.force))
+                        {
+                                if (this.MoveToTargetRange(target, IID_Heal))
+                            {
+                                this.SetNextState("HEAL.CHASING");
+                                return;
+                            }
+                        }
+                    }
+                    // Can't reach it, healed to max hp or doesn't exist any more - give up
+                    if (this.FinishOrder())
+                        return;
+
+                    // TODO implement autoheal next (like in ATTACK FindNewTargets())
+                    // needs FindNewHealTargets() and maybe a something like RespondToTargetedEntities() ; figure out how SortEntitiesByPriority is working
+//                  if (this.FindNewHealTargets())
+//                  return;
+
+                    // Return to our original position
+                    if (this.GetStance().respondHoldGround)
+                        this.WalkToHeldPosition();
+                },
+                "Attacked": function(msg) {
+                    // If we stand ground we will rather die than flee
+                    if (!this.GetStance().respondStandGround)
+                        this.Flee(msg.data.attacker, false);
+                },
+            },
+            "CHASING": {
+                "enter": function () {
+                    this.SelectAnimation("move");
+                    this.StartTimer(1000, 1000);
+                },
+
+                "leave": function () {
+                    this.StopTimer();
+                },
+                "Timer": function(msg) {
+                    if (this.ShouldAbandonChase(this.order.data.target, this.order.data.force))
+                    {
+                        this.StopMoving();
+                        this.FinishOrder();
+
+                        // Return to our original position
+                        if (this.GetStance().respondHoldGround)
+                            this.WalkToHeldPosition();
+                    }
+                },
+                "MoveCompleted": function () {
+                    this.SetNextState("HEALING");
+                },
+            },  
+        },
+
         // Returning to dropsite
         "RETURNRESOURCE": {
             "APPROACHING": {
@@ -1746 +1920 @@
 };
 
 /**
+ * Returns true if the target exists and the current hitpoints are at maximum.
+ */
+UnitAI.prototype.TargetIsAtMaxHitpoints = function(ent)
+{
+    var cmpHealth = Engine.QueryInterface(ent, IID_Health);
+    if(!cmpHealth)
+        return false;
+
+    return (cmpHealth.GetHitpoints() == cmpHealth.GetMaxHitpoints());
+};
+
+/**
  * Returns true if the target exists and needs to be killed before
  * beginning to gather resources from it.
  */
@@ -2236 +2422 @@
         case "Flee":
         case "LeaveFoundation":
         case "Attack":
+        case "Heal":
         case "Gather":
         case "ReturnResource":
         case "Repair":
@@ -2359 +2546 @@
     this.AddOrder("GatherNearPosition", { "type": type, "x": position[0], "z": position[1] }, queued);
 }
 
+UnitAI.prototype.Heal = function(target, queued)
+{
+    if (!this.CanHeal(target))
+    {
+        this.WalkToTarget(target, queued);
+        return;
+    }
+    // We don't want to chase units that leave our visibility -> force = false
+    this.AddOrder("Heal", { "target": target, "force": false }, queued);
+};
+
 UnitAI.prototype.ReturnResource = function(target, queued)
 {
     if (!this.CanReturnResource(target, true))
@@ -2555 +2753 @@
     return true;
 };
 
+UnitAI.prototype.CanHeal = function(target)
+{
+    // Formation controllers should always respond to commands
+    // (then the individual units can make up their own minds)
+    if (this.IsFormationController())
+        return true;
+
+    // Verify that we're able to respond to Heal commands
+    var cmpHeal = Engine.QueryInterface(this.entity, IID_Heal);
+    if (!cmpHeal)
+        return false;
+
+    // Verify that the target is alive
+    if (!this.TargetIsAlive(target))
+        return false;
+
+    // Verify that the target is owned by the same player as the entity
+    // or of an ally
+    var cmpOwnership = Engine.QueryInterface(this.entity, IID_Ownership);
+    if (!cmpOwnership || !(IsOwnedByPlayer(cmpOwnership.GetOwner(), target) || IsOwnedByAllyOfPlayer(cmpOwnership.GetOwner(), target)))
+        return false;
+
+    // Verify that the target is a healable class
+    // We could also use cmpIdentity.GetClassesList but this way is cleaner
+    var cmpIdentity = Engine.QueryInterface(target, IID_Identity);
+    if (!cmpIdentity)
+        return false;
+    var healable = false;
+    for each (var healableClass in cmpHeal.GetHealableClasses())
+    {
+        if (cmpIdentity.HasClass(healableClass) != -1)
+        {
+            healable = true;
+        }
+    }
+    if (!healable)
+        return false;
+
+    // Check that the target is not at MaxHealth
+    if (this.TargetIsAtMaxHitpoints(target))
+        return false; 
+
+    return true;
+};
+
 UnitAI.prototype.CanReturnResource = function(target, checkCarriedResource)
 {
     // Formation controllers should always respond to commands

0ad/binaries/data/mods/public/simulation/components/Heal.js

@@ +1 @@
+function Heal() {}
+
+Heal.prototype.Schema = 
+    "<a:help>Controls the healing abilities of the unit.</a:help>" +
+    "<a:example>" +
+        "<Range>20</Range>" +
+        "<HP>5</HP>" +
+        "<Rate>2000</Rate>" +
+        "<HealableClasses datatype=\"tokens\">Support Infantry</HealableClasses>" +
+    "</a:example>" +
+    "<element name='Range' a:help='Range (in metres) where healing is possible'>" +
+        "<ref name='nonNegativeDecimal'/>" +
+    "</element>" +
+    "<element name='HP' a:help='Hitpoints healed per Rate'>" +
+        "<ref name='nonNegativeDecimal'/>" +
+    "</element>" +
+    "<element name='Rate' a:help='A heal is performed every Rate ms'>" +
+        "<ref name='nonNegativeDecimal'/>" +
+    "</element>" +
+    "<element name='HealableClasses'>" +
+        "<attribute name='datatype'>" +
+            "<value>tokens</value>" +
+        "</attribute>" +
+        "<text/>" +
+    "</element>";
+
+Heal.prototype.Init = function()
+{
+};
+
+Heal.prototype.Serialize = null; // we have no dynamic state to save
+
+Heal.prototype.GetTimers = function()
+{
+    var prepare = 1000;
+    var repeat = +(this.template.Rate || 1000);
+    return { "prepare": prepare, "repeat": repeat };
+}
+
+Heal.prototype.GetRange = function()
+{
+    var max = +this.template.Range;
+    var min = 0;
+    return { "max": max, "min": min };
+};
+
+Heal.prototype.GetHealableClasses = function()
+{
+    var classes = this.template.HealableClasses._string;
+    return classes.split(/\s+/);
+};
+
+/**
+ * Heal the target entity. This should only be called after a successful range 
+ * check, and should only be called after GetTimers().repeat msec has passed 
+ * since the last call to PerformHeal.
+ */
+Heal.prototype.PerformHeal = function(target)
+{
+    this.CauseHeal({"target": target});
+};
+
+/**
+ * Heal target
+ */
+Heal.prototype.CauseHeal = function(data)
+{
+    var cmpHealth = Engine.QueryInterface(data.target, IID_Health);
+    if (!cmpHealth)
+        return;
+    var targetState = cmpHealth.Increase(Math.max(0,this.template.HP));
+
+    // Add XP
+    var cmpLoot = Engine.QueryInterface(data.target, IID_Loot);
+    var cmpPromotion = Engine.QueryInterface(this.entity, IID_Promotion);
+    if (targetState.old && targetState.new && cmpLoot && cmpPromotion)
+    {
+        // HP healed * XP per HP
+        cmpPromotion.IncreaseXp((targetState.new-targetState.old)*(cmpLoot.GetXp()/cmpHealth.GetMaxHitpoints()));
+    }
+    //TODO add some Engine.PostMessage? - shouldn't be needed as Increase already posts a message
+    //TODO we need a sound file
+//  PlaySound("heal_impact", this.entity);
+};
+
+Engine.RegisterComponentType(IID_Heal, "Heal", Heal);

0ad/binaries/data/mods/public/simulation/templates/template_unit_support_healer.xml

@@ -5 +5 @@
     <Pierce>2.0</Pierce>
     <Crush>2.0</Crush>
   </Armour>
-  <Auras>
+<!--  <Auras>
     <Heal>
       <Radius>20</Radius>
       <Speed>2000</Speed>
     </Heal>
-  </Auras>
+ </Auras>-->
+  <Heal>
+    <Range>30</Range>
+    <HP>5</HP>
+    <Rate>2000</Rate>
+    <HealableClasses datatype="tokens">Support Infantry Cavalry</HealableClasses>
+  </Heal>
   <Cost>
     <Resources>
       <metal>120</metal>
@@ -23 +29 @@
   <Identity>
     <Classes datatype="tokens">Healer</Classes>
     <GenericName>Healer</GenericName>
-    <Tooltip>Heal units within his Aura. (Not implemented yet)</Tooltip>
+    <Tooltip>Heal units.</Tooltip>
+    <Rank>Basic</Rank>
   </Identity>
+  <Promotion>
+    <RequiredXp>100</RequiredXp>
+  </Promotion>
   <Sound>
     <SoundGroups>
       <select>voice/hellenes/civ/civ_male_select.xml</select>

0ad/binaries/data/mods/public/simulation/templates/template_structure_civic_temple.xml

@@ -57 +57 @@
   </TerritoryInfluence>
   <TrainingQueue>
     <Entities datatype="tokens">
-      units/{civ}_support_healer
+      units/{civ}_support_healer_b
     </Entities>
   </TrainingQueue>
   <Vision>

0ad/binaries/data/mods/public/simulation/templates/units/hele_support_healer_a.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/hele_support_healer_b">
   <Identity>
-    <Civ>hele</Civ>
-    <SpecificName>Hiereús</SpecificName>
-    <History>The art of medicine was widely practised in Classical Greece. Hippocrates was the first physician to separate religion and superstition from actual medicine, and many others followed his lead.</History>
-    <Icon>units/hele_support_healer.png</Icon>
+    <Rank>Advanced</Rank>
   </Identity>
+  <Promotion>
+    <Entity>units/hele_support_healer_e</Entity>
+  </Promotion>
   <VisualActor>
     <Actor>units/hellenes/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/hele_support_healer_e.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/hele_support_healer_a">
   <Identity>
-    <Civ>hele</Civ>
-    <SpecificName>Hiereús</SpecificName>
-    <History>The art of medicine was widely practised in Classical Greece. Hippocrates was the first physician to separate religion and superstition from actual medicine, and many others followed his lead.</History>
-    <Icon>units/hele_support_healer.png</Icon>
+    <Rank>Elite</Rank>
   </Identity>
+  <Promotion disable=""/>
   <VisualActor>
     <Actor>units/hellenes/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/celt_support_healer_a.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/celt_support_healer_b">
   <Identity>
-    <Civ>celt</Civ>
-    <SpecificName>Druides </SpecificName>
-    <History>A druid may be one of many different professions; priest, historian, lawyer, judges, teachers, philosophers, poets, composers, musicians, astronomers, prophets, councillors, high craftsmen like a blacksmith, the classes of the 'men of art', and sometimes kings, chieftains, or other politicians. Druids were very hierarchal, with classes and ranks based on the length of their education and what fields they practiced. They learned their trades through mnemonics by way of poetry and songs, as writing was rarely used by Celts outside of prayers on votive objects, or lists of names for migratory records.</History>
-    <Icon>units/celt_support_healer.png</Icon>
+    <Rank>Advanced</Rank>
   </Identity>
+  <Promotion>
+    <Entity>units/celt_support_healer_e</Entity>
+  </Promotion>
   <VisualActor>
     <Actor>units/celts/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/celt_support_healer_e.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/celt_support_healer_a">
   <Identity>
-    <Civ>celt</Civ>
-    <SpecificName>Druides </SpecificName>
-    <History>A druid may be one of many different professions; priest, historian, lawyer, judges, teachers, philosophers, poets, composers, musicians, astronomers, prophets, councillors, high craftsmen like a blacksmith, the classes of the 'men of art', and sometimes kings, chieftains, or other politicians. Druids were very hierarchal, with classes and ranks based on the length of their education and what fields they practiced. They learned their trades through mnemonics by way of poetry and songs, as writing was rarely used by Celts outside of prayers on votive objects, or lists of names for migratory records.</History>
-    <Icon>units/celt_support_healer.png</Icon>
+    <Rank>Elite</Rank>
   </Identity>
+  <Promotion disable=""/>
   <VisualActor>
     <Actor>units/celts/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/celt_support_healer.xml

@@ -1 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
-  <Identity>
-    <Civ>celt</Civ>
-    <SpecificName>Druides </SpecificName>
-    <History>A druid may be one of many different professions; priest, historian, lawyer, judges, teachers, philosophers, poets, composers, musicians, astronomers, prophets, councillors, high craftsmen like a blacksmith, the classes of the 'men of art', and sometimes kings, chieftains, or other politicians. Druids were very hierarchal, with classes and ranks based on the length of their education and what fields they practiced. They learned their trades through mnemonics by way of poetry and songs, as writing was rarely used by Celts outside of prayers on votive objects, or lists of names for migratory records.</History>
-    <Icon>units/celt_support_healer.png</Icon>
-  </Identity>
-  <VisualActor>
-    <Actor>units/celts/healer.xml</Actor>
-  </VisualActor>
-</Entity>

0ad/binaries/data/mods/public/simulation/templates/units/hele_support_healer.xml

@@ -1 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
-  <Identity>
-    <Civ>hele</Civ>
-    <SpecificName>Hiereús</SpecificName>
-    <History>The art of medicine was widely practised in Classical Greece. Hippocrates was the first physician to separate religion and superstition from actual medicine, and many others followed his lead.</History>
-    <Icon>units/hele_support_healer.png</Icon>
-  </Identity>
-  <VisualActor>
-    <Actor>units/hellenes/healer.xml</Actor>
-  </VisualActor>
-</Entity>

0ad/binaries/data/mods/public/simulation/templates/units/iber_support_healer.xml

@@ -1 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
-  <Identity>
-    <Civ>iber</Civ>
-    <SpecificName>Sacerdotisa de Ataekina</SpecificName>
-    <History> To the best of our knowledge, only one 'temple'-like structure has been found on the Iberian Peninsula dating from the times and the Iberians worshiped their pantheon of gods at small home altars; however, a very special sculptured head and torso was found in a farmer's field around the turn of the 20th century of a personage who was obviously someone of great substance. As the two principal gods, of the many worshiped, were male Endovellikos and female Ataekina, we thought it would be nice to adopt The Lady of Elche as our priestess-healer representing Ataekina. We know from archelogy and the Romans that Ataekina was associated with spring, the changing of seasons, and nature in general. Ataekina also seems to have been associated with the cycle of birth-death-rebirth.</History>
-    <Icon>units/iber_support_healer.png</Icon>
-  </Identity>
-  <VisualActor>
-    <Actor>units/iberians/healer.xml</Actor>
-  </VisualActor>
-</Entity>

0ad/binaries/data/mods/public/simulation/templates/units/rome_support_healer.xml

@@ -1 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
-  <Identity>
-    <Civ>rome</Civ>
-    <GenericName>State Priest</GenericName>
-    <SpecificName>Pontifex Minoris</SpecificName>
-    <History>During the Republic, the position of priest was elevated and required a lot of responsibilities, which is why priests were by no means chosen randomly. The position of Pontifex Maximus, the high priest of the Roman religion, was occupied by such prominent figures as Julius Caesar, Marcus Aemilius Lepidus and Augustus.</History>
-    <Icon>units/rome_support_healer.png</Icon>
-  </Identity>
-  <VisualActor>
-    <Actor>units/romans/healer.xml</Actor>
-  </VisualActor>
-</Entity>

0ad/binaries/data/mods/public/simulation/templates/units/pers_support_healer.xml

@@ -1 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
-  <Identity>
-    <Civ>hele</Civ>
-    <GenericName>Median Magus</GenericName>
-    <SpecificName>Maguš Mada</SpecificName>
-    <History>Under both the Medes and later the Persian the tribe of the Magi or the Magians were the masters of religious and oral tradition, comparable to the Levites of the Bible. They were connected to Zoroastrianism, but likely tended to other Iranian cults as well. Aside from religious duties the Magians also functioned as the Great King's bureaucrats and kept his administration running.</History>
-    <Icon>units/pers_support_healer.png</Icon>
-  </Identity>
-  <VisualActor>
-    <Actor>units/persians/healer.xml</Actor>
-  </VisualActor>
-</Entity>

0ad/binaries/data/mods/public/simulation/templates/units/iber_support_healer_a.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/iber_support_healer_b">
   <Identity>
-    <Civ>iber</Civ>
-    <SpecificName>Sacerdotisa de Ataekina</SpecificName>
-    <History> To the best of our knowledge, only one 'temple'-like structure has been found on the Iberian Peninsula dating from the times and the Iberians worshiped their pantheon of gods at small home altars; however, a very special sculptured head and torso was found in a farmer's field around the turn of the 20th century of a personage who was obviously someone of great substance. As the two principal gods, of the many worshiped, were male Endovellikos and female Ataekina, we thought it would be nice to adopt The Lady of Elche as our priestess-healer representing Ataekina. We know from archelogy and the Romans that Ataekina was associated with spring, the changing of seasons, and nature in general. Ataekina also seems to have been associated with the cycle of birth-death-rebirth.</History>
-    <Icon>units/iber_support_healer.png</Icon>
+    <Rank>Advanced</Rank>
   </Identity>
+  <Promotion>
+    <Entity>units/iber_support_healer_e</Entity>
+  </Promotion>
   <VisualActor>
     <Actor>units/iberians/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/iber_support_healer_e.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/iber_support_healer_a">
   <Identity>
-    <Civ>iber</Civ>
-    <SpecificName>Sacerdotisa de Ataekina</SpecificName>
-    <History> To the best of our knowledge, only one 'temple'-like structure has been found on the Iberian Peninsula dating from the times and the Iberians worshiped their pantheon of gods at small home altars; however, a very special sculptured head and torso was found in a farmer's field around the turn of the 20th century of a personage who was obviously someone of great substance. As the two principal gods, of the many worshiped, were male Endovellikos and female Ataekina, we thought it would be nice to adopt The Lady of Elche as our priestess-healer representing Ataekina. We know from archelogy and the Romans that Ataekina was associated with spring, the changing of seasons, and nature in general. Ataekina also seems to have been associated with the cycle of birth-death-rebirth.</History>
-    <Icon>units/iber_support_healer.png</Icon>
+    <Rank>Elite</Rank>
   </Identity>
+  <Promotion disable=""/>
   <VisualActor>
     <Actor>units/iberians/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/cart_support_healer.xml

@@ -1 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
-  <Identity>
-    <Civ>cart</Civ>
-    <SpecificName>Kehinit</SpecificName>
-    <History>Tanit (also spelled TINITH, TINNIT, or TINT), chief goddess of Carthage, equivalent of Astarte. Although she seems to have had some connection with the heavens, she was also a mother goddess, and fertility symbols often accompany representations of her. She was probably the consort of Baal Hammon (or Amon), the chief god of Carthage, and was often given the attribute "face of Baal." Although Tanit did not appear at Carthage before the 5th century BC, she soon eclipsed the more established cult of Baal Hammon and, in the Carthaginian area at least, was frequently listed before him on the monuments. In the worship of Tanit and Baal Hammon, children, probably firstborn, were sacrificed. Ample evidence of the practice has been found west of Carthage in the precinct of Tanit, where a tofet (a sanctuary for the sacrifice of children) was discovered. Tanit was also worshipped on Malta, Sardinia, and in Spain. There is no other reason for giving the Carthaginians a priestess instead of a priest in 0 A.D., although Tanit was the most popular of their two main gods with the people. </History>
-    <Tooltip>Heal units within her aura. (Not implemented yet)</Tooltip>
-    <Icon>units/cart_support_healer.png</Icon>
-  </Identity>
-  <VisualActor>
-    <Actor>units/carthaginians/healer.xml</Actor>
-  </VisualActor>
-</Entity>

0ad/binaries/data/mods/public/simulation/templates/units/pers_support_healer_a.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/pers_support_healer_b">
   <Identity>
-    <Civ>hele</Civ>
-    <GenericName>Median Magus</GenericName>
-    <SpecificName>Maguš Mada</SpecificName>
-    <History>Under both the Medes and later the Persian the tribe of the Magi or the Magians were the masters of religious and oral tradition, comparable to the Levites of the Bible. They were connected to Zoroastrianism, but likely tended to other Iranian cults as well. Aside from religious duties the Magians also functioned as the Great King's bureaucrats and kept his administration running.</History>
-    <Icon>units/pers_support_healer.png</Icon>
+    <Rank>Advanced</Rank>
   </Identity>
+  <Promotion>
+    <Entity>units/pers_support_healer_e</Entity>
+  </Promotion>
   <VisualActor>
     <Actor>units/persians/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/pers_support_healer_e.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/pers_support_healer_a">
   <Identity>
-    <Civ>hele</Civ>
-    <GenericName>Median Magus</GenericName>
-    <SpecificName>Maguš Mada</SpecificName>
-    <History>Under both the Medes and later the Persian the tribe of the Magi or the Magians were the masters of religious and oral tradition, comparable to the Levites of the Bible. They were connected to Zoroastrianism, but likely tended to other Iranian cults as well. Aside from religious duties the Magians also functioned as the Great King's bureaucrats and kept his administration running.</History>
-    <Icon>units/pers_support_healer.png</Icon>
+    <Rank>Elite</Rank>
   </Identity>
+  <Promotion disable=""/>
   <VisualActor>
     <Actor>units/persians/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/cart_support_healer_a.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/cart_support_healer_b">
   <Identity>
-    <Civ>cart</Civ>
-    <SpecificName>Kehinit</SpecificName>
-    <History>Tanit (also spelled TINITH, TINNIT, or TINT), chief goddess of Carthage, equivalent of Astarte. Although she seems to have had some connection with the heavens, she was also a mother goddess, and fertility symbols often accompany representations of her. She was probably the consort of Baal Hammon (or Amon), the chief god of Carthage, and was often given the attribute "face of Baal." Although Tanit did not appear at Carthage before the 5th century BC, she soon eclipsed the more established cult of Baal Hammon and, in the Carthaginian area at least, was frequently listed before him on the monuments. In the worship of Tanit and Baal Hammon, children, probably firstborn, were sacrificed. Ample evidence of the practice has been found west of Carthage in the precinct of Tanit, where a tofet (a sanctuary for the sacrifice of children) was discovered. Tanit was also worshipped on Malta, Sardinia, and in Spain. There is no other reason for giving the Carthaginians a priestess instead of a priest in 0 A.D., although Tanit was the most popular of their two main gods with the people. </History>
-    <Tooltip>Heal units within her aura. (Not implemented yet)</Tooltip>
-    <Icon>units/cart_support_healer.png</Icon>
+    <Rank>Advanced</Rank>
   </Identity>
+  <Promotion>
+    <Entity>units/cart_support_healer_e</Entity>
+  </Promotion>
   <VisualActor>
     <Actor>units/carthaginians/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/cart_support_healer_e.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/cart_support_healer_a">
   <Identity>
-    <Civ>cart</Civ>
-    <SpecificName>Kehinit</SpecificName>
-    <History>Tanit (also spelled TINITH, TINNIT, or TINT), chief goddess of Carthage, equivalent of Astarte. Although she seems to have had some connection with the heavens, she was also a mother goddess, and fertility symbols often accompany representations of her. She was probably the consort of Baal Hammon (or Amon), the chief god of Carthage, and was often given the attribute "face of Baal." Although Tanit did not appear at Carthage before the 5th century BC, she soon eclipsed the more established cult of Baal Hammon and, in the Carthaginian area at least, was frequently listed before him on the monuments. In the worship of Tanit and Baal Hammon, children, probably firstborn, were sacrificed. Ample evidence of the practice has been found west of Carthage in the precinct of Tanit, where a tofet (a sanctuary for the sacrifice of children) was discovered. Tanit was also worshipped on Malta, Sardinia, and in Spain. There is no other reason for giving the Carthaginians a priestess instead of a priest in 0 A.D., although Tanit was the most popular of their two main gods with the people. </History>
-    <Tooltip>Heal units within her aura. (Not implemented yet)</Tooltip>
-    <Icon>units/cart_support_healer.png</Icon>
+    <Rank>Elite</Rank>
   </Identity>
+  <Promotion disable=""/>
   <VisualActor>
     <Actor>units/carthaginians/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/rome_support_healer_a.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/rome_support_healer_b">
   <Identity>
-    <Civ>rome</Civ>
-    <GenericName>State Priest</GenericName>
-    <SpecificName>Pontifex Minoris</SpecificName>
-    <History>During the Republic, the position of priest was elevated and required a lot of responsibilities, which is why priests were by no means chosen randomly. The position of Pontifex Maximus, the high priest of the Roman religion, was occupied by such prominent figures as Julius Caesar, Marcus Aemilius Lepidus and Augustus.</History>
-    <Icon>units/rome_support_healer.png</Icon>
+    <Rank>Advanced</Rank>
   </Identity>
+  <Promotion>
+    <Entity>units/rome_support_healer_e</Entity>
+  </Promotion>
   <VisualActor>
     <Actor>units/romans/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/simulation/templates/units/rome_support_healer_e.xml

@@ -1 +1 @@
 <?xml version="1.0" encoding="utf-8"?>
-<Entity parent="template_unit_support_healer">
+<Entity parent="units/rome_support_healer_a">
   <Identity>
-    <Civ>rome</Civ>
-    <GenericName>State Priest</GenericName>
-    <SpecificName>Pontifex Minoris</SpecificName>
-    <History>During the Republic, the position of priest was elevated and required a lot of responsibilities, which is why priests were by no means chosen randomly. The position of Pontifex Maximus, the high priest of the Roman religion, was occupied by such prominent figures as Julius Caesar, Marcus Aemilius Lepidus and Augustus.</History>
-    <Icon>units/rome_support_healer.png</Icon>
+    <Rank>Elite</Rank>
   </Identity>
+  <Promotion disable=""/>
   <VisualActor>
     <Actor>units/romans/healer.xml</Actor>
   </VisualActor>

0ad/binaries/data/mods/public/gui/session/session.xml

@@ -725 +725 @@
                     </object>
                 </object>
 
+                <object name="unitAbilityPanel"
+                    size="14 12 100% 100%"
+                >
+                    <object size="0 0 100% 100%">
+                        <repeat count="24">
+                            <object name="unitAbilityButton[n]" hidden="true" style="iconButton" type="button" size="0 0 46 46" tooltip_style="sessionToolTipBottom">
+                                <object name="unitAbilityIcon[n]" type="image" ghost="true" size="3 3 43 43"/>
+                            </object>
+                        </repeat>
+                    </object>
+                </object>
+
                 <object name="unitResearchPanel"
                     style="TranslucentPanelThinBorder"
                     size="0 100%-56 100% 100%"

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

@@ -15 +15 @@
 const ACTION_NONE = 0;
 const ACTION_GARRISON = 1;
 const ACTION_REPAIR = 2;
+const ACTION_HEAL = 3;
 var preSelectedAction = ACTION_NONE;
 
 var INPUT_NORMAL = 0;
@@ -255 +256 @@
                 }
             }
             break;
+        case "heal": //TODO add something like && targetState.needsheal ?
+            if (isUnit(targetState) && (playerOwned || allyOwned))
+            {
+                var healableClasses = entState.Ability.Healer.healableClasses;
+                for each (var unitClass in targetState.identity.classes)
+                {
+                    if (healableClasses.indexOf(unitClass) != -1)
+                    {
+                        return {"possible": true};
+                    }
+                }
+            }
+            break;
         case "gather":
             if (targetState.resourceSupply && (playerOwned || gaiaOwned))
             {
@@ -350 +364 @@
             else
                 return {"type": "none", "cursor": "action-repair-disabled", "target": undefined};
             break;
+        case ACTION_HEAL:
+            if (getActionInfo("heal", target).possible)
+                return {"type": "heal", "cursor": "action-heal", "target": target};
+            else
+                return {"type": "none", "cursor": "action-heal-disabled", "target": undefined};
+            break;
         }
     }
     else if (Engine.HotkeyIsPressed("session.garrison"))
@@ -992 +1012 @@
         Engine.GuiInterfaceCall("PlaySound", { "name": "order_attack", "entity": selection[0] });
         return true;
 
+    case "promote":
+        Engine.PostNetworkCommand({"type": "promote", "entities": selection, "target": action.target, "queued": queued});
+        //TODO play sound???
+        return true;
+
+    case "heal":
+        Engine.PostNetworkCommand({"type": "heal", "entities": selection, "target": action.target, "queued": queued});
+        //TODO play sound
+//      Engine.GuiInterfaceCall("PlaySound", { "name": "order_heal", "entity": selection[0] });
+        return true;
+
     case "build": // (same command as repair)
     case "repair":
         Engine.PostNetworkCommand({"type": "repair", "entities": selection, "target": action.target, "autocontinue": true, "queued": queued});
@@ -1204 +1235 @@
                 inputState = INPUT_PRESELECTEDACTION;
                 preSelectedAction = ACTION_REPAIR;
                 break;
+            case "heal":
+                inputState = INPUT_PRESELECTEDACTION;
+                preSelectedAction = ACTION_HEAL;
+                break;
+            case "promote":
+                doAction({ "type": "promote"})
+                break;
             case "unload-all":
                 unloadAll(entity);
                 break;

0ad/binaries/data/mods/public/gui/session/unit_commands.js

@@ -5 +5 @@
 const FORMATION = "Formation";
 const TRAINING = "Training";
 const CONSTRUCTION = "Construction";
+const ABILITY = "Ability";
 const COMMAND = "Command";
 const STANCE = "Stance";
 
@@ -20 +21 @@
 const BARTER_ACTIONS = ["Sell", "Buy"];
 
 // The number of currently visible buttons (used to optimise showing/hiding)
-var g_unitPanelButtons = {"Selection": 0, "Queue": 0, "Formation": 0, "Garrison": 0, "Barter": 0, "Training": 0, "Construction": 0, "Command": 0, "Stance": 0};
+var g_unitPanelButtons = {"Selection": 0, "Queue": 0, "Formation": 0, "Garrison": 0, "Barter": 0, "Training": 0, "Construction": 0, "Ability": 0, "Command": 0, "Stance": 0};
 
 // Unit panels are panels with row(s) of buttons
-var g_unitPanels = ["Selection", "Queue", "Formation", "Garrison", "Barter", "Training", "Construction", "Research", "Stance", "Command"];
+var g_unitPanels = ["Selection", "Queue", "Formation", "Garrison", "Barter", "Training", "Construction", "Ability", "Research", "Stance", "Command"];
 
 // Indexes of resources to sell and buy on barter panel
 var g_barterSell = 0;
@@ -174 +175 @@
                 numberOfItems =  24;
             break;
 
+        case ABILITY:
+            if (numberOfItems > 24)
+                numberOfItems = 24;
+            break;
+
         case COMMAND:
             if (numberOfItems > 6)
                 numberOfItems = 6;
@@ -190 +196 @@
         var item = items[i];
         var entType = ((guiName == "Queue")? item.template : item);
         var template;
-        if (guiName != "Formation" && guiName != "Command" && guiName != "Stance")
+        if (guiName != "Formation" && guiName != "Command" && guiName != "Stance" && guiName != "Ability")
         {
             template = GetTemplateData(entType);
             if (!template)
@@ -269 +275 @@
 
                 break;
 
+            case ABILITY:
+                // TODO read tooltips from some file or template based on 'item'
+                var tooltip;
+                switch(item)
+                {
+                    case "heal":
+                        tooltip = "Heal units";
+                        break;
+                    case "promote":
+                        tooltip = "Promote this unit";
+                        break;
+                    default:
+                        tooltip = "No tooltip defined";
+                        break;
+                }
+                break;
+
             case COMMAND:
                 // here, "item" is an object with properties .name (command name), .tooltip and .icon (relative to session/icons/single)
                 if (item.name == "unload-all")
@@ -338 +361 @@
             icon.sprite = "stretched:session/icons/single/" + item.icon;
 
         }
+        else if (guiName == "Ability")
+        {
+            icon.sprite = "stretched:session/icons/single/"+item+".png";
+        }
         else if (template.icon)
         {
             icon.sprite = "stretched:session/portraits/" + template.icon;
@@ -511 +538 @@
             setupUnitBarterPanel(entState);
         }
 
+        if (entState.Ability)
+        {
+            var abilities = [];
+            if (entState.Ability.Healer)
+                abilities.push("heal");
+            if (entState.Ability.Promote)
+                abilities.push("promote");
+            setupUnitPanel("Ability", usedPanels, entState, abilities, function (item) { performCommand(entState.id, item); });
+        }
+
         if (entState.buildEntities && entState.buildEntities.length)
         {
             setupUnitPanel("Construction", usedPanels, entState, entState.buildEntities, startBuildingPlacement);

0ad/binaries/data/mods/public/art/textures/cursors/action-heal.txt

@@ +1 @@
+1 1

0ad/binaries/data/mods/public/maps/scenarios/Territory

@@ -423 +423 @@
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="70">
-            <Template>units/cart_support_healer</Template>
+            <Template>units/cart_support_healer_b</Template>
             <Player>1</Player>
             <Position x="466.09253" z="360.52533"/>
             <Orientation y="2.35621"/>

0ad/binaries/data/mods/public/maps/scenarios/Carthaginian

@@ -393 +393 @@
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="155">
-            <Template>units/cart_support_healer</Template>
+            <Template>units/cart_support_healer_b</Template>
             <Player>2</Player>
             <Position x="317.48292" z="426.92215"/>
             <Orientation y="2.35621"/>

0ad/binaries/data/mods/public/maps/scenarios/Carthaginians

@@ -698 +698 @@
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="144">
-            <Template>units/cart_support_healer</Template>
+            <Template>units/cart_support_healer_b</Template>
             <Player>1</Player>
             <Position x="357.91175" z="352.26969"/>
             <Orientation y="2.35621"/>

0ad/binaries/data/mods/public/maps/scenarios/Iberians_Sandbox.xml

@@ -625 +625 @@
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="102">
-            <Template>units/iber_support_healer</Template>
+            <Template>units/iber_support_healer_b</Template>
             <Player>1</Player>
             <Position x="461.1692" z="253.42179"/>
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="103">
-            <Template>units/iber_support_healer</Template>
+            <Template>units/iber_support_healer_b</Template>
             <Player>1</Player>
             <Position x="463.86204" z="255.47721"/>
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="104">
-            <Template>units/iber_support_healer</Template>
+            <Template>units/iber_support_healer_b</Template>
             <Player>1</Player>
             <Position x="467.43818" z="259.12848"/>
             <Orientation y="2.35621"/>

0ad/binaries/data/mods/public/maps/scenarios/Hellenic

@@ -1048 +1048 @@
             <Orientation y="-2.7069"/>
         </Entity>
         <Entity uid="186">
-            <Template>units/hele_support_healer</Template>
+            <Template>units/hele_support_healer_b</Template>
             <Player>1</Player>
             <Position x="575.92756" z="588.6966"/>
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="187">
-            <Template>units/hele_support_healer</Template>
+            <Template>units/hele_support_healer_b</Template>
             <Player>1</Player>
             <Position x="578.18409" z="590.42273"/>
             <Orientation y="2.35621"/>
         </Entity>
         <Entity uid="188">
-            <Template>units/hele_support_healer</Template>
+            <Template>units/hele_support_healer_b</Template>
             <Player>1</Player>
             <Position x="580.17115" z="593.37488"/>
             <Orientation y="2.35621"/>

0ad/binaries/data/mods/public/maps/scenarios/Celtic_Buildings.xml

@@ -462 +462 @@
             <Orientation y="-0.01775"/>
         </Entity>
         <Entity uid="222">
-            <Template>units/celt_support_healer</Template>
+            <Template>units/celt_support_healer_b</Template>
             <Player>1</Player>
             <Position x="168.80031" z="46.27845"/>
             <Orientation y="2.35621"/>