# HG changeset patch
# User bjarni <bjarni@openttd.org>
# Date 1160468070 0
# Node ID 5a0e1c701644af4dfed2b251272cea9298b279d2
# Parent  12222b80509234006ba028320b0b25fcd783abf4
(svn r6714) -Codechange: replaced a direct manipulation of windows with InvalidateWindowData() in rail_cmd.c
   Moved the actual modification of railtype to WE_INVALIDATE_DATA in the train depot handler
-Codechange: added SetWindowDirty() to WE_INVALIDATE_DATA as it made no sense to update the list without making the window dirty

diff --git a/aircraft_gui.c b/aircraft_gui.c
--- a/aircraft_gui.c
+++ b/aircraft_gui.c
@@ -548,6 +548,7 @@
 
 		case WE_INVALIDATE_DATA:
 			WP(w,buildvehicle_d).data_invalidated = true;
+			SetWindowDirty(w);
 			break;
 
 		case WE_DESTROY:
diff --git a/engine.c b/engine.c
--- a/engine.c
+++ b/engine.c
@@ -214,7 +214,6 @@
 	e->preview_player = 0xFF;
 	if (player == _local_player) {
 		InvalidateWindowClassesData(WC_BUILD_VEHICLE);
-		InvalidateWindowClasses(WC_BUILD_VEHICLE);
 		InvalidateWindowClasses(WC_REPLACE_VEHICLE);
 	}
 }
@@ -339,7 +338,6 @@
 
 	e->flags = (e->flags & ~ENGINE_INTRODUCING) | ENGINE_AVAILABLE;
 	InvalidateWindowClassesData(WC_BUILD_VEHICLE);
-	InvalidateWindowClasses(WC_BUILD_VEHICLE);
 	InvalidateWindowClasses(WC_REPLACE_VEHICLE);
 
 	// Now available for all players
diff --git a/rail_cmd.c b/rail_cmd.c
--- a/rail_cmd.c
+++ b/rail_cmd.c
@@ -888,14 +888,9 @@
 
 		if (IsTileDepotType(tile, TRANSPORT_RAIL)) {
 			Vehicle *v;
-			Window *w;
 
 			/* Update build vehicle window related to this depot */
-			w = FindWindowById(WC_BUILD_VEHICLE, tile);
-			if (w != NULL) {
-				WP(w,buildvehicle_d).railtype = totype;
-				SetWindowDirty(w);
-			}
+			InvalidateWindowData(WC_BUILD_VEHICLE, tile);
 
 			/* update power of trains in this depot */
 			FOR_ALL_VEHICLES(v) {
diff --git a/train_gui.c b/train_gui.c
--- a/train_gui.c
+++ b/train_gui.c
@@ -546,7 +546,9 @@
 			break;
 
 		case WE_INVALIDATE_DATA:
+			if (w->window_number != 0) WP(w,buildvehicle_d).railtype = GetRailType(w->window_number);
 			WP(w,buildvehicle_d).data_invalidated = true;
+			SetWindowDirty(w);
 			break;
 
 		case WE_DESTROY: