diff options
author | bjarni <bjarni@openttd.org> | 2008-01-19 23:34:52 +0000 |
---|---|---|
committer | bjarni <bjarni@openttd.org> | 2008-01-19 23:34:52 +0000 |
commit | e91230415d37882176da2e9bf5f863779b3db745 (patch) | |
tree | 7afaef32fd8a76ab03791943c4a87e2a8a5d3413 | |
parent | 2f6ee3a4a119fbddb7dcde34c8ceaf5ac63efb14 (diff) | |
download | openttd-e91230415d37882176da2e9bf5f863779b3db745.tar.xz |
(svn r11929) -Fix (r9981)[FS#1624]: [autoreplace] fixed a case where a single headed locomotive caused an assert when being replaced to a dualheaded one
This triggered if the player had enough money to replace and refit one of them but not enough to refit the last one as well
-rw-r--r-- | src/autoreplace_cmd.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/autoreplace_cmd.cpp b/src/autoreplace_cmd.cpp index ef5e0d70f..b451be8de 100644 --- a/src/autoreplace_cmd.cpp +++ b/src/autoreplace_cmd.cpp @@ -21,6 +21,7 @@ #include "functions.h" #include "variables.h" #include "autoreplace_func.h" +#include "articulated_vehicles.h" #include "table/strings.h" @@ -175,7 +176,10 @@ static CommandCost ReplaceVehicle(Vehicle **w, byte flags, Money total_cost) if (replacement_cargo_type != CT_NO_REFIT) { /* add refit cost */ CommandCost refit_cost = GetRefitCost(new_engine_type); - if (old_v->type == VEH_TRAIN && IsMultiheaded(old_v)) refit_cost.AddCost(refit_cost); // pay for both ends + if (old_v->type == VEH_TRAIN && RailVehInfo(new_engine_type)->railveh_type == RAILVEH_MULTIHEAD) { + /* Since it's a dualheaded engine we have to pay once more because the rear end is being refitted too. */ + refit_cost.AddCost(refit_cost); + } cost.AddCost(refit_cost); } |