mirrors/Unciv
1
0
Fork 0
mirror of https://github.com/yairm210/Unciv.git synced 2025-07-13 09:18:43 +07:00
Code Issues Packages Projects Releases Wiki Activity

Fix Garrison bonus logic (#8952)

Browse Source 
* City filter for Military Caste
* Garrison display in Overview - all credit to @WhoIsJohannes
This commit is contained in:
SomeTroglodyte
2023-03-18 18:24:33 +01:00
committed by GitHub
parent 20e1b9b978
commit 9bb49342cc
2 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
core/src/com/unciv/logic/city/City.kt
View File

@ -10,6 +10,7 @@ import com.unciv.logic.city.managers.CityReligionManager
import com.unciv.logic.civilization.Civilization
import com.unciv.logic.civilization.diplomacy.DiplomacyFlags
import com.unciv.logic.map.TileMap
import com.unciv.logic.map.mapunit.MapUnit
import com.unciv.logic.map.tile.RoadStatus
import com.unciv.logic.map.tile.Tile
import com.unciv.models.Counter
@ -153,6 +154,12 @@ class City : IsPartOfGameInfoSerialization {
return connectionTypePredicate(mediumTypes)
}
fun isGarrisoned() = getGarrison() != null
fun getGarrison(): MapUnit? =
getCenterTile().militaryUnit?.takeIf {
it.civ == this.civ && it.canGarrison()
}
fun hasFlag(flag: CityFlags) = flagsCountdown.containsKey(flag.name)
fun getFlag(flag: CityFlags) = flagsCountdown[flag.name]!!
@ -512,7 +519,7 @@ class City : IsPartOfGameInfoSerialization {
"in all cities with a world wonder" -> cityConstructions.getBuiltBuildings()
.any { it.isWonder }
"in all cities connected to capital" -> isConnectedToCapital()
"in all cities with a garrison" -> getCenterTile().militaryUnit != null
"in all cities with a garrison" -> isGarrisoned()
"in all cities in which the majority religion is a major religion" ->
religion.getMajorityReligionName() != null
&& religion.getMajorityReligion()!!.isMajorReligion()

8
core/src/com/unciv/ui/screens/overviewscreen/CityOverviewTable.kt
View File

@ -126,8 +126,8 @@ class CityOverviewTab(
"Population" -> city2.population.population - city1.population.population
WLTK -> city2.isWeLoveTheKingDayActive().compareTo(city1.isWeLoveTheKingDayActive())
GARRISON -> collator.compare(
city2.getCenterTile().militaryUnit?.name?.tr() ?: "",
city1.getCenterTile().militaryUnit?.name?.tr() ?: "",
city2.getGarrison()?.name?.tr() ?: "",
city1.getGarrison()?.name?.tr() ?: "",
)
else -> {
val stat = Stat.safeValueOf(persistableData.sortedBy)!!
@ -236,7 +236,7 @@ class CityOverviewTab(
else -> cityInfoTableDetails.add()
}
val garrisonUnit = city.getCenterTile().militaryUnit
val garrisonUnit = city.getGarrison()
if (garrisonUnit == null) {
cityInfoTableDetails.add()
} else {
@ -270,7 +270,7 @@ class CityOverviewTab(
else cityInfoTableTotal.add(viewingPlayer.cities.sumOf { it.getStat(stat) }.toCenteredLabel())
}
cityInfoTableTotal.add(viewingPlayer.cities.count { it.isWeLoveTheKingDayActive() }.toCenteredLabel())
cityInfoTableTotal.add(viewingPlayer.cities.count { it.getCenterTile().militaryUnit != null }.toCenteredLabel())
cityInfoTableTotal.add(viewingPlayer.cities.count { it.isGarrisoned() }.toCenteredLabel())
cityInfoTableTotal.pack()
}
}