mirror of https://github.com/OpenTTD/OpenTTD.git
(svn r1947) As in r1946, permit DC_QUERY_COST even on non-depot tiles - so that it works for the ai_new. It is de iure not a bug yet but let's be safe against future annoying headaches. Signed-Off-By: TrueLight
This commit is contained in:
parent
4a6a7028a9
commit
a0c157eea8
|
@ -878,16 +878,17 @@ int32 CmdBuildShip(int x, int y, uint32 flags, uint32 p1, uint32 p2)
|
|||
|
||||
if (!IsEngineBuildable(p1, VEH_Ship)) return CMD_ERROR;
|
||||
|
||||
if (!IsTileDepotType(tile, TRANSPORT_WATER)) return CMD_ERROR;
|
||||
|
||||
if (_map_owner[tile] != _current_player) return CMD_ERROR;
|
||||
|
||||
SET_EXPENSES_TYPE(EXPENSES_NEW_VEHICLES);
|
||||
|
||||
value = EstimateShipCost(p1);
|
||||
if (flags & DC_QUERY_COST)
|
||||
return value;
|
||||
|
||||
/* The ai_new queries the vehicle cost before building the route,
|
||||
* so we must check against cheaters no sooner than now. --pasky */
|
||||
if (!IsTileDepotType(tile, TRANSPORT_WATER)) return CMD_ERROR;
|
||||
if (_map_owner[tile] != _current_player) return CMD_ERROR;
|
||||
|
||||
v = AllocateVehicle();
|
||||
if (v == NULL || IsOrderPoolFull() ||
|
||||
(unit_num = GetFreeUnitNumber(VEH_Ship)) > _patches.max_ships)
|
||||
|
|
|
@ -542,9 +542,10 @@ int32 CmdBuildRailVehicle(int x, int y, uint32 flags, uint32 p1, uint32 p2)
|
|||
|
||||
if (!IsEngineBuildable(p1, VEH_Train)) return CMD_ERROR;
|
||||
|
||||
if (!IsTileDepotType(tile, TRANSPORT_RAIL)) return CMD_ERROR;
|
||||
|
||||
if (_map_owner[tile] != _current_player) return CMD_ERROR;
|
||||
if (!(flags & DC_QUERY_COST)) {
|
||||
if (!IsTileDepotType(tile, TRANSPORT_RAIL)) return CMD_ERROR;
|
||||
if (_map_owner[tile] != _current_player) return CMD_ERROR;
|
||||
}
|
||||
|
||||
_cmd_build_rail_veh_var1 = 0;
|
||||
|
||||
|
|
Loading…
Reference in New Issue