Merge #4007 Sort mods satisfying the same recommendation by download …

…count
KSP-CKAN · Jan 24, 2024 · 0bd6f6c · 0bd6f6c
2 parents 5d4a210 + 9bd84ab
commit 0bd6f6c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@ All notable changes to this project will be documented in this file.
 ### Features
 
 - [Updater] Support dev builds for auto updater (#3997 by: HebaruSan)
+- [GUI] Sort mods satisfying the same recommendation by download count (#4007 by: HebaruSan)
 
 ### Bugfixes
 

diff --git a/Core/Registry/Registry.cs b/Core/Registry/Registry.cs
@@ -814,6 +814,10 @@ public List<CkanModule> LatestAvailableWithProvides(
                     .Select(am => am.Latest(gameVersion, relationship_descriptor,
                                             installed, toInstall))
                     .Where(m => m?.ProvidesList?.Contains(identifier) ?? false)
+                    // Put the most popular one on top
+                    .OrderByDescending(m => repoDataMgr.GetDownloadCount(Repositories.Values,
+                                                                         m.identifier)
+                                            ?? 0)
                     .ToList();
             }
             else