Sort dependencies first in modpacks

KSP-CKAN · Nov 7, 2024 · 5f5c81b · 5f5c81b
1 parent d5fd5d7
commit 5f5c81b
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 12 deletions.
diff --git a/Core/Registry/RegistryManager.cs b/Core/Registry/RegistryManager.cs
@@ -501,12 +501,20 @@ public CkanModule GenerateModpack(bool recommends = false, bool with_versions =
                 release_date          = DateTime.Now,
             };
 
-            var rels = registry.InstalledModules
-                .Where(inst => !inst.Module.IsDLC && !inst.AutoInstalled && IsAvailable(inst))
-                .OrderBy(inst => inst.identifier, StringComparer.OrdinalIgnoreCase)
-                .Select(with_versions ? (Func<InstalledModule, RelationshipDescriptor>) RelationshipWithVersion
-                                      : RelationshipWithoutVersion)
-                .ToList();
+            var mods = registry.InstalledModules
+                               .Where(inst => !inst.Module.IsDLC && !inst.AutoInstalled && IsAvailable(inst))
+                               .Select(inst => inst.Module)
+                               .ToHashSet();
+            // Sort dependencies before dependers
+            var resolver = new RelationshipResolver(mods, null,
+                                                    RelationshipResolverOptions.ConflictsOpts(),
+                                                    registry, gameInstance.game, gameInstance.VersionCriteria());
+            var rels = resolver.ModList()
+                               .Intersect(mods)
+                               .Select(with_versions ? (Func<CkanModule, RelationshipDescriptor>)
+                                                       RelationshipWithVersion
+                                                     : RelationshipWithoutVersion)
+                               .ToList();
 
             if (recommends)
             {
@@ -535,17 +543,17 @@ private bool IsAvailable(InstalledModule inst)
             }
         }
 
-        private RelationshipDescriptor RelationshipWithVersion(InstalledModule inst)
+        private RelationshipDescriptor RelationshipWithVersion(CkanModule mod)
             => new ModuleRelationshipDescriptor()
             {
-                name    = inst.identifier,
-                version = inst.Module.version,
+                name    = mod.identifier,
+                version = mod.version,
             };
 
-        private RelationshipDescriptor RelationshipWithoutVersion(InstalledModule inst)
+        private RelationshipDescriptor RelationshipWithoutVersion(CkanModule mod)
             => new ModuleRelationshipDescriptor()
             {
-                name = inst.identifier,
+                name = mod.identifier,
             };
 
         /// <summary>

diff --git a/GUI/Controls/EditModpack.cs b/GUI/Controls/EditModpack.cs
@@ -103,6 +103,7 @@ private void LoadRelationships(CkanModule module, IRegistryQuerier registry)
                         && !module.recommends.Any(rel => rel.ContainsAny(ids))
                         && !module.suggests.Any(rel => rel.ContainsAny(ids));
                 })
+                .OrderBy(kvp => kvp.Key)
                 .Select(kvp => (RelationshipDescriptor) new ModuleRelationshipDescriptor()
                     {
                         name    = kvp.Key,
@@ -141,7 +142,6 @@ private void AddGroup(List<RelationshipDescriptor> relationships,
             {
                 RelationshipsListView.Items.AddRange(
                     relationships.OfType<ModuleRelationshipDescriptor>()
-                                 .OrderBy(r => r.name)
                                  .Select(r => new ListViewItem(new string[]
                                      {
                                          r.name,