Merge pull request #163 from SubnauticaModding/FixDependencyLoading

Proper recursive dependency checks

Author: PrimeSonic

QModManager/Patching/ManifestValidator.cs

@@ -29,7 +29,7 @@ public void ValidateManifest(QMod mod)
             if (mod.PatchMethods.Count > 0)
                 return;
 
-            Logger.Debug($"Validating mod in '{mod.SubDirectory}'");
+            Logger.Debug($"Validating mod '{mod.Id}'");
             if (string.IsNullOrEmpty(mod.Id) ||
                 string.IsNullOrEmpty(mod.DisplayName) ||
                 string.IsNullOrEmpty(mod.Author))
@@ -155,6 +155,16 @@ public void CheckRequiredMods(QMod mod)
             }
 
             mod.RequiredMods = requiredMods.Values;
+            if (Logger.DebugLogsEnabled && requiredMods.Count > 0)
+            {
+                string msg = $"{mod.Id} has required mods: ";
+                foreach (var required in requiredMods.Values)
+                {
+                    msg += $"{required.Id} ";
+                }
+
+                Logger.Debug(msg);
+            }
         }
 
         internal ModStatus FindPatchMethods(QMod qMod)

QModManager/Patching/QModFactory.cs

@@ -141,6 +141,11 @@ private void ValidateDependencies(List<QMod> modsToLoad, QMod mod)
                     continue;
                 }
 
+                if (dependencyQMod.HasDependencies)
+                {
+                    ValidateDependencies(modsToLoad, dependencyQMod);
+                }
+
                 if (dependencyQMod.LoadedAssembly == null)
                 {
                     // Dependency hasn't been validated yet