Specify original owner when showing option to liberate city

This commit is contained in:
Yair Morgenstern 2021-04-30 12:44:51 +03:00
parent 9847d089b3
commit 9900795e02
2 changed files with 3 additions and 2 deletions

View File

@ -722,7 +722,7 @@ Puppeted cities do not increase your tech or policy cost, but their citizens gen
You have no control over the the production of puppeted cities. =
Puppeted cities also generate 25% less Gold and Science. =
A puppeted city can be annexed at any time. =
Liberate =
Liberate (city returns to [originalOwner]) =
Liberating a city returns it to its original owner, giving you a massive relationship boost with them! =
Raze =
Razing the city annexes it, and starts razing the city to the ground. =

View File

@ -7,6 +7,7 @@ import com.unciv.Constants
import com.unciv.logic.civilization.AlertType
import com.unciv.logic.civilization.CivilizationInfo
import com.unciv.logic.civilization.PopupAlert
import com.unciv.models.translations.fillPlaceholders
import com.unciv.ui.utils.*
class AlertPopup(val worldScreen: WorldScreen, val popupAlert: PopupAlert): Popup(worldScreen) {
@ -66,7 +67,7 @@ class AlertPopup(val worldScreen: WorldScreen, val popupAlert: PopupAlert): Popu
if (city.foundingCiv != ""
&& city.civInfo.civName != city.foundingCiv // can't liberate if the city actually belongs to those guys
&& conqueringCiv.civName != city.foundingCiv) { // or belongs originally to us
add("Liberate".toTextButton().onClick {
add("Liberate (city returns to [originalOwner])".fillPlaceholders(city.foundingCiv).toTextButton().onClick {
city.liberateCity(conqueringCiv)
worldScreen.shouldUpdate = true
close()