# HG changeset patch # User rubidium # Date 1207763302 0 # Node ID 32505929420d152ee72f062a1079e6fca53f5027 # Parent 24e1116b32f606959e0aefec85c9d3feca142643 (svn r12639) -Fix: order window was not marked dirty on changed orders. diff --git a/src/order_cmd.cpp b/src/order_cmd.cpp --- a/src/order_cmd.cpp +++ b/src/order_cmd.cpp @@ -810,24 +810,22 @@ /* 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); } }