(svn r1710) - Fix: [autoreplace] vehicles do no longer go to a depot all the time if the owner clicked 'stop replacing' ( ship+aircraft+road vehicles )
authorbjarni
Sat, 29 Jan 2005 01:02:47 +0000
changeset 1206 9a4ebc971c41
parent 1205 98a430b41d81
child 1207 a2a433807a99
(svn r1710) - Fix: [autoreplace] vehicles do no longer go to a depot all the time if the owner clicked 'stop replacing' ( ship+aircraft+road vehicles )
aircraft_cmd.c
roadveh_cmd.c
ship_cmd.c
--- a/aircraft_cmd.c	Fri Jan 28 20:43:46 2005 +0000
+++ b/aircraft_cmd.c	Sat Jan 29 01:02:47 2005 +0000
@@ -399,8 +399,12 @@
 	return 0;
 }
 
-// p1 = vehicle
-// p2 = if set, the aircraft will try to goto a depot, but not stop
+/*  Send an aircraft to the hangar in the next station
+	p1 = index of the aircraft
+	p2 = bit 0..15 = index of station with hangar to use (only used if bit 17 is set)
+		 bit 16 = set v->set_for_replacement & do not stop in hangar
+		 bit 17 = goto the hangar in airport given in bit 0..15 instead of next stop
+		 bit 18 = clear v->set_for_replacement */
 int32 CmdSendAircraftToHangar(int x, int y, uint32 flags, uint32 p1, uint32 p2)
 {
 	Vehicle *v;
@@ -409,10 +413,15 @@
 
 	v = GetVehicle(p1);
 
+	if (!CheckOwnership(v->owner))
+		return CMD_ERROR;
+
 	if (HASBIT(p2, 16)) v->set_for_replacement = true; //now all clients knows that the vehicle wants to be replaced
 
-	if (!CheckOwnership(v->owner))
-		return CMD_ERROR;
+	if (HASBIT(p2, 18)) {
+		v->set_for_replacement = false;
+		return CMD_ERROR; // no need to send aircraft to a depot. We just told that it don't have to anymore
+	}
 
 	if (v->current_order.type == OT_GOTO_DEPOT && p2 == 0) {
 		if (flags & DC_EXEC) {
@@ -1525,6 +1534,13 @@
 				DoCommandP(v->tile, v->index, next_airport, NULL, CMD_SEND_AIRCRAFT_TO_HANGAR | CMD_SHOW_NO_ERROR);
 				_current_player = OWNER_NONE;
 			}
+		} else { // no need to go to a depot
+			if (v->set_for_replacement) {
+				// it seems that the user clicked "Stop replacing"
+				_current_player = _local_player;
+				DoCommandP(v->tile, v->index, (1 | (1 << 2)) << 16, NULL, CMD_SEND_AIRCRAFT_TO_HANGAR | CMD_SHOW_NO_ERROR);
+				_current_player = OWNER_NONE;
+			}
 		}
 	}
 }
@@ -1589,6 +1605,13 @@
 			_current_player = _local_player;
 			DoCommandP(v->tile, v->index, 1 << 16, NULL, CMD_SEND_AIRCRAFT_TO_HANGAR | CMD_SHOW_NO_ERROR);
 			_current_player = OWNER_NONE;
+		} else { // no need to go to a depot
+			if (v->set_for_replacement) {
+				// it seems that the user clicked "Stop replacing"
+				_current_player = _local_player;
+				DoCommandP(v->tile, v->index, (1 | (1 << 2)) << 16, NULL, CMD_SEND_AIRCRAFT_TO_HANGAR | CMD_SHOW_NO_ERROR);
+				_current_player = OWNER_NONE;
+			}
 		}
 	}
 }
