(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:
		@@ -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)
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user