Fix ScriptValueCollection not loading values using values defined bel…

…ow (#524)

* Fix ScriptValueCollection not loading values using values defined below

* Update ScriptValueCollection.cs

commonItems.UnitTests/ScriptValueCollectionTests.cs

@@ -17,6 +17,7 @@ public void SimpleScriptValuesAreReadFromGameAndMods() {
 
 		scriptValueCollection.Keys.Should()
 			.BeEquivalentTo(
+				"value_using_value_defined_below",
 				"value1",
 				"value2",
 				"value3",
@@ -31,6 +32,7 @@ public void SimpleScriptValuesAreReadFromGameAndMods() {
 				"cheap_building_tier_1_cost"
 			);
 
+		Assert.Equal(-0.4d, scriptValueCollection["value_using_value_defined_below"]); // same as value2
 		Assert.Equal(0.4d, scriptValueCollection["value1"]);
 		Assert.Equal(-0.4d, scriptValueCollection["value2"]);
 		Assert.Equal(1d, scriptValueCollection["value3"]);

commonItems.UnitTests/TestFiles/CK3/game/common/script_values/script_values.txt

@@ -1,6 +1,8 @@
 @variable1 = 420
 @variable2 = 69
 
+value_using_value_defined_below = value2 # simple value using another simple value defined below
+
 value1 = 0.4
 value2 = -0.4
 value3 = 1

commonItems/ScriptValueCollection.cs

@@ -16,16 +16,28 @@ public class ScriptValueCollection : IReadOnlyDictionary<string, double> {
 	/// <param name="modFilesystem">mod file system to load script values from</param>
 	public void LoadScriptValues(ModFilesystem modFilesystem) {
 		Logger.Info("Reading script values...");
-
-		var parser = new Parser();
-		parser.RegisterRegex(CommonRegexes.String, (reader, name) => {
-			var value = ParseValue(reader);
-			if (value is not null) {
-				dict[name] = (double)value;
-			}
-		});
-		parser.RegisterRegex(CommonRegexes.Catchall, ParserHelpers.IgnoreAndLogItem);
-		parser.ParseGameFolder("common/script_values", modFilesystem, "txt", recursive: true);
+
+		// Some values can be used in other values before they are defined, for example:
+		//		scheme_agent_general_bonuses_contribution_score_bonus_max_value = agent_max_skill_value
+		//		agent_max_skill_value = 10
+		// To handle this, we read the script values multiple times until no new values are added.
+		int addedValuesCount;
+		do {
+			addedValuesCount = 0;
+
+			var parser = new Parser();
+			parser.RegisterRegex(CommonRegexes.String, (reader, name) => {
+				var value = ParseValue(reader);
+				if (value is not null) {
+					if (!dict.ContainsKey(name)) {
+						++addedValuesCount;
+					}
+					dict[name] = (double)value;
+				}
+			});
+			parser.RegisterRegex(CommonRegexes.Catchall, ParserHelpers.IgnoreAndLogItem);
+			parser.ParseGameFolder("common/script_values", modFilesystem, "txt", recursive: true);
+		} while (addedValuesCount > 0);
 	}
 
 	private double? ParseValue(BufferedReader reader) {

commonItems/commonItems.csproj

@@ -6,7 +6,7 @@
 
     <GeneratePackageOnBuild>False</GeneratePackageOnBuild>
     <PackageId>PGCG.$(AssemblyName)</PackageId>
-    <Version>14.1.1</Version>
+    <Version>14.1.2</Version>
     <Authors>PGCG</Authors>
     <PackageProjectUrl>https://github.com/ParadoxGameConverters/commonItems.NET</PackageProjectUrl>
     <RepositoryUrl>https://github.com/ParadoxGameConverters/commonItems.NET</RepositoryUrl>