--- a/roadveh_cmd.c	Fri Jan 28 20:43:46 2005 +0000
+++ b/roadveh_cmd.c	Sat Jan 29 01:02:47 2005 +0000
@@ -293,7 +293,9 @@
 
 /*  Send a road vehicle to the nearest depot
 	p1 = index of the road vehicle
-	p2 = bit 0 = do not stop in depot, bit 1 = have depot in orders */
+	p2 = bit 0 = do not stop in depot
+		 bit 1 = set v->set_for_replacement
+		 bit 2 = clear v->set_for_replacement */
 int32 CmdSendRoadVehToDepot(int x, int y, uint32 flags, uint32 p1, uint32 p2)
 {
 	Vehicle *v = GetVehicle(p1);
@@ -303,8 +305,9 @@
 		return CMD_ERROR;
 
 	if (HASBIT(p2, 0)) v->set_for_replacement = true;
+	if (HASBIT(p2, 2)) v->set_for_replacement = false;
 
-	if (HASBIT(p2, 1)) return CMD_ERROR;   // vehicle has a depot in schedule. It just needed to set set_for_replacement
+	if (HASBIT(p2, 1) || HASBIT(p2, 2)) return CMD_ERROR;   // vehicle has a depot in schedule. It just needed to alter set_for_replacement
 
 	if (v->current_order.type == OT_GOTO_DEPOT) {
 		if (flags & DC_EXEC) {
@@ -658,6 +661,13 @@
 				DoCommandP(v->tile, v->index, flags, NULL, CMD_SEND_ROADVEH_TO_DEPOT | CMD_SHOW_NO_ERROR);
 				_current_player = OWNER_NONE;
 			}
+		} else { // no need to go to a depot
+			if (v->set_for_replacement) {
+				// it seems that the user clicked "Stop replacing"
+				_current_player = _local_player;
+				DoCommandP(v->tile, v->index, 1 | (1 << 2), NULL, CMD_SEND_ROADVEH_TO_DEPOT | CMD_SHOW_NO_ERROR);
+				_current_player = OWNER_NONE;
+			}
 		}
 	}
 }
--- a/ship_cmd.c	Fri Jan 28 20:43:46 2005 +0000
+++ b/ship_cmd.c	Sat Jan 29 01:02:47 2005 +0000
@@ -299,6 +299,13 @@
 						DoCommandP(v->tile, v->index, flags, NULL, CMD_SEND_SHIP_TO_DEPOT | CMD_SHOW_NO_ERROR);
 						_current_player = OWNER_NONE;
 					}
+				} else { // no need to go to a depot
+					if (v->set_for_replacement) {
+						// it seems that the user clicked "Stop replacing"
+						_current_player = _local_player;
+						DoCommandP(v->tile, v->index, 1 | (1 << 2), NULL, CMD_SEND_SHIP_TO_DEPOT | CMD_SHOW_NO_ERROR);
+						_current_player = OWNER_NONE;
+					}
 				}
 			}
 			if (!(b.flags & OF_NON_STOP))
@@ -949,7 +956,9 @@
 
 /*  Send a ship to the nearest depot
 	p1 = index of the ship
-	p2 = bit 0 = do not stop in depot, bit 1 = have depot in orders */
+	p2 = bit 0 = do not stop in depot
+		 bit 1 = set v->set_for_replacement
+		 bit 2 = clear v->set_for_replacement */
 int32 CmdSendShipToDepot(int x, int y, uint32 flags, uint32 p1, uint32 p2)
 {
 	Vehicle *v;
@@ -961,8 +970,9 @@
 		return CMD_ERROR;
 
 	if (HASBIT(p2, 0)) v->set_for_replacement = true;
+	if (HASBIT(p2, 2)) v->set_for_replacement = false;
 
-	if (HASBIT(p2, 1)) return CMD_ERROR;   // vehicle has a depot in schedule. It just needed to set set_for_replacement
+	if (HASBIT(p2, 1) || HASBIT(p2, 2)) return CMD_ERROR;   // vehicle has a depot in schedule. It just needed to alter set_for_replacement
 
 	if (v->current_order.type == OT_GOTO_DEPOT) {
 		if (flags & DC_EXEC) {