Actually *use* the new parameter

This commit is contained in:
yairm210
2024-09-02 17:53:15 +03:00
parent 83baecd8a5
commit 9c6c0018c2

View File

@ -297,7 +297,7 @@ class CityConstructions : IsPartOfGameInfoSerialization {
fun cheapestStatBuilding(stat: Stat): Building? {
return city.getRuleset().buildings.values
.filter { !it.isAnyWonder() && it.isStatRelated(stat) &&
.filter { !it.isAnyWonder() && it.isStatRelated(stat, city) &&
(it.isBuildable(this) || isBeingConstructedOrEnqueued(it.name)) }
.minByOrNull { it.cost }
}
@ -539,12 +539,12 @@ class CityConstructions : IsPartOfGameInfoSerialization {
civ.cache.updateHasActiveEnemyMovementPenalty()
// Korean unique - apparently gives the same as the research agreement
if (building.isStatRelated(Stat.Science) && civ.hasUnique(UniqueType.TechBoostWhenScientificBuildingsBuiltInCapital)
if (building.isStatRelated(Stat.Science, city) && civ.hasUnique(UniqueType.TechBoostWhenScientificBuildingsBuiltInCapital)
&& city.isCapital())
civ.tech.addScience(civ.tech.scienceOfLast8Turns.sum() / 8)
// Happiness is global, so it could affect all cities
if (building.isStatRelated(Stat.Happiness)) {
if (building.isStatRelated(Stat.Happiness, city)) {
for (city in civ.cities) {
city.reassignPopulationDeferred()
}