From 7ce519c5d4a202e1f7afc4066d9e5af6cf2b8cb1 Mon Sep 17 00:00:00 2001 From: yairm210 Date: Mon, 26 Aug 2024 13:59:56 +0300 Subject: [PATCH] Resolved #12159 - Cities can be conquered in the middle of automation, so we need to "finalize" the sequence before iterating --- .../unciv/logic/automation/civilization/NextTurnAutomation.kt | 1 + 1 file changed, 1 insertion(+) diff --git a/core/src/com/unciv/logic/automation/civilization/NextTurnAutomation.kt b/core/src/com/unciv/logic/automation/civilization/NextTurnAutomation.kt index 47af7f9887..ed8412e25a 100644 --- a/core/src/com/unciv/logic/automation/civilization/NextTurnAutomation.kt +++ b/core/src/com/unciv/logic/automation/civilization/NextTurnAutomation.kt @@ -423,6 +423,7 @@ object NextTurnAutomation { val citiesRequiringManualPlacement = civInfo.getKnownCivs().filter { it.isAtWarWith(civInfo) } .flatMap { it.cities } .filter { it.getCenterTile().getTilesInDistance(4).count { it.militaryUnit?.civ == civInfo } > 4 } + .toList() for (city in citiesRequiringManualPlacement) automateCityConquer(civInfo, city)