don't load invalid definitions

KSPModdingLibs · Jul 27, 2024 · 43e6cab · 43e6cab
1 parent 05fb97d
commit 43e6cab
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/Source/MaterialDef.cs b/Source/MaterialDef.cs
@@ -45,7 +45,10 @@ public static void Load()
 			var def = new MaterialDef(node);
 			if (string.IsNullOrEmpty(def.name)) {
 				Debug.LogError("[Shabby] material definition must have a valid name");
-			} else {
+			} else if (!def.isValid) {
+				Debug.LogError($"[Shabby][MaterialDef {def.name}] removing invalid definition");
+			}
+			else {
 				items[def.name] = def;
 			}
 		}

diff --git a/Source/MaterialReplacement.cs b/Source/MaterialReplacement.cs
@@ -65,7 +65,7 @@ static void Postfix(ref GameObject __result, ConfigNode partCfg)
 		foreach (ConfigNode node in partCfg.nodes) {
 			if (node.name != replacementNodeName) continue;
 			var replacement = new MaterialReplacement(node);
-			if (replacement.materialDef?.isValid == true) replacements.Add(replacement);
+			if (replacement.materialDef != null) replacements.Add(replacement);
 		}
 
 		// Apply blanket replacements or material name replacements.