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

Fix bugs and crashes (#5245)

Browse Source 
* Fixed crash when liberating city-states

* Fixed crash & building duplication from 'legalism' policy
This commit is contained in:
Xander Lenstra 2021-09-17 07:44:48 +02:00 committed by GitHub
parent 0e88ca978b
commit b0dff6a71c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 17 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/src/com/unciv/logic/city/CityInfoConquestFunctions.kt
View File

@ -87,9 +87,9 @@ class CityInfoConquestFunctions(val city: CityInfo){
val reconqueredCityWhileStillInResistance = previousOwner == conqueringCiv.civName && resistanceCounter != 0
this@CityInfoConquestFunctions.moveToCiv(receivingCiv)
destroyBuildingsOnCapture()
this@CityInfoConquestFunctions.moveToCiv(receivingCiv)
Battle.destroyIfDefeated(conqueredCiv, conqueringCiv)

7
core/src/com/unciv/logic/city/CityStats.kt
View File

@ -153,9 +153,12 @@ class CityStats(val cityInfo: CityInfo) {
stats.add(getStatPercentBonusesFromUniques(currentConstruction, cityInfo.civInfo.nation.uniqueObjects.asSequence()))
if (currentConstruction is Building
&& cityInfo.civInfo.getCapital().cityConstructions.builtBuildings.contains(currentConstruction.name)
&& cityInfo.civInfo.hasUnique("+25% Production towards any buildings that already exist in the Capital"))
&& cityInfo.civInfo.cities.isNotEmpty()
&& cityInfo.civInfo.getCapital().cityConstructions.builtBuildings.contains(currentConstruction.name)
&& cityInfo.civInfo.hasUnique("+25% Production towards any buildings that already exist in the Capital")
) {
stats.production += 25f
}
return stats
}

19
core/src/com/unciv/logic/civilization/CivConstructions.kt
View File

@ -15,19 +15,20 @@ class CivConstructions() {
val boughtItemsWithIncreasingPrice: Counter<String> = Counter()
// Maps to cities to all free buildings they contain
private val freeBuildings: HashMap<String,HashSet<String>> = hashMapOf()
private val freeBuildings: HashMap<String, HashSet<String>> = hashMapOf()
// Maps stats to the cities that have received a building of that stat
// Android Studio says an EnumMap would be better, but that thing isn't serializable.
// I don't know how to suppress that hint :(
private val freeStatBuildingsProvided: HashMap<Stat, HashSet<String>> = hashMapOf()
// We can't use an enum instead of a string, due to the inability of the JSON-parser
// to function properly and forcing this to be an `HashMap<String, HashSet<String>>`
// when loading, even if this wasn't the original type, leading to run-time errors.
private val freeStatBuildingsProvided: HashMap<String, HashSet<String>> = hashMapOf()
// Maps buildings to the cities that have received that building
private val freeSpecificBuildingsProvided: HashMap<String, HashSet<String>> = hashMapOf()
init {
for (stat in Stat.values()) {
freeStatBuildingsProvided[stat] = hashSetOf()
freeStatBuildingsProvided[stat.name] = hashSetOf()
}
}
@ -43,7 +44,7 @@ class CivConstructions() {
fun setTransients(civInfo: CivilizationInfo) {
this.civInfo = civInfo
// civInfo.boughtConstructionsWithGloballyIncreasingPrice deprecated since 3.16.15, this is replacement code
if (civInfo.boughtConstructionsWithGloballyIncreasingPrice.isNotEmpty()) {
for (item in civInfo.boughtConstructionsWithGloballyIncreasingPrice) {
@ -71,7 +72,7 @@ class CivConstructions() {
if (civInfo.policies.cultureBuildingsAdded.isNotEmpty()) {
for ((cityId, building) in civInfo.policies.cultureBuildingsAdded) {
freeStatBuildingsProvided[Stat.Culture]!!.add(cityId)
freeStatBuildingsProvided[Stat.Culture.name]!!.add(cityId)
if (cityId !in freeBuildings)
freeBuildings[cityId] = hashSetOf()
@ -126,11 +127,11 @@ class CivConstructions() {
private fun addFreeStatBuildings(stat: Stat, amount: Int) {
for (city in civInfo.cities.take(amount)) {
if (freeStatBuildingsProvided[stat]!!.contains(city.id) || !city.cityConstructions.hasBuildableStatBuildings(stat)) continue
if (freeStatBuildingsProvided[stat.name]!!.contains(city.id) || !city.cityConstructions.hasBuildableStatBuildings(stat)) continue
val builtBuilding = city.cityConstructions.addCheapestBuildableStatBuilding(stat)
if (builtBuilding != null) {
freeStatBuildingsProvided[stat]!!.add(city.id)
freeStatBuildingsProvided[stat.name]!!.add(city.id)
addFreeBuilding(city.id, builtBuilding)
}
}