From f7697a06cd2a92392552d061ccd7a4871d79867f Mon Sep 17 00:00:00 2001
From: cat-bro <cathjb@gmail.com>
Date: Thu, 1 Oct 2020 14:41:25 +1000
Subject: [PATCH 1/2] sort installed tools by ctx_rev prior to selecting the
 first from each name/owner

sort installed tools by ctx_rev prior to selecting the first from each name/owner

add some ;;;;;

prettier --write

another prettier file
---
 .../components/Toolshed/InstalledList/Index.vue  | 10 +++-------
 client/src/components/Toolshed/services.js       | 16 ++++++++++++++--
 client/src/components/Toolshed/services.test.js  |  7 ++++++-
 3 files changed, 23 insertions(+), 10 deletions(-)

diff --git a/client/src/components/Toolshed/InstalledList/Index.vue b/client/src/components/Toolshed/InstalledList/Index.vue
index 5cd31cedf64..5f61b916b61 100644
--- a/client/src/components/Toolshed/InstalledList/Index.vue
+++ b/client/src/components/Toolshed/InstalledList/Index.vue
@@ -33,9 +33,7 @@
                     <template v-slot:cell(name)="row">
                         <b-link href="#" role="button" class="font-weight-bold" @click="row.toggleDetails">
                             <div v-if="!isLatest(row.item)">
-                                <b-badge variant="danger" class="mb-2">
-                                    Newer version available!
-                                </b-badge>
+                                <b-badge variant="danger" class="mb-2"> Newer version available! </b-badge>
                             </div>
                             <div class="name">{{ row.item.name }}</div>
                         </b-link>
@@ -49,9 +47,7 @@
                     No matching entries found for: <span class="font-weight-bold">{{ this.filter }}</span
                     >.
                 </div>
-                <div v-if="showNotAvailable">
-                    No installed repositories found.
-                </div>
+                <div v-if="showNotAvailable">No installed repositories found.</div>
             </div>
         </div>
     </div>
@@ -123,7 +119,7 @@ export default {
         load() {
             this.loading = true;
             this.services
-                .getInstalledRepositories()
+                .getInstalledRepositories({ selectLatest: true })
                 .then((repositories) => {
                     this.repositories = repositories;
                     this.nRepositories = repositories.length;
diff --git a/client/src/components/Toolshed/services.js b/client/src/components/Toolshed/services.js
index 0aab5f52fc9..0c78f080134 100644
--- a/client/src/components/Toolshed/services.js
+++ b/client/src/components/Toolshed/services.js
@@ -78,7 +78,7 @@ export class Services {
         const url = `${getAppRoot()}api/tool_shed_repositories/?uninstalled=False`;
         try {
             const response = await axios.get(url);
-            const repositories = this._groupByNameOwner(response.data, options.filter);
+            const repositories = this._groupByNameOwner(response.data, options.filter, options.selectLatest);
             this._fixToolshedUrls(repositories, Galaxy.config.tool_shed_urls);
             return repositories;
         } catch (e) {
@@ -125,7 +125,19 @@ export class Services {
             rethrowSimple(e);
         }
     }
-    _groupByNameOwner(incoming, filter) {
+    _groupByNameOwner(incoming, filter, selectLatest) {
+        if (selectLatest) {
+            const getSortValue = (x, y) => {
+                return x == y ? 0 : x < y ? -1 : 1;
+            };
+            incoming = incoming.sort((a, b) => {
+                return (
+                    getSortValue(a.name, b.name) ||
+                    getSortValue(a.owner, b.owner) ||
+                    getSortValue(parseInt(b.ctx_rev), parseInt(a.ctx_rev))
+                );
+            });
+        }
         const hash = {};
         const repositories = [];
         incoming.forEach((x) => {
diff --git a/client/src/components/Toolshed/services.test.js b/client/src/components/Toolshed/services.test.js
index ba17d54b19a..ed3d573bffd 100644
--- a/client/src/components/Toolshed/services.test.js
+++ b/client/src/components/Toolshed/services.test.js
@@ -8,6 +8,7 @@ describe("Toolshed service helpers", () => {
             status: "status_0_0",
             description: "description_0_0",
             tool_shed: "url_1",
+            ctx_rev: "1",
         },
         {
             name: "name_0",
@@ -15,6 +16,7 @@ describe("Toolshed service helpers", () => {
             status: "status_0_1",
             description: "description_0_1",
             tool_shed: "url_1",
+            ctx_rev: "3",
         },
         {
             name: "name_1",
@@ -22,6 +24,7 @@ describe("Toolshed service helpers", () => {
             status: "status_1",
             description: "description_1",
             tool_shed: "url_2",
+            ctx_rev: "42",
         },
         {
             name: "name_2",
@@ -29,6 +32,7 @@ describe("Toolshed service helpers", () => {
             status: "Installed",
             description: "description_2",
             tool_shed: "url_2",
+            ctx_rev: "1",
         },
     ];
     const urls = ["http://url_1.com", "http://url_2.com"];
@@ -36,11 +40,12 @@ describe("Toolshed service helpers", () => {
     it("test fix toolshed helper", () => {
         const services = new Services();
         const filter = (x) => x.status !== "Installed";
-        const grouped = services._groupByNameOwner(incoming, filter);
+        const grouped = services._groupByNameOwner(incoming, filter, true);
         services._fixToolshedUrls(grouped, urls);
         expect(grouped.length).toBe(2);
         expect(grouped[0].name).toBe("name_0");
         expect(grouped[0].tool_shed_url).toBe("http://url_1.com");
+        expect(grouped[0].ctx_rev).toBe("3");
         expect(grouped[1].name === "name_1");
         expect(grouped[1].tool_shed_url).toBe("http://url_2.com");
     });

From 60a83c446d34eb75c3c2b3565716a0b61c173d75 Mon Sep 17 00:00:00 2001
From: mvdbeek <m.vandenbeek@gmail.com>
Date: Tue, 6 Oct 2020 10:03:24 +0200
Subject: [PATCH 2/2] Also order repository revisions on the backend

---
 lib/galaxy/webapps/galaxy/api/tool_shed_repositories.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/lib/galaxy/webapps/galaxy/api/tool_shed_repositories.py b/lib/galaxy/webapps/galaxy/api/tool_shed_repositories.py
index ba5b5a99966..8755ccdc863 100644
--- a/lib/galaxy/webapps/galaxy/api/tool_shed_repositories.py
+++ b/lib/galaxy/webapps/galaxy/api/tool_shed_repositories.py
@@ -6,7 +6,11 @@ from paste.httpexceptions import (
     HTTPBadRequest,
     HTTPForbidden
 )
-from sqlalchemy import and_
+from sqlalchemy import (
+    and_,
+    cast,
+    Integer,
+)
 
 from galaxy import (
     exceptions,
@@ -82,7 +86,8 @@ class ToolShedRepositoriesController(BaseAPIController):
             clause_list.append(self.app.install_model.ToolShedRepository.table.c.uninstalled == util.asbool(kwd.get('uninstalled')))
         tool_shed_repository_dicts = []
         query = trans.install_model.context.query(self.app.install_model.ToolShedRepository) \
-                                           .order_by(self.app.install_model.ToolShedRepository.table.c.name)
+                                           .order_by(self.app.install_model.ToolShedRepository.table.c.name) \
+                                           .order_by(cast(self.app.install_model.ToolShedRepository.ctx_rev, Integer).desc())
         if len(clause_list) > 0:
             query = query.filter(and_(*clause_list))
         for tool_shed_repository in query.all():