(svn r11707) -Fix: do not allow refitting flooded (destroyed) vehicles
This commit is contained in:
		@@ -640,6 +640,7 @@ CommandCost CmdRefitAircraft(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 | 
			
		||||
 | 
			
		||||
	if (v->type != VEH_AIRCRAFT || !CheckOwnership(v->owner)) return CMD_ERROR;
 | 
			
		||||
	if (!v->IsStoppedInDepot()) return_cmd_error(STR_A01B_AIRCRAFT_MUST_BE_STOPPED);
 | 
			
		||||
	if (v->vehstatus & VS_CRASHED) return_cmd_error(STR_CAN_T_REFIT_DESTROYED_VEHICLE);
 | 
			
		||||
 | 
			
		||||
	/* Check cargo */
 | 
			
		||||
	CargoID new_cid = GB(p2, 0, 8);
 | 
			
		||||
 
 | 
			
		||||
@@ -2796,6 +2796,7 @@ STR_NEW_VEHICLE_TYPE                                            :{BLACK}{BIGFONT
 | 
			
		||||
STR_NEW_VEHICLE_NOW_AVAILABLE_WITH_TYPE                         :{BLACK}New {STRING} now available!  -  {ENGINE}
 | 
			
		||||
 | 
			
		||||
STR_CAN_T_SELL_DESTROYED_VEHICLE                                :{WHITE}Can't sell destroyed vehicle...
 | 
			
		||||
STR_CAN_T_REFIT_DESTROYED_VEHICLE                               :{WHITE}Can't refit destroyed vehicle...
 | 
			
		||||
 | 
			
		||||
STR_CAN_T_TIMETABLE_VEHICLE                                     :{WHITE}Can't timetable vehicle...
 | 
			
		||||
STR_TIMETABLE_ONLY_WAIT_AT_STATIONS                             :{WHITE}Vehicles can only wait at stations.
 | 
			
		||||
 
 | 
			
		||||
@@ -2134,6 +2134,7 @@ CommandCost CmdRefitRoadVeh(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 | 
			
		||||
 | 
			
		||||
	if (v->type != VEH_ROAD || !CheckOwnership(v->owner)) return CMD_ERROR;
 | 
			
		||||
	if (!CheckRoadVehInDepotStopped(v)) return_cmd_error(STR_9013_MUST_BE_STOPPED_INSIDE);
 | 
			
		||||
	if (v->vehstatus & VS_CRASHED) return_cmd_error(STR_CAN_T_REFIT_DESTROYED_VEHICLE);
 | 
			
		||||
 | 
			
		||||
	if (new_cid >= NUM_CARGO) return CMD_ERROR;
 | 
			
		||||
 | 
			
		||||
 
 | 
			
		||||
@@ -1067,10 +1067,8 @@ CommandCost CmdRefitShip(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 | 
			
		||||
	v = GetVehicle(p1);
 | 
			
		||||
 | 
			
		||||
	if (v->type != VEH_SHIP || !CheckOwnership(v->owner)) return CMD_ERROR;
 | 
			
		||||
 | 
			
		||||
	if (!v->IsStoppedInDepot()) {
 | 
			
		||||
		return_cmd_error(STR_980B_SHIP_MUST_BE_STOPPED_IN);
 | 
			
		||||
	}
 | 
			
		||||
	if (!v->IsStoppedInDepot()) return_cmd_error(STR_980B_SHIP_MUST_BE_STOPPED_IN);
 | 
			
		||||
	if (v->vehstatus & VS_CRASHED) return_cmd_error(STR_CAN_T_REFIT_DESTROYED_VEHICLE);
 | 
			
		||||
 | 
			
		||||
	/* Check cargo */
 | 
			
		||||
	if (!ShipVehInfo(v->engine_type)->refittable) return CMD_ERROR;
 | 
			
		||||
 
 | 
			
		||||
@@ -1720,6 +1720,7 @@ CommandCost CmdRefitRailVehicle(TileIndex tile, uint32 flags, uint32 p1, uint32
 | 
			
		||||
 | 
			
		||||
	if (v->type != VEH_TRAIN || !CheckOwnership(v->owner)) return CMD_ERROR;
 | 
			
		||||
	if (CheckTrainStoppedInDepot(v) < 0) return_cmd_error(STR_TRAIN_MUST_BE_STOPPED);
 | 
			
		||||
	if (v->vehstatus & VS_CRASHED) return_cmd_error(STR_CAN_T_REFIT_DESTROYED_VEHICLE);
 | 
			
		||||
 | 
			
		||||
	/* Check cargo */
 | 
			
		||||
	if (new_cid >= NUM_CARGO) return CMD_ERROR;
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user