From be09051295cdf52ba46bc71a6a9330e83cdf6b3f Mon Sep 17 00:00:00 2001 From: lyrjie Date: Wed, 25 Dec 2019 11:59:53 +0300 Subject: [PATCH] Fix: cities can be connected to capital via coast even if they don't have `Harbor` building (#1528) Reported by discord@tap wature --- .../com/unciv/logic/civilization/CivInfoTransientUpdater.kt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/core/src/com/unciv/logic/civilization/CivInfoTransientUpdater.kt b/core/src/com/unciv/logic/civilization/CivInfoTransientUpdater.kt index cf36552b02..d92b78011f 100644 --- a/core/src/com/unciv/logic/civilization/CivInfoTransientUpdater.kt +++ b/core/src/com/unciv/logic/civilization/CivInfoTransientUpdater.kt @@ -161,7 +161,10 @@ class CivInfoTransientUpdater(val civInfo: CivilizationInfo){ && cityToConnectFrom.cityConstructions.containsBuildingOrEquivalent(harbor)){ val seaBfs = BFS(cityToConnectFrom.getCenterTile()) { it.isWater || it.isCityCenter() } seaBfs.stepToEnd() - val reachedCities = allCivCities.filter { seaBfs.tilesReached.containsKey(it.getCenterTile())} + val reachedCities = allCivCities.filter { + seaBfs.tilesReached.containsKey(it.getCenterTile()) + && it.cityConstructions.containsBuildingOrEquivalent(harbor) + } for(reachedCity in reachedCities){ if(!citiesReachedToMediums.containsKey(reachedCity)){ newCitiesToCheck.add(reachedCity)