# HG changeset patch
# User bjarni <bjarni@openttd.org>
# Date 1169587466 0
# Node ID 75bda39deacde8c7c49683f32a48b2407b0ad6b7
# Parent  5ed4e35e8161d4d123f948dc3a2c4b185a7d3bce
(svn r8381) -Fix (r8377): [build vehicle window] cleaned up the train refitable detection. It had two variables doing the same thing, but not always set to the same (oops)

diff --git a/src/build_vehicle_gui.cpp b/src/build_vehicle_gui.cpp
--- a/src/build_vehicle_gui.cpp
+++ b/src/build_vehicle_gui.cpp
@@ -535,12 +535,12 @@
 		case VEH_Train: {
 			const RailVehicleInfo *rvi = RailVehInfo(engine_number);
 
+			refitable = (EngInfo(engine_number)->refit_mask != 0) && (rvi->capacity > 0);
+
 			if (rvi->flags & RVI_WAGON) {
 				y = DrawRailWagonPurchaseInfo(x, y, engine_number, rvi);
-				refitable = true;
 			} else {
 				y = DrawRailEnginePurchaseInfo(x, y, engine_number, rvi, e);
-				refitable = (rvi->capacity > 0);
 			}
 
 			/* Cargo type + capacity, or N/A */
@@ -549,11 +549,10 @@
 				SetDParam(2, STR_EMPTY);
 			} else {
 				int multihead = (rvi->flags & RVI_MULTIHEAD ? 1 : 0);
-				bool refittable = (EngInfo(engine_number)->refit_mask != 0);
 
 				SetDParam(0, rvi->cargo_type);
 				SetDParam(1, (rvi->capacity * (CountArticulatedParts(engine_number) + 1)) << multihead);
-				SetDParam(2, refittable ? STR_9842_REFITTABLE : STR_EMPTY);
+				SetDParam(2, refitable ? STR_9842_REFITTABLE : STR_EMPTY);
 			}
 			DrawString(x,y, STR_PURCHASE_INFO_CAPACITY, 0);
 			y += 10;