From 5e9059cd1ea1485b052eaf6d4b9a1d1ef81d54fa Mon Sep 17 00:00:00 2001 From: Xander Lenstra <71121390+xlenstra@users.noreply.github.com> Date: Sun, 18 Jun 2023 15:35:17 +0200 Subject: [PATCH] Small comment changes in UniqueType.kt (#9617) --- core/src/com/unciv/models/ruleset/unique/UniqueType.kt | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/core/src/com/unciv/models/ruleset/unique/UniqueType.kt b/core/src/com/unciv/models/ruleset/unique/UniqueType.kt index 00bd2785be..e8d0a594a2 100644 --- a/core/src/com/unciv/models/ruleset/unique/UniqueType.kt +++ b/core/src/com/unciv/models/ruleset/unique/UniqueType.kt @@ -150,7 +150,7 @@ enum class UniqueType(val text: String, vararg targets: UniqueTarget, val flags: /// Growth GrowthPercentBonus("[relativeAmount]% growth [cityFilter]", UniqueTarget.Global, UniqueTarget.FollowerBelief), - CarryOverFood("[relativeAmount]% Food is carried over after population increases [cityFilter]", UniqueTarget.Global, UniqueTarget.FollowerBelief), + CarryOverFood("[amount]% Food is carried over after population increases [cityFilter]", UniqueTarget.Global, UniqueTarget.FollowerBelief), // Todo: moddability: specialists -> [populationFilter] FoodConsumptionBySpecialists("[relativeAmount]% Food consumption by specialists [cityFilter]", UniqueTarget.Global, UniqueTarget.FollowerBelief), @@ -229,7 +229,7 @@ enum class UniqueType(val text: String, vararg targets: UniqueTarget, val flags: ReceiveFreeUnitWhenDiscoveringTech("Receive free [unit] when you discover [tech]", UniqueTarget.Global), // Units entering Tiles - // ToDo: make per unit and use unit filters? + // ToDo: make per unit and use unit filters? "Enables embarkation " LandUnitEmbarkation("Enables embarkation for land units", UniqueTarget.Global), UnitsMayEnterOcean("Enables [mapUnitFilter] units to enter ocean tiles", UniqueTarget.Global), LandUnitsCrossTerrainAfterUnitGained("Land units may cross [terrainName] tiles after the first [baseUnitFilter] is earned", UniqueTarget.Global), @@ -248,8 +248,9 @@ enum class UniqueType(val text: String, vararg targets: UniqueTarget, val flags: /// Resource production & consumption ConsumesResources("Consumes [amount] [resource]", UniqueTarget.Improvement, UniqueTarget.Building, UniqueTarget.Unit), ProvidesResources("Provides [amount] [resource]", UniqueTarget.Improvement, UniqueTarget.Global), - // Todo: Sign should not be part of the unique placeholder CostsResources("Costs [amount] [stockpiledResource]", UniqueTarget.Improvement, UniqueTarget.Building, UniqueTarget.Unit), + // Todo: Get rid of forced sign (+[relativeAmount]) and unify these two, e.g.: "[relativeAmount]% [resource/resourceType] production" + // Note that the parameter type 'resourceType' (strategic, luxury, bonus) currently doesn't exist and should then be added as well StrategicResourcesIncrease("Quantity of strategic resources produced by the empire +[relativeAmount]%", UniqueTarget.Global), // used by Policies DoubleResourceProduced("Double quantity of [resource] produced", UniqueTarget.Global),