# HG changeset patch
# User frosch <frosch@openttd.org>
# Date 2009-03-12 19:40:16
# Node ID 9b00fd51efbbc4b6c87f650b72fc85d9fa29ca52
# Parent  5c2d4656acdf2033435c12deceba45a1467ddca6

(svn r15692) -Fix [FS#2721]: Just sell the old engines after autorenew/replace. Don't bother about trains exceeding the trainlimit, which will be sold anyway.

diff --git a/src/autoreplace_cmd.cpp b/src/autoreplace_cmd.cpp
--- a/src/autoreplace_cmd.cpp
+++ b/src/autoreplace_cmd.cpp
@@ -513,7 +513,7 @@ static CommandCost ReplaceChain(Vehicle 
 					*chain = new_head;
 				}
 
-				/* Transfer cargo of old vehicles and sell them*/
+				/* Transfer cargo of old vehicles and sell them */
 				for (int i = 0; i < num_units; i++) {
 					Vehicle *w = old_vehs[i];
 					/* Is the vehicle again part of the new chain?
@@ -522,7 +522,10 @@ static CommandCost ReplaceChain(Vehicle 
 
 					if ((flags & DC_EXEC) != 0) TransferCargo(w, new_head, true);
 
-					cost.AddCost(DoCommand(0, w->index, 0, flags, GetCmdSellVeh(w)));
+					/* Sell the vehicle.
+					 * Note: This might temporarly construct new trains, so use DC_AUTOREPLACE to prevent
+					 *       it from failing due to engine limits. */
+					cost.AddCost(DoCommand(0, w->index, 0, flags | DC_AUTOREPLACE, GetCmdSellVeh(w)));
 					if ((flags & DC_EXEC) != 0) {
 						old_vehs[i] = NULL;
 						if (i == 0) old_head = NULL;