Added a warning when not loading the DATA_EXPAND node due to validati…

…on errors. Partially addresses #653.
jrossignol · Mar 15, 2018 · ffe9e4e · ffe9e4e
1 parent 6654558
commit ffe9e4e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -1,5 +1,6 @@
 Contract Configurator 1.24.1
 - Fix error message when despawning spawned Kerbals (thanks steve_v).
+- Added a warning message when validation errors prevent a DATA_EXPAND node from being parsed (thanks gyf1214).
 
 Contract Configurator 1.24.0
 - Recompile for KSP 1.4.0

diff --git a/source/ContractConfigurator/ContractType.cs b/source/ContractConfigurator/ContractType.cs
@@ -404,6 +404,10 @@ public bool Load(ConfigNode configNode)
                                     {
                                         LoggingUtil.LogWarning(this, ErrorPrefix() + ": Received an empty list of values when trying to do a DATA_EXPAND");
                                     }
+                                    else
+                                    {
+                                        LoggingUtil.LogWarning(this, ErrorPrefix() + ": Not expanding DATA_EXPAND node as the contract had validation errors.");
+                                    }
                                     valid = false;
                                     break;
                                 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -404,6 +404,10 @@ public bool Load(ConfigNode configNode) @@
                                         {
                                             LoggingUtil.LogWarning(this, ErrorPrefix() + ": Received an empty list of values when trying to do a DATA_EXPAND");
                                         }
+                                        else
+                                        {
+                                            LoggingUtil.LogWarning(this, ErrorPrefix() + ": Not expanding DATA_EXPAND node as the contract had validation errors.");
+                                        }
                                         valid = false;
                                         break;
                                     }
@@ Expand Down @@