diff --git a/core/src/com/unciv/models/ruleset/validation/RulesetValidator.kt b/core/src/com/unciv/models/ruleset/validation/RulesetValidator.kt index 7ad3746e68..3d2d8be1d4 100644 --- a/core/src/com/unciv/models/ruleset/validation/RulesetValidator.kt +++ b/core/src/com/unciv/models/ruleset/validation/RulesetValidator.kt @@ -669,10 +669,8 @@ class RulesetValidator(val ruleset: Ruleset) { private fun checkUnitRulesetInvariant(unit: BaseUnit, lines: RulesetErrorList) { if (unit.upgradesTo == unit.name || (unit.upgradesTo != null && unit.upgradesTo == unit.replaces)) lines += "${unit.name} upgrades to itself!" - if (!unit.isCivilian() && unit.strength == 0) + if (unit.isMilitary() && unit.strength == 0) // Should only match ranged units with 0 strength lines += "${unit.name} is a military unit but has no assigned strength!" - if (unit.isRanged() && unit.rangedStrength == 0 && !unit.hasUnique(UniqueType.CannotAttack)) - lines += "${unit.name} is a ranged unit but has no assigned rangedStrength!" } /** Collects all RulesetSpecific checks for a BaseUnit */ @@ -683,9 +681,6 @@ class RulesetValidator(val ruleset: Ruleset) { for (obsoleteTech: String in unit.techsAtWhichNoLongerAvailable()) if (!ruleset.technologies.containsKey(obsoleteTech)) lines += "${unit.name} obsoletes at tech ${obsoleteTech} which does not exist!" - for (obsoleteTech: String in unit.techsAtWhichAutoUpgradeInProduction()) - if (!ruleset.technologies.containsKey(obsoleteTech)) - lines += "${unit.name} upgrades at tech ${obsoleteTech} which does not exist!" if (unit.upgradesTo != null && !ruleset.units.containsKey(unit.upgradesTo!!)) lines += "${unit.name} upgrades to unit ${unit.upgradesTo} which does not exist!"