Fixed RepairBridgeOrderTargeter & hut logic - it now ignores dangling bridges & rejects bridges under repair

Closes #4354

OpenRA.Mods.RA/Bridge.cs

@@ -80,10 +80,12 @@ namespace OpenRA.Mods.RA
 		readonly BridgeInfo info;
 		readonly string type;
 
+		readonly Lazy<bool> isDangling;
 		ushort template;
 		Dictionary<CPos, byte> footprint;
 
-		public BridgeHut Hut { get; internal set; }
+		public BridgeHut Hut { get; private set; }
+		public bool IsDangling { get { return isDangling.Value; } }
 
 		public Bridge(Actor self, BridgeInfo info)
 		{
@@ -92,6 +94,7 @@ namespace OpenRA.Mods.RA
 			health.RemoveOnDeath = false;
 			this.info = info;
 			type = self.Info.Name;
+			isDangling = new Lazy<bool>(() => huts[0] == huts[1] && (neighbours[0] == null || neighbours[1] == null));
 			building = self.Trait<Building>();
 		}
 
@@ -144,18 +147,24 @@ namespace OpenRA.Mods.RA
 			}
 		}
 
-		public BridgeHut GetHut(int index)
+		internal void AddHut(BridgeHut hut)
 		{
-			if (huts[index] != null)
-				return huts[index]; // Already found
-
-			var n = neighbours[index];
-			if (n == null)
-				return huts[index] = Hut; // End piece
-
-			return huts[index] = n.Hut ?? n.GetHut(index);
+			if (huts[0] == huts[1])
+				huts[1] = hut;
+			if (Hut == null)
+			{
+				Hut = hut; // Assume only one until called again
+				if (huts[0] == null)
+					huts[0] = hut; // Set only first time
+				for (var d = 0; d <= 1; d++)
+					for (var b = neighbours[d]; b != null; b = b.Hut == null ? b.neighbours[d] : null)
+						b.huts[1 - d] = hut;
+			}
+			else
+				Hut = null;
 		}
 
+		public BridgeHut GetHut(int index) { return huts[index]; }
 		public Bridge GetNeighbor(int[] offset, BridgeLayer bridges)
 		{
 			if (offset == null)
@@ -309,7 +318,7 @@ namespace OpenRA.Mods.RA
 		{
 			if (b.health.DamageState > damage)
 				damage = b.health.DamageState;
-			if (b.Hut == null && b.neighbours[d] != null)
+			if (b.Hut == null && d >= 0 && b.neighbours[d] != null)
 				AggregateDamageState(b.neighbours[d], d, ref damage);
 		}
 

OpenRA.Mods.RA/BridgeHut.cs

@@ -23,18 +23,17 @@ namespace OpenRA.Mods.RA
 
 	class BridgeHut : IDemolishable
 	{
-		Lazy<Bridge> firstBridge;
-		int repairDirections = 0;
+		public readonly Bridge FirstBridge;
 		public readonly Bridge Bridge;
-		public Bridge FirstBridge { get { return firstBridge.Value; } }
 		public DamageState BridgeDamageState { get { return Bridge.AggregateDamageState(); } }
 		public bool Repairing { get { return repairDirections > 0; } }
+		int repairDirections = 0;
 
 		public BridgeHut(ActorInitializer init)
 		{
 			Bridge = init.Get<ParentActorInit>().value.Trait<Bridge>();
-			Bridge.Hut = this;
-			firstBridge = new Lazy<Bridge>(() => Bridge.Enumerate(0, true).Last());
+			Bridge.AddHut(this);
+			FirstBridge = Bridge.Enumerate(0, true).Last();
 		}
 
 		public void Repair(Actor repairer)

OpenRA.Mods.RA/RepairsBridges.cs

@@ -38,22 +38,22 @@ namespace OpenRA.Mods.RA
 			if (order.OrderString != "RepairBridge")
 				return null;
 
-			var bridge = order.TargetActor.TraitOrDefault<BridgeHut>();
-			if (bridge == null)
+			var hut = order.TargetActor.TraitOrDefault<BridgeHut>();
+			if (hut == null)
 				return null;
 
-			return bridge.BridgeDamageState > DamageState.Undamaged ? "Attack" : null;
+			return hut.BridgeDamageState == DamageState.Undamaged || hut.Repairing || hut.Bridge.IsDangling ? null : "Attack";
 		}
 
 		public void ResolveOrder(Actor self, Order order)
 		{
 			if (order.OrderString == "RepairBridge")
 			{
-				var bridge = order.TargetActor.TraitOrDefault<BridgeHut>();
-				if (bridge == null)
+				var hut = order.TargetActor.TraitOrDefault<BridgeHut>();
+				if (hut == null)
 					return;
 
-				if (bridge.BridgeDamageState == DamageState.Undamaged)
+				if (hut.BridgeDamageState == DamageState.Undamaged || hut.Repairing || hut.Bridge.IsDangling)
 					return;
 
 				self.SetTargetLine(Target.FromOrder(self.World, order), Color.Yellow);
@@ -70,12 +70,12 @@ namespace OpenRA.Mods.RA
 
 			public override bool CanTargetActor(Actor self, Actor target, TargetModifiers modifiers, ref string cursor)
 			{
-				var bridge = target.TraitOrDefault<BridgeHut>();
-				if (bridge == null)
+				var hut = target.TraitOrDefault<BridgeHut>();
+				if (hut == null)
 					return false;
 
 				// Require force attack to heal partially damaged bridges to avoid unnecessary cursor noise
-				var damage = bridge.BridgeDamageState;
+				var damage = hut.BridgeDamageState;
 				if (!modifiers.HasModifier(TargetModifiers.ForceAttack) && damage != DamageState.Dead)
 					return false;
 
@@ -83,8 +83,8 @@ namespace OpenRA.Mods.RA
 				if (modifiers.HasModifier(TargetModifiers.ForceMove))
 					return false;
 
-				// Can't repair an undamaged bridge
-				if (damage == DamageState.Undamaged)
+				// Can't repair a bridge that is undamaged, already under repair, or dangling
+				if (damage == DamageState.Undamaged || hut.Repairing || hut.Bridge.IsDangling)
 					cursor = "goldwrench-blocked";
 
 				return true;

mods/ra/maps/intervention/map.yaml

@@ -2240,7 +2240,6 @@ Rules:
 		Inherits: E6
 		Buildable:
 			Prerequisites: ~barracks
-		-RepairsBridges:
 		Captures:
 			CaptureTypes: building
 			Sabotage: False