summaryrefslogtreecommitdiff
path: root/src/order_cmd.cpp
diff options
context:
space:
mode:
authorrubidium <rubidium@openttd.org>2008-04-09 17:48:22 +0000
committerrubidium <rubidium@openttd.org>2008-04-09 17:48:22 +0000
commitd1783459473251e2d7f392a3013fe6a8bbef4316 (patch)
treef7755dae5eac93a44b37f4d9b2f580f7c49b5884 /src/order_cmd.cpp
parent55f0d01235b68e01030f8a137cd2622b2e0269e7 (diff)
downloadopenttd-d1783459473251e2d7f392a3013fe6a8bbef4316.tar.xz
(svn r12639) -Fix: order window was not marked dirty on changed orders.
Diffstat (limited to 'src/order_cmd.cpp')
-rw-r--r--src/order_cmd.cpp32
1 files changed, 15 insertions, 17 deletions
diff --git a/src/order_cmd.cpp b/src/order_cmd.cpp
index d65631665..bbd1c01bd 100644
--- a/src/order_cmd.cpp
+++ b/src/order_cmd.cpp
@@ -810,24 +810,22 @@ CommandCost CmdModifyOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
/* Update the windows and full load flags, also for vehicles that share the same order list */
Vehicle *u = GetFirstVehicleFromSharedList(v);
DeleteOrderWarnings(u);
- if (mof == MOF_LOAD || mof == MOF_UNLOAD) {
- for (; u != NULL; u = u->next_shared) {
- /* Toggle u->current_order "Full load" flag if it changed.
- * However, as the same flag is used for depot orders, check
- * whether we are not going to a depot as there are three
- * cases where the full load flag can be active and only
- * one case where the flag is used for depot orders. In the
- * other cases for the OrderTypeByte the flags are not used,
- * so do not care and those orders should not be active
- * when this function is called.
- */
- if (sel_ord == u->cur_order_index &&
- (u->current_order.IsType(OT_GOTO_STATION) || u->current_order.IsType(OT_LOADING)) &&
- u->current_order.GetLoadType() != order->GetLoadType()) {
- u->current_order.SetLoadType(order->GetLoadType());
- }
- InvalidateVehicleOrder(u);
+ for (; u != NULL; u = u->next_shared) {
+ /* Toggle u->current_order "Full load" flag if it changed.
+ * However, as the same flag is used for depot orders, check
+ * whether we are not going to a depot as there are three
+ * cases where the full load flag can be active and only
+ * one case where the flag is used for depot orders. In the
+ * other cases for the OrderTypeByte the flags are not used,
+ * so do not care and those orders should not be active
+ * when this function is called.
+ */
+ if (sel_ord == u->cur_order_index &&
+ (u->current_order.IsType(OT_GOTO_STATION) || u->current_order.IsType(OT_LOADING)) &&
+ u->current_order.GetLoadType() != order->GetLoadType()) {
+ u->current_order.SetLoadType(order->GetLoadType());
}
+ InvalidateVehicleOrder(u);
}
}