Harvesters deliver to the correct cell, even if it's blocked.

OpenRa.Game/Traits/Activities/DeliverOre.cs

@@ -10,10 +10,34 @@ namespace OpenRa.Game.Traits.Activities
 		public Activity NextActivity { get; set; }
 
 		bool isDone;
+		Actor refinery;
+
+		public DeliverOre( Actor refinery )
+		{
+			this.refinery = refinery;
+		}
+
+		static readonly int2 refineryDeliverOffset = new int2( 1, 2 );
 
 		public void Tick(Actor self, Mobile mobile)
 		{
-			if (isDone)
+			if( self.Location != refinery.Location + refineryDeliverOffset )
+			{
+				var move = new Move( refinery.Location + refineryDeliverOffset, 0 );
+				mobile.InternalSetActivity( move );
+				mobile.QueueActivity( this );
+				move.Tick( self, mobile );
+				return;
+			}
+			else if( mobile.facing != 64 )
+			{
+				var turn = new Turn( 64 );
+				mobile.InternalSetActivity( turn );
+				mobile.QueueActivity( this );
+				turn.Tick( self, mobile );
+				return;
+			}
+			else if (isDone)
 			{
 				var harv = self.traits.Get<Harvester>();
 

OpenRa.Game/Traits/Activities/Harvest.cs

@@ -53,11 +53,8 @@ namespace OpenRa.Game.Traits.Activities
 			/* find a proc */
 			var proc = ChooseReturnLocation(self);
 			if( proc != null )
-			{
+				mobile.QueueActivity( new DeliverOre( proc ) );
-				mobile.QueueActivity( new Move( proc.Location + new int2( 1, 2 ), 0 ) );
+
-				mobile.QueueActivity( new Turn( 64 ) );
-				mobile.QueueActivity( new DeliverOre() );
-			}
 			mobile.InternalSetActivity(NextActivity);
 		}
 

OpenRa.Game/UnitOrders.cs

@@ -57,9 +57,7 @@ namespace OpenRa.Game
 				{
 					var mobile = order.Subject.traits.Get<Mobile>();
 					mobile.Cancel(order.Subject);
-					mobile.QueueActivity(new Traits.Activities.Move(order.TargetActor.Location + new int2(1, 2), 0));
+					mobile.QueueActivity(new Traits.Activities.DeliverOre(order.TargetActor));
-					mobile.QueueActivity(new Traits.Activities.Turn(64));
-					mobile.QueueActivity(new Traits.Activities.DeliverOre());
 					break;
 				}
 			case "Harvest":