diff --git a/android/assets/jsons/Civ V - Vanilla/Units.json b/android/assets/jsons/Civ V - Vanilla/Units.json index 04e7ab615d..2b42897b78 100644 --- a/android/assets/jsons/Civ V - Vanilla/Units.json +++ b/android/assets/jsons/Civ V - Vanilla/Units.json @@ -914,12 +914,12 @@ { "name": "Foreign Legion", "unitType": "Gunpowder", - "replaces": "Infantry", + "replaces": "Rifleman", "uniqueTo": "France", "movement": 2, "strength": 50, "cost": 320, - "requiredTech": "Replaceable Parts", + "requiredTech": "Rifling", "upgradesTo": "Infantry", "obsoleteTech": "Plastics", "uniques": ["[+20]% Strength "], diff --git a/core/src/com/unciv/logic/automation/NextTurnAutomation.kt b/core/src/com/unciv/logic/automation/NextTurnAutomation.kt index e1b0cbc369..dce04027c1 100644 --- a/core/src/com/unciv/logic/automation/NextTurnAutomation.kt +++ b/core/src/com/unciv/logic/automation/NextTurnAutomation.kt @@ -767,7 +767,7 @@ object NextTurnAutomation { unit.baseUnit.isRanged() -> rangedUnits.add(unit) unit.baseUnit.isMelee() -> meleeUnits.add(unit) unit.hasUnique("Bonus for units in 2 tile radius 15%") - -> generals.add(unit) //generals move after military units + -> generals.add(unit) // Generals move after military units else -> civilianUnits.add(unit) } } diff --git a/core/src/com/unciv/models/ruleset/Ruleset.kt b/core/src/com/unciv/models/ruleset/Ruleset.kt index f77a4650db..37cbe4ec02 100644 --- a/core/src/com/unciv/models/ruleset/Ruleset.kt +++ b/core/src/com/unciv/models/ruleset/Ruleset.kt @@ -373,7 +373,7 @@ class Ruleset { // When not checking the entire ruleset, we can only really detect ruleset-invariant errors in uniques for (unit in units.values) { - if (unit.upgradesTo == unit.name) + if (unit.upgradesTo == unit.name || (unit.upgradesTo != null && unit.upgradesTo == unit.replaces)) lines += "${unit.name} upgrades to itself!" if (!unit.isCivilian() && unit.strength == 0) lines += "${unit.name} is a military unit but has no assigned strength!"