From 3541c304c6c32054b7e874d9d1d7b5bfd33568f8 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Sun, 16 Oct 2022 13:27:59 -0400
Subject: [PATCH 01/10] Consistently serialize tags in Visualization

---
 lib/galaxy/model/__init__.py                    | 4 ++--
 lib/galaxy/webapps/galaxy/api/visualizations.py | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/galaxy/model/__init__.py b/lib/galaxy/model/__init__.py
index 2e3dcd5116a..82cda0e6645 100644
--- a/lib/galaxy/model/__init__.py
+++ b/lib/galaxy/model/__init__.py
@@ -8836,7 +8836,7 @@ class CloudAuthz(Base, _HasTable):
         )
 
 
-class Page(Base, Dictifiable, RepresentById):
+class Page(Base, HasTags, Dictifiable, RepresentById):
     __tablename__ = "page"
     __table_args__ = (Index("ix_page_slug", "slug", mysql_length=200),)
 
@@ -8942,7 +8942,7 @@ class PageUserShareAssociation(Base, UserShareAssociation):
     page = relationship("Page", back_populates="users_shared_with")
 
 
-class Visualization(Base, RepresentById):
+class Visualization(Base, HasTags, RepresentById):
     __tablename__ = "visualization"
     __table_args__ = (
         Index("ix_visualization_dbkey", "dbkey", mysql_length=200),
diff --git a/lib/galaxy/webapps/galaxy/api/visualizations.py b/lib/galaxy/webapps/galaxy/api/visualizations.py
index 3d854f7fb20..2a588e965c6 100644
--- a/lib/galaxy/webapps/galaxy/api/visualizations.py
+++ b/lib/galaxy/webapps/galaxy/api/visualizations.py
@@ -191,6 +191,7 @@ class VisualizationsController(BaseGalaxyAPIController, UsesVisualizationMixin,
             slug=visualization.slug,
         )
         dictionary["username"] = visualization.user.username
+        dictionary["tags"] = visualization.make_tag_string_list()
         dictionary["annotation"] = self.get_item_annotation_str(trans.sa_session, trans.user, visualization)
         # need to encode ids in revisions as well
         encoded_revisions = []

From e35ffce78be64eb8ef996871a4f3c2e10a409c23 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Sun, 16 Oct 2022 14:39:50 -0400
Subject: [PATCH 02/10] Restore rating stars and display for published items

---
 client/package.json                           |  1 +
 client/src/components/Common/Published.vue    | 35 ++++++++++++----
 client/src/components/Common/Ratings.vue      | 42 +++++++++++++++++++
 .../components/History/HistoryPublished.vue   | 11 ++++-
 .../components/PageDisplay/PageDisplay.vue    |  9 +++-
 .../Visualizations/VisualizationPublished.vue | 11 ++++-
 .../components/Workflow/WorkflowPublished.vue | 11 ++++-
 client/yarn.lock                              |  9 +++-
 .../webapps/galaxy/controllers/history.py     |  1 -
 9 files changed, 113 insertions(+), 17 deletions(-)
 create mode 100644 client/src/components/Common/Ratings.vue

diff --git a/client/package.json b/client/package.json
index 1ea15517c35..bd139e04005 100644
--- a/client/package.json
+++ b/client/package.json
@@ -91,6 +91,7 @@
     "vue-prismjs": "^1.2.0",
     "vue-router": "^3.5.3",
     "vue-rx": "^6.2.0",
+    "vue-star-rating": "^1.7.0",
     "vue-virtual-scroll-list": "^2.3.3",
     "vuedraggable": "2.24.3",
     "vuex": "^3.6.2",
diff --git a/client/src/components/Common/Published.vue b/client/src/components/Common/Published.vue
index 063b53a6f10..cd83592585c 100644
--- a/client/src/components/Common/Published.vue
+++ b/client/src/components/Common/Published.vue
@@ -11,8 +11,8 @@
             <div class="m-3">
                 <div v-if="modelTitle">
                     <h1 class="h-sm">About this {{ modelTitle }}</h1>
-                    <h2 class="h-md">{{ details.title || details.name }}</h2>
-                    <StatelessTags v-if="details.tags" class="tags mt-2" :value="details.tags" :disabled="true" />
+                    <h2 class="h-md">{{ item.title || item.name }}</h2>
+                    <StatelessTags v-if="item.tags" class="tags mt-2" :value="item.tags" :disabled="true" />
                     <br />
                     <h2 class="h-sm">Author</h2>
                     <div>{{ owner }}</div>
@@ -24,8 +24,15 @@
                     <div>
                         <router-link :to="publishedByUser"> Published {{ plural }} by {{ owner }}. </router-link>
                     </div>
+                    <br />
+                    <ratings
+                        v-if="hasRatingData"
+                        :ave-item-rating="aveItemRating"
+                        :user-item-rating="userItemRating"
+                        :num-ratings="numRatings"
+                        @set-rating="$emit('set-rating', $event)" />
                 </div>
-                <LoadingSpan v-else message="Loading details" />
+                <LoadingSpan v-else message="Loading item details" />
             </div>
         </div>
     </div>
@@ -33,28 +40,39 @@
 
 <script>
 import { StatelessTags } from "components/Tags";
+import Ratings from "components/Common/Ratings";
 import LoadingSpan from "components/LoadingSpan";
 export default {
     components: {
         LoadingSpan,
+        Ratings,
         StatelessTags,
     },
     props: {
-        details: {
+        item: {
             type: Object,
             required: true,
         },
     },
     computed: {
+        aveItemRating() {
+            return Number(this.item.ave_item_rating);
+        },
+        hasRatingData() {
+            return this.item.num_ratings !== undefined;
+        },
         modelTitle() {
-            const modelClass = this.details ? this.details.model_class : "Item";
+            const modelClass = this.item ? this.item.model_class : "Item";
             if (modelClass == "StoredWorkflow") {
                 return "Workflow";
             }
             return modelClass;
         },
+        numRatings() {
+            return parseInt(this.item.num_ratings);
+        },
         owner() {
-            return this.details.owner || this.details.username || "Unavailable";
+            return this.item.owner || this.item.username || "Unavailable";
         },
         plural() {
             if (this.modelTitle == "History") {
@@ -66,11 +84,14 @@ export default {
             return this.plural.toLowerCase();
         },
         publishedByUser() {
-            return `/${this.pluralPath}/list_published?f-username=${this.details.username}`;
+            return `/${this.pluralPath}/list_published?f-username=${this.item.username}`;
         },
         urlAll() {
             return `/${this.pluralPath}/list_published`;
         },
+        userItemRating() {
+            return Number(this.item.user_item_rating);
+        },
     },
 };
 </script>
diff --git a/client/src/components/Common/Ratings.vue b/client/src/components/Common/Ratings.vue
new file mode 100644
index 00000000000..2cacefb5963
--- /dev/null
+++ b/client/src/components/Common/Ratings.vue
@@ -0,0 +1,42 @@
+<template>
+    <div>
+        <h4>Rating</h4>
+        <div>Community ({{ numRatings }} total, {{ aveItemRating.toFixed(1) }} average):</div>
+        <star-rating read-only v-model="aveItemRating" :star-size="25" :show-rating="false" />
+        <div class="mt-2">Yours:</div>
+        <star-rating v-model="currentValue" :star-size="25" :show-rating="false" />
+    </div>
+</template>
+<script>
+import StarRating from "vue-star-rating";
+import { urlData } from "utils/url";
+export default {
+    components: {
+        StarRating,
+    },
+    props: {
+        aveItemRating: {
+            type: Number,
+            default: 0,
+        },
+        userItemRating: {
+            type: Number,
+            default: 0,
+        },
+        numRatings: {
+            type: Number,
+            default: 0,
+        },
+    },
+    computed: {
+        currentValue: {
+            get() {
+                return this.userItemRating;
+            },
+            set(val) {
+                this.$emit("set-rating", val);
+            },
+        },
+    },
+};
+</script>
diff --git a/client/src/components/History/HistoryPublished.vue b/client/src/components/History/HistoryPublished.vue
index 3ccbdd55779..c827912aa7c 100644
--- a/client/src/components/History/HistoryPublished.vue
+++ b/client/src/components/History/HistoryPublished.vue
@@ -1,5 +1,5 @@
 <template>
-    <Published :details="history">
+    <Published :item="history" @set-rating="onSetRating">
         <template v-slot>
             <HistoryView :id="id" />
         </template>
@@ -29,9 +29,16 @@ export default {
     },
     created() {
         const url = `/api/histories/${this.id}`;
+        const query = this.$route.query;
         urlData({ url }).then((data) => {
-            this.history = data;
+            this.history = { ...data, ...query };
         });
     },
+    methods: {
+        onSetRating(newRating) {
+            const url = `/history/rate_async?id=${this.id}&rating=${newRating}`;
+            urlData({ url });
+        },
+    },
 };
 </script>
diff --git a/client/src/components/PageDisplay/PageDisplay.vue b/client/src/components/PageDisplay/PageDisplay.vue
index a5dae1998b0..56c98845667 100644
--- a/client/src/components/PageDisplay/PageDisplay.vue
+++ b/client/src/components/PageDisplay/PageDisplay.vue
@@ -1,6 +1,6 @@
 <template>
     <config-provider v-slot="{ config, loading }">
-        <Published :details="page">
+        <Published :item="page" @set-rating="onSetRating">
             <template v-slot>
                 <div v-if="!loading">
                     <markdown
@@ -56,14 +56,19 @@ export default {
         },
     },
     created() {
+        const query = this.$route.query;
         urlData({ url: this.dataUrl }).then((data) => {
-            this.page = { ...data };
+            this.page = { ...data, ...query };
         });
     },
     methods: {
         onEdit() {
             window.location = safePath(this.editUrl);
         },
+        onSetRating(newRating) {
+            const url = `/page/rate_async?id=${this.id}&rating=${newRating}`;
+            urlData({ url });
+        },
         stsUrl(config) {
             return `${this.dataUrl}/prepare_download`;
         },
diff --git a/client/src/components/Visualizations/VisualizationPublished.vue b/client/src/components/Visualizations/VisualizationPublished.vue
index 199d94d7096..b2bd32cf44d 100644
--- a/client/src/components/Visualizations/VisualizationPublished.vue
+++ b/client/src/components/Visualizations/VisualizationPublished.vue
@@ -1,5 +1,5 @@
 <template>
-    <Published :details="visualization">
+    <Published :item="visualization" @set-rating="onSetRating">
         <template v-slot>
             <CenterFrame :src="getUrl" />
         </template>
@@ -33,9 +33,16 @@ export default {
     },
     created() {
         const url = `/api/visualizations/${this.id}`;
+        const query = this.$route.query;
         urlData({ url }).then((data) => {
-            this.visualization = data;
+            this.visualization = { ...data, ...query };
         });
     },
+    methods: {
+        onSetRating(newRating) {
+            const url = `/visualization/rate_async?id=${this.id}&rating=${newRating}`;
+            urlData({ url });
+        },
+    },
 };
 </script>
diff --git a/client/src/components/Workflow/WorkflowPublished.vue b/client/src/components/Workflow/WorkflowPublished.vue
index 5547c9181a0..2340d0a7083 100644
--- a/client/src/components/Workflow/WorkflowPublished.vue
+++ b/client/src/components/Workflow/WorkflowPublished.vue
@@ -1,5 +1,5 @@
 <template>
-    <Published :details="workflow">
+    <Published :item="workflow" @set-rating="onSetRating">
         <template v-slot>
             <WorkflowDisplay :args="{ workflow_id: id }" :workflow="workflow" :expanded="true" />
         </template>
@@ -29,9 +29,16 @@ export default {
     },
     created() {
         const url = `/api/workflows/${this.id}`;
+        const query = this.$route.query;
         urlData({ url }).then((data) => {
-            this.workflow = data;
+            this.workflow = { ...data, ...query };
         });
     },
+    methods: {
+        onSetRating(newRating) {
+            const url = `/workflow/rate_async?id=${this.id}&rating=${newRating}`;
+            urlData({ url });
+        },
+    },
 };
 </script>
diff --git a/client/yarn.lock b/client/yarn.lock
index df24b3709ee..5e543d7f1ab 100644
--- a/client/yarn.lock
+++ b/client/yarn.lock
@@ -10003,6 +10003,13 @@ vue-rx@^6.2.0:
   resolved "https://registry.yarnpkg.com/vue-rx/-/vue-rx-6.2.0.tgz#c3b86462b252626edd16417385bb9054a3a23acb"
   integrity sha512-tpKUcqS5IUYS+HsdbR5TlE5LL9PK4zwlplEtmMeydnbqaUTa9ciLMplJXAtFSiQw1vuURoyEJmCXqMxaVEIloQ==
 
+vue-star-rating@^1.7.0:
+  version "1.7.0"
+  resolved "https://registry.yarnpkg.com/vue-star-rating/-/vue-star-rating-1.7.0.tgz#07576aa1a83d54d4e9ea80eebccf058ff3c868b6"
+  integrity sha512-2qdONqnIlvFzJoallfr7VuFRB6hBzYgBVEbXC3yYZz2TfMhx36WtYQ5p7JRjW+pzZx2PGQYZ75UFoBE0jwIANw==
+  dependencies:
+    vue "^2.3.3"
+
 vue-style-loader@^4.1.0:
   version "4.1.3"
   resolved "https://registry.yarnpkg.com/vue-style-loader/-/vue-style-loader-4.1.3.tgz#6d55863a51fa757ab24e89d9371465072aa7bc35"
@@ -10029,7 +10036,7 @@ vue-virtual-scroll-list@^2.3.3:
   resolved "https://registry.yarnpkg.com/vue-virtual-scroll-list/-/vue-virtual-scroll-list-2.3.3.tgz#290f0d04be814b93585f2ef232097b671141e5b1"
   integrity sha512-heuwlZ+lEdcVSp66CpVXnyNdsiHl/XU1cvqJb0yUHGlLv6DGK8OX16FH8I3w/wKNXtHSPF9itdBBGJFrXkyHeg==
 
-vue@2.7.8, vue@^2.6.10, vue@^2.7.8:
+vue@2.7.8, vue@^2.3.3, vue@^2.6.10, vue@^2.7.8:
   version "2.7.8"
   resolved "https://registry.yarnpkg.com/vue/-/vue-2.7.8.tgz#34e06553137611d8cecc4b0cd78b7a59f80b1299"
   integrity sha512-ncwlZx5qOcn754bCu5/tS/IWPhXHopfit79cx+uIlLMyt3vCMGcXai5yCG5y+I6cDmEj4ukRYyZail9FTQh7lQ==
diff --git a/lib/galaxy/webapps/galaxy/controllers/history.py b/lib/galaxy/webapps/galaxy/controllers/history.py
index 11962483e9d..5b0e273aea1 100644
--- a/lib/galaxy/webapps/galaxy/controllers/history.py
+++ b/lib/galaxy/webapps/galaxy/controllers/history.py
@@ -752,7 +752,6 @@ class HistoryController(BaseUIController, SharableMixin, UsesAnnotations, UsesIt
         # Rate history.
         self.rate_item(trans.sa_session, trans.get_user(), history, rating)
         return self.get_ave_item_rating_data(trans.sa_session, history)
-        # TODO: used in display_base.mako
 
     @web.expose
     @web.json

From 4aa66433b161391759321c4355343c9b4cd4b0c9 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Sun, 16 Oct 2022 19:11:51 -0400
Subject: [PATCH 03/10] Add email to sharable serializer

---
 lib/galaxy/managers/sharable.py                 | 6 ++++++
 lib/galaxy/webapps/galaxy/api/visualizations.py | 1 +
 2 files changed, 7 insertions(+)

diff --git a/lib/galaxy/managers/sharable.py b/lib/galaxy/managers/sharable.py
index 53f90351dfa..691f0231025 100644
--- a/lib/galaxy/managers/sharable.py
+++ b/lib/galaxy/managers/sharable.py
@@ -394,6 +394,7 @@ class SharableModelSerializer(
         self.serializers.update(
             {
                 "id": self.serialize_id,
+                "email": self.serialize_email,
                 "title": self.serialize_title,
                 "username": self.serialize_username,
                 "username_and_slug": self.serialize_username_and_slug,
@@ -403,6 +404,11 @@ class SharableModelSerializer(
         # these use the default serializer but must still be white-listed
         self.serializable_keyset.update(["importable", "published", "slug"])
 
+    def serialize_email(self, item, key, **context):
+        if not (item.user and item.user.email):
+            return None
+        return item.user.email
+
     def serialize_title(self, item, key, **context):
         if hasattr(item, "title"):
             return item.title
diff --git a/lib/galaxy/webapps/galaxy/api/visualizations.py b/lib/galaxy/webapps/galaxy/api/visualizations.py
index 2a588e965c6..15daeaa1087 100644
--- a/lib/galaxy/webapps/galaxy/api/visualizations.py
+++ b/lib/galaxy/webapps/galaxy/api/visualizations.py
@@ -191,6 +191,7 @@ class VisualizationsController(BaseGalaxyAPIController, UsesVisualizationMixin,
             slug=visualization.slug,
         )
         dictionary["username"] = visualization.user.username
+        dictionary["email"] = visualization.user.email
         dictionary["tags"] = visualization.make_tag_string_list()
         dictionary["annotation"] = self.get_item_annotation_str(trans.sa_session, trans.user, visualization)
         # need to encode ids in revisions as well

From c85f896d8124d7d6cdc5c952a31465c5416a4b29 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Sun, 16 Oct 2022 19:50:09 -0400
Subject: [PATCH 04/10] Add hashed email for gravatar to serializer

---
 lib/galaxy/managers/sharable.py                 |  8 ++++----
 lib/galaxy/schema/schema.py                     | 15 ++++++++++-----
 lib/galaxy/webapps/galaxy/api/visualizations.py |  3 ++-
 3 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/lib/galaxy/managers/sharable.py b/lib/galaxy/managers/sharable.py
index 691f0231025..4d78532eb5e 100644
--- a/lib/galaxy/managers/sharable.py
+++ b/lib/galaxy/managers/sharable.py
@@ -24,8 +24,8 @@ from sqlalchemy import (
     func,
     true,
 )
-
 from galaxy import exceptions
+from galaxy.web.framework.helpers import md5
 from galaxy.managers import (
     annotatable,
     base,
@@ -394,8 +394,8 @@ class SharableModelSerializer(
         self.serializers.update(
             {
                 "id": self.serialize_id,
-                "email": self.serialize_email,
                 "title": self.serialize_title,
+                "userhash": self.serialize_userhash,
                 "username": self.serialize_username,
                 "username_and_slug": self.serialize_username_and_slug,
                 "users_shared_with": self.serialize_users_shared_with,
@@ -404,10 +404,10 @@ class SharableModelSerializer(
         # these use the default serializer but must still be white-listed
         self.serializable_keyset.update(["importable", "published", "slug"])
 
-    def serialize_email(self, item, key, **context):
+    def serialize_userhash(self, item, key, **context):
         if not (item.user and item.user.email):
             return None
-        return item.user.email
+        return md5(item.user.email)
 
     def serialize_title(self, item, key, **context):
         if hasattr(item, "title"):
diff --git a/lib/galaxy/schema/schema.py b/lib/galaxy/schema/schema.py
index cce48d50316..fd42999de1a 100644
--- a/lib/galaxy/schema/schema.py
+++ b/lib/galaxy/schema/schema.py
@@ -3014,11 +3014,6 @@ class SharingStatus(Model):
         title="Importable",
         description="Whether this resource can be published using a link.",
     )
-    username: Optional[str] = Field(
-        None,
-        title="Username",
-        description="The owner's username.",
-    )
     published: bool = Field(
         ...,
         title="Published",
@@ -3029,6 +3024,16 @@ class SharingStatus(Model):
         title="Users shared with",
         description="The list of encoded ids for users the resource has been shared.",
     )
+    userhash: Optional[str] = Field(
+        None,
+        title="Hash",
+        description="The encoded email of the user.",
+    )
+    username: Optional[str] = Field(
+        None,
+        title="Username",
+        description="The owner's username.",
+    )
     username_and_slug: Optional[str] = Field(
         None,
         title="Username and slug",
diff --git a/lib/galaxy/webapps/galaxy/api/visualizations.py b/lib/galaxy/webapps/galaxy/api/visualizations.py
index 15daeaa1087..18b744008c3 100644
--- a/lib/galaxy/webapps/galaxy/api/visualizations.py
+++ b/lib/galaxy/webapps/galaxy/api/visualizations.py
@@ -29,6 +29,7 @@ from galaxy.schema.schema import (
     SharingStatus,
 )
 from galaxy.web import expose_api
+from galaxy.web.framework.helpers import md5
 from galaxy.webapps.base.controller import UsesVisualizationMixin
 from galaxy.webapps.base.webapp import GalaxyWebTransaction
 from galaxy.webapps.galaxy.api import (
@@ -191,7 +192,7 @@ class VisualizationsController(BaseGalaxyAPIController, UsesVisualizationMixin,
             slug=visualization.slug,
         )
         dictionary["username"] = visualization.user.username
-        dictionary["email"] = visualization.user.email
+        dictionary["userhash"] = md5(visualization.user.email)
         dictionary["tags"] = visualization.make_tag_string_list()
         dictionary["annotation"] = self.get_item_annotation_str(trans.sa_session, trans.user, visualization)
         # need to encode ids in revisions as well

From 325d12a73715730fecce4d579077b67b92ea7fa5 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Sun, 16 Oct 2022 19:58:34 -0400
Subject: [PATCH 05/10] Restore gravater in vueified published items

---
 client/src/components/Common/Published.vue    |  4 ++++
 client/src/components/Common/Ratings.vue      | 14 +++++++++-----
 .../components/PageDisplay/PageDisplay.vue    |  2 +-
 lib/galaxy/managers/histories.py              |  1 +
 lib/galaxy/managers/sharable.py               | 19 +++++++++++++++----
 lib/galaxy/managers/workflows.py              |  2 ++
 lib/galaxy/model/__init__.py                  |  7 +++++++
 lib/galaxy/schema/schema.py                   | 11 ++++++++---
 .../webapps/galaxy/api/visualizations.py      |  2 +-
 9 files changed, 48 insertions(+), 14 deletions(-)

diff --git a/client/src/components/Common/Published.vue b/client/src/components/Common/Published.vue
index cd83592585c..fbd83e16bfd 100644
--- a/client/src/components/Common/Published.vue
+++ b/client/src/components/Common/Published.vue
@@ -12,6 +12,7 @@
                 <div v-if="modelTitle">
                     <h1 class="h-sm">About this {{ modelTitle }}</h1>
                     <h2 class="h-md">{{ item.title || item.name }}</h2>
+                    <img class="float-right" :src="gravatarSource" alt="user avatar" />
                     <StatelessTags v-if="item.tags" class="tags mt-2" :value="item.tags" :disabled="true" />
                     <br />
                     <h2 class="h-sm">Author</h2>
@@ -58,6 +59,9 @@ export default {
         aveItemRating() {
             return Number(this.item.ave_item_rating);
         },
+        gravatarSource() {
+            return `https://secure.gravatar.com/avatar/${this.item.email_hash}?d=identicon`;
+        },
         hasRatingData() {
             return this.item.num_ratings !== undefined;
         },
diff --git a/client/src/components/Common/Ratings.vue b/client/src/components/Common/Ratings.vue
index 2cacefb5963..bc625fad707 100644
--- a/client/src/components/Common/Ratings.vue
+++ b/client/src/components/Common/Ratings.vue
@@ -1,15 +1,14 @@
 <template>
     <div>
         <h4>Rating</h4>
-        <div>Community ({{ numRatings }} total, {{ aveItemRating.toFixed(1) }} average):</div>
-        <star-rating read-only v-model="aveItemRating" :star-size="25" :show-rating="false" />
+        <div>Community ({{ numRatings }} total, {{ averageRating.toFixed(1) }} average):</div>
+        <star-rating read-only v-model="averageRating" :star-size="25" :show-rating="false" />
         <div class="mt-2">Yours:</div>
-        <star-rating v-model="currentValue" :star-size="25" :show-rating="false" />
+        <star-rating v-model="userRating" :star-size="25" :show-rating="false" />
     </div>
 </template>
 <script>
 import StarRating from "vue-star-rating";
-import { urlData } from "utils/url";
 export default {
     components: {
         StarRating,
@@ -29,7 +28,12 @@ export default {
         },
     },
     computed: {
-        currentValue: {
+        averageRating: {
+            get() {
+                return this.aveItemRating;
+            },
+        },
+        userRating: {
             get() {
                 return this.userItemRating;
             },
diff --git a/client/src/components/PageDisplay/PageDisplay.vue b/client/src/components/PageDisplay/PageDisplay.vue
index 56c98845667..5b6b20337f7 100644
--- a/client/src/components/PageDisplay/PageDisplay.vue
+++ b/client/src/components/PageDisplay/PageDisplay.vue
@@ -66,7 +66,7 @@ export default {
             window.location = safePath(this.editUrl);
         },
         onSetRating(newRating) {
-            const url = `/page/rate_async?id=${this.id}&rating=${newRating}`;
+            const url = `/page/rate_async?id=${this.pageId}&rating=${newRating}`;
             urlData({ url });
         },
         stsUrl(config) {
diff --git a/lib/galaxy/managers/histories.py b/lib/galaxy/managers/histories.py
index c7e143bbbc3..007f2bfe2b6 100644
--- a/lib/galaxy/managers/histories.py
+++ b/lib/galaxy/managers/histories.py
@@ -440,6 +440,7 @@ class HistorySerializer(sharable.SharableModelSerializer, deletable.PurgableSeri
             "detailed",
             [
                 "contents_url",
+                "email_hash",
                 "empty",
                 "size",
                 "user_id",
diff --git a/lib/galaxy/managers/sharable.py b/lib/galaxy/managers/sharable.py
index 4d78532eb5e..6fa9814fd1a 100644
--- a/lib/galaxy/managers/sharable.py
+++ b/lib/galaxy/managers/sharable.py
@@ -24,8 +24,8 @@ from sqlalchemy import (
     func,
     true,
 )
+
 from galaxy import exceptions
-from galaxy.web.framework.helpers import md5
 from galaxy.managers import (
     annotatable,
     base,
@@ -45,6 +45,7 @@ from galaxy.schema.schema import (
 )
 from galaxy.structured_app import MinimalManagerApp
 from galaxy.util import ready_name_for_url
+from galaxy.web.framework.helpers import md5
 
 if TYPE_CHECKING:
     from sqlalchemy.orm import Query
@@ -383,7 +384,17 @@ class SharableModelSerializer(
     def __init__(self, app, **kwargs):
         super().__init__(app, **kwargs)
         self.add_view(
-            "sharing", ["id", "title", "importable", "published", "username", "username_and_slug", "users_shared_with"]
+            "sharing",
+            [
+                "id",
+                "title",
+                "email_hash",
+                "importable",
+                "published",
+                "username",
+                "username_and_slug",
+                "users_shared_with",
+            ],
         )
 
     def add_serializers(self):
@@ -395,16 +406,16 @@ class SharableModelSerializer(
             {
                 "id": self.serialize_id,
                 "title": self.serialize_title,
-                "userhash": self.serialize_userhash,
                 "username": self.serialize_username,
                 "username_and_slug": self.serialize_username_and_slug,
                 "users_shared_with": self.serialize_users_shared_with,
+                "email_hash": self.serialize_email_hash,
             }
         )
         # these use the default serializer but must still be white-listed
         self.serializable_keyset.update(["importable", "published", "slug"])
 
-    def serialize_userhash(self, item, key, **context):
+    def serialize_email_hash(self, item, key, **context):
         if not (item.user and item.user.email):
             return None
         return md5(item.user.email)
diff --git a/lib/galaxy/managers/workflows.py b/lib/galaxy/managers/workflows.py
index 795ea44fa84..418930a4665 100644
--- a/lib/galaxy/managers/workflows.py
+++ b/lib/galaxy/managers/workflows.py
@@ -80,6 +80,7 @@ from galaxy.util.search import (
     RawTextTerm,
 )
 from galaxy.web import url_for
+from galaxy.web.framework.helpers import md5
 from galaxy.work.context import WorkRequestContext
 from galaxy.workflow.modules import (
     is_tool_module_type,
@@ -1480,6 +1481,7 @@ class WorkflowContentsManager(UsesAnnotations):
         item["name"] = workflow.name
         item["url"] = url_for("workflow", id=item["id"])
         item["owner"] = stored.user.username
+        item["email_hash"] = md5(stored.user.email)
         item["slug"] = stored.slug
         inputs = {}
         for step in workflow.input_steps:
diff --git a/lib/galaxy/model/__init__.py b/lib/galaxy/model/__init__.py
index 82cda0e6645..545c39e07ea 100644
--- a/lib/galaxy/model/__init__.py
+++ b/lib/galaxy/model/__init__.py
@@ -149,6 +149,7 @@ from galaxy.util.form_builder import (
 from galaxy.util.hash_util import new_insecure_hash
 from galaxy.util.json import safe_loads
 from galaxy.util.sanitize_html import sanitize_html
+from galaxy.web.framework.helpers import md5
 
 log = logging.getLogger(__name__)
 
@@ -8892,6 +8893,7 @@ class Page(Base, HasTags, Dictifiable, RepresentById):
         "importable",
         "deleted",
         "username",
+        "email_hash",
     ]
 
     def to_dict(self, view="element"):
@@ -8907,6 +8909,11 @@ class Page(Base, HasTags, Dictifiable, RepresentById):
     def username(self):
         return self.user.username
 
+    # email needed for hash generation
+    @property
+    def email_hash(self):
+        return md5(self.user.email)
+
 
 class PageRevision(Base, Dictifiable, RepresentById):
     __tablename__ = "page_revision"
diff --git a/lib/galaxy/schema/schema.py b/lib/galaxy/schema/schema.py
index fd42999de1a..cb011c284b3 100644
--- a/lib/galaxy/schema/schema.py
+++ b/lib/galaxy/schema/schema.py
@@ -3024,10 +3024,10 @@ class SharingStatus(Model):
         title="Users shared with",
         description="The list of encoded ids for users the resource has been shared.",
     )
-    userhash: Optional[str] = Field(
+    email_hash: Optional[str] = Field(
         None,
-        title="Hash",
-        description="The encoded email of the user.",
+        title="Encoded Email",
+        description="Encoded owner email.",
     )
     username: Optional[str] = Field(
         None,
@@ -3204,6 +3204,11 @@ class PageSummary(PageSummaryBase):
         title="Username",
         description="The name of the user owning this Page.",
     )
+    email_hash: str = Field(
+        ...,  # Required
+        title="Encoded email",
+        description="The encoded email of the user",
+    )
     published: bool = Field(
         ...,  # Required
         title="Published",
diff --git a/lib/galaxy/webapps/galaxy/api/visualizations.py b/lib/galaxy/webapps/galaxy/api/visualizations.py
index 18b744008c3..17ccd3ee490 100644
--- a/lib/galaxy/webapps/galaxy/api/visualizations.py
+++ b/lib/galaxy/webapps/galaxy/api/visualizations.py
@@ -192,7 +192,7 @@ class VisualizationsController(BaseGalaxyAPIController, UsesVisualizationMixin,
             slug=visualization.slug,
         )
         dictionary["username"] = visualization.user.username
-        dictionary["userhash"] = md5(visualization.user.email)
+        dictionary["email_hash"] = md5(visualization.user.email)
         dictionary["tags"] = visualization.make_tag_string_list()
         dictionary["annotation"] = self.get_item_annotation_str(trans.sa_session, trans.user, visualization)
         # need to encode ids in revisions as well

From c4d9194c94bd458f037484f4f67d8b3f0d88ff71 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Wed, 26 Oct 2022 19:17:47 -0400
Subject: [PATCH 06/10] Fix header class

---
 client/src/components/Common/Ratings.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client/src/components/Common/Ratings.vue b/client/src/components/Common/Ratings.vue
index bc625fad707..af04097ef4e 100644
--- a/client/src/components/Common/Ratings.vue
+++ b/client/src/components/Common/Ratings.vue
@@ -1,6 +1,6 @@
 <template>
     <div>
-        <h4>Rating</h4>
+        <h1 class="h-sm">Rating</h1>
         <div>Community ({{ numRatings }} total, {{ averageRating.toFixed(1) }} average):</div>
         <star-rating read-only v-model="averageRating" :star-size="25" :show-rating="false" />
         <div class="mt-2">Yours:</div>

From 0f5dfe6ead505258bf8042ae1959225ebaafd741 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Thu, 3 Nov 2022 12:37:15 -0400
Subject: [PATCH 07/10] Move md5 utility to util

---
 lib/galaxy/managers/sharable.py                 |  4 ++--
 lib/galaxy/managers/workflows.py                |  4 ++--
 lib/galaxy/model/__init__.py                    |  8 +++++---
 lib/galaxy/util/hash_util.py                    |  9 +++++++++
 lib/galaxy/web/framework/helpers/__init__.py    | 16 +---------------
 lib/galaxy/webapps/galaxy/api/visualizations.py |  4 ++--
 6 files changed, 21 insertions(+), 24 deletions(-)

diff --git a/lib/galaxy/managers/sharable.py b/lib/galaxy/managers/sharable.py
index 6fa9814fd1a..aebef5bb039 100644
--- a/lib/galaxy/managers/sharable.py
+++ b/lib/galaxy/managers/sharable.py
@@ -45,7 +45,7 @@ from galaxy.schema.schema import (
 )
 from galaxy.structured_app import MinimalManagerApp
 from galaxy.util import ready_name_for_url
-from galaxy.web.framework.helpers import md5
+from galaxy.util.hash_util import md5_hash_str
 
 if TYPE_CHECKING:
     from sqlalchemy.orm import Query
@@ -418,7 +418,7 @@ class SharableModelSerializer(
     def serialize_email_hash(self, item, key, **context):
         if not (item.user and item.user.email):
             return None
-        return md5(item.user.email)
+        return md5_hash_str(item.user.email)
 
     def serialize_title(self, item, key, **context):
         if hasattr(item, "title"):
diff --git a/lib/galaxy/managers/workflows.py b/lib/galaxy/managers/workflows.py
index 418930a4665..79451a454e6 100644
--- a/lib/galaxy/managers/workflows.py
+++ b/lib/galaxy/managers/workflows.py
@@ -69,6 +69,7 @@ from galaxy.tools.parameters.basic import (
     RuntimeValue,
     workflow_building_modes,
 )
+from galaxy.util.hash_util import md5_hash_str
 from galaxy.util.json import (
     safe_dumps,
     safe_loads,
@@ -80,7 +81,6 @@ from galaxy.util.search import (
     RawTextTerm,
 )
 from galaxy.web import url_for
-from galaxy.web.framework.helpers import md5
 from galaxy.work.context import WorkRequestContext
 from galaxy.workflow.modules import (
     is_tool_module_type,
@@ -1481,7 +1481,7 @@ class WorkflowContentsManager(UsesAnnotations):
         item["name"] = workflow.name
         item["url"] = url_for("workflow", id=item["id"])
         item["owner"] = stored.user.username
-        item["email_hash"] = md5(stored.user.email)
+        item["email_hash"] = md5_hash_str(stored.user.email)
         item["slug"] = stored.slug
         inputs = {}
         for step in workflow.input_steps:
diff --git a/lib/galaxy/model/__init__.py b/lib/galaxy/model/__init__.py
index 545c39e07ea..aa41c8b782d 100644
--- a/lib/galaxy/model/__init__.py
+++ b/lib/galaxy/model/__init__.py
@@ -146,10 +146,12 @@ from galaxy.util.form_builder import (
     WorkflowField,
     WorkflowMappingField,
 )
-from galaxy.util.hash_util import new_insecure_hash
+from galaxy.util.hash_util import (
+    md5_hash_str,
+    new_insecure_hash,
+)
 from galaxy.util.json import safe_loads
 from galaxy.util.sanitize_html import sanitize_html
-from galaxy.web.framework.helpers import md5
 
 log = logging.getLogger(__name__)
 
@@ -8912,7 +8914,7 @@ class Page(Base, HasTags, Dictifiable, RepresentById):
     # email needed for hash generation
     @property
     def email_hash(self):
-        return md5(self.user.email)
+        return md5_hash_str(self.user.email)
 
 
 class PageRevision(Base, Dictifiable, RepresentById):
diff --git a/lib/galaxy/util/hash_util.py b/lib/galaxy/util/hash_util.py
index 53fe69a8aa0..6c5f6a5424e 100644
--- a/lib/galaxy/util/hash_util.py
+++ b/lib/galaxy/util/hash_util.py
@@ -90,6 +90,15 @@ def md5_hash_file(path: Union[str, os.PathLike]) -> Optional[str]:
         return None
 
 
+def md5_hash_str(s):
+    """
+    Return hex encoded md5 hash of string s
+    """
+    m = hashlib.md5()
+    m.update(smart_str(s))
+    return m.hexdigest()
+
+
 def new_secure_hash_v2(text_type: Union[bytes, str]) -> str:
     """More modern version of new_secure_hash.
 
diff --git a/lib/galaxy/web/framework/helpers/__init__.py b/lib/galaxy/web/framework/helpers/__init__.py
index 9c75b88e976..d52b0a56cd5 100644
--- a/lib/galaxy/web/framework/helpers/__init__.py
+++ b/lib/galaxy/web/framework/helpers/__init__.py
@@ -14,11 +14,7 @@ from babel import default_locale
 from babel.dates import format_timedelta
 from routes import url_for
 
-from galaxy.util import (
-    hash_util,
-    smart_str,
-    unicodify,
-)
+from galaxy.util import unicodify
 from galaxy.util.json import safe_dumps as dumps  # noqa: F401
 from .tags import (
     javascript_link,
@@ -102,16 +98,6 @@ def dist_js(*args):
     return js_helper("static/dist/", *args)
 
 
-# Hashes
-def md5(s):
-    """
-    Return hex encoded md5 hash of string s
-    """
-    m = hash_util.md5()
-    m.update(smart_str(s))
-    return m.hexdigest()
-
-
 # Unicode help
 def to_unicode(a_string):
     """
diff --git a/lib/galaxy/webapps/galaxy/api/visualizations.py b/lib/galaxy/webapps/galaxy/api/visualizations.py
index 17ccd3ee490..1208b138a57 100644
--- a/lib/galaxy/webapps/galaxy/api/visualizations.py
+++ b/lib/galaxy/webapps/galaxy/api/visualizations.py
@@ -28,8 +28,8 @@ from galaxy.schema.schema import (
     ShareWithStatus,
     SharingStatus,
 )
+from galaxy.util.hash_util import md5_hash_str
 from galaxy.web import expose_api
-from galaxy.web.framework.helpers import md5
 from galaxy.webapps.base.controller import UsesVisualizationMixin
 from galaxy.webapps.base.webapp import GalaxyWebTransaction
 from galaxy.webapps.galaxy.api import (
@@ -192,7 +192,7 @@ class VisualizationsController(BaseGalaxyAPIController, UsesVisualizationMixin,
             slug=visualization.slug,
         )
         dictionary["username"] = visualization.user.username
-        dictionary["email_hash"] = md5(visualization.user.email)
+        dictionary["email_hash"] = md5_hash_str(visualization.user.email)
         dictionary["tags"] = visualization.make_tag_string_list()
         dictionary["annotation"] = self.get_item_annotation_str(trans.sa_session, trans.user, visualization)
         # need to encode ids in revisions as well

From 2a9c4893c32c7a47a46038758ed2e05b3da51c94 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Fri, 4 Nov 2022 11:01:43 -0400
Subject: [PATCH 08/10] Remove rating from published items

---
 client/package.json                           |  1 -
 client/src/components/Common/Published.vue    | 21 ---------
 client/src/components/Common/Ratings.vue      | 46 -------------------
 .../components/History/HistoryPublished.vue   | 11 +----
 .../components/PageDisplay/PageDisplay.vue    |  9 +---
 .../Visualizations/VisualizationPublished.vue | 11 +----
 .../components/Workflow/WorkflowPublished.vue | 11 +----
 client/yarn.lock                              |  9 +---
 8 files changed, 9 insertions(+), 110 deletions(-)
 delete mode 100644 client/src/components/Common/Ratings.vue

diff --git a/client/package.json b/client/package.json
index bd139e04005..1ea15517c35 100644
--- a/client/package.json
+++ b/client/package.json
@@ -91,7 +91,6 @@
     "vue-prismjs": "^1.2.0",
     "vue-router": "^3.5.3",
     "vue-rx": "^6.2.0",
-    "vue-star-rating": "^1.7.0",
     "vue-virtual-scroll-list": "^2.3.3",
     "vuedraggable": "2.24.3",
     "vuex": "^3.6.2",
diff --git a/client/src/components/Common/Published.vue b/client/src/components/Common/Published.vue
index fbd83e16bfd..e19b8aaa337 100644
--- a/client/src/components/Common/Published.vue
+++ b/client/src/components/Common/Published.vue
@@ -25,13 +25,6 @@
                     <div>
                         <router-link :to="publishedByUser"> Published {{ plural }} by {{ owner }}. </router-link>
                     </div>
-                    <br />
-                    <ratings
-                        v-if="hasRatingData"
-                        :ave-item-rating="aveItemRating"
-                        :user-item-rating="userItemRating"
-                        :num-ratings="numRatings"
-                        @set-rating="$emit('set-rating', $event)" />
                 </div>
                 <LoadingSpan v-else message="Loading item details" />
             </div>
@@ -41,12 +34,10 @@
 
 <script>
 import { StatelessTags } from "components/Tags";
-import Ratings from "components/Common/Ratings";
 import LoadingSpan from "components/LoadingSpan";
 export default {
     components: {
         LoadingSpan,
-        Ratings,
         StatelessTags,
     },
     props: {
@@ -56,15 +47,9 @@ export default {
         },
     },
     computed: {
-        aveItemRating() {
-            return Number(this.item.ave_item_rating);
-        },
         gravatarSource() {
             return `https://secure.gravatar.com/avatar/${this.item.email_hash}?d=identicon`;
         },
-        hasRatingData() {
-            return this.item.num_ratings !== undefined;
-        },
         modelTitle() {
             const modelClass = this.item ? this.item.model_class : "Item";
             if (modelClass == "StoredWorkflow") {
@@ -72,9 +57,6 @@ export default {
             }
             return modelClass;
         },
-        numRatings() {
-            return parseInt(this.item.num_ratings);
-        },
         owner() {
             return this.item.owner || this.item.username || "Unavailable";
         },
@@ -93,9 +75,6 @@ export default {
         urlAll() {
             return `/${this.pluralPath}/list_published`;
         },
-        userItemRating() {
-            return Number(this.item.user_item_rating);
-        },
     },
 };
 </script>
diff --git a/client/src/components/Common/Ratings.vue b/client/src/components/Common/Ratings.vue
deleted file mode 100644
index af04097ef4e..00000000000
--- a/client/src/components/Common/Ratings.vue
+++ /dev/null
@@ -1,46 +0,0 @@
-<template>
-    <div>
-        <h1 class="h-sm">Rating</h1>
-        <div>Community ({{ numRatings }} total, {{ averageRating.toFixed(1) }} average):</div>
-        <star-rating read-only v-model="averageRating" :star-size="25" :show-rating="false" />
-        <div class="mt-2">Yours:</div>
-        <star-rating v-model="userRating" :star-size="25" :show-rating="false" />
-    </div>
-</template>
-<script>
-import StarRating from "vue-star-rating";
-export default {
-    components: {
-        StarRating,
-    },
-    props: {
-        aveItemRating: {
-            type: Number,
-            default: 0,
-        },
-        userItemRating: {
-            type: Number,
-            default: 0,
-        },
-        numRatings: {
-            type: Number,
-            default: 0,
-        },
-    },
-    computed: {
-        averageRating: {
-            get() {
-                return this.aveItemRating;
-            },
-        },
-        userRating: {
-            get() {
-                return this.userItemRating;
-            },
-            set(val) {
-                this.$emit("set-rating", val);
-            },
-        },
-    },
-};
-</script>
diff --git a/client/src/components/History/HistoryPublished.vue b/client/src/components/History/HistoryPublished.vue
index c827912aa7c..fb9904e9ae7 100644
--- a/client/src/components/History/HistoryPublished.vue
+++ b/client/src/components/History/HistoryPublished.vue
@@ -1,5 +1,5 @@
 <template>
-    <Published :item="history" @set-rating="onSetRating">
+    <Published :item="history">
         <template v-slot>
             <HistoryView :id="id" />
         </template>
@@ -29,16 +29,9 @@ export default {
     },
     created() {
         const url = `/api/histories/${this.id}`;
-        const query = this.$route.query;
         urlData({ url }).then((data) => {
-            this.history = { ...data, ...query };
+            this.history = data;
         });
     },
-    methods: {
-        onSetRating(newRating) {
-            const url = `/history/rate_async?id=${this.id}&rating=${newRating}`;
-            urlData({ url });
-        },
-    },
 };
 </script>
diff --git a/client/src/components/PageDisplay/PageDisplay.vue b/client/src/components/PageDisplay/PageDisplay.vue
index 5b6b20337f7..60b9e8134ea 100644
--- a/client/src/components/PageDisplay/PageDisplay.vue
+++ b/client/src/components/PageDisplay/PageDisplay.vue
@@ -1,6 +1,6 @@
 <template>
     <config-provider v-slot="{ config, loading }">
-        <Published :item="page" @set-rating="onSetRating">
+        <Published :item="page">
             <template v-slot>
                 <div v-if="!loading">
                     <markdown
@@ -56,19 +56,14 @@ export default {
         },
     },
     created() {
-        const query = this.$route.query;
         urlData({ url: this.dataUrl }).then((data) => {
-            this.page = { ...data, ...query };
+            this.page = data;
         });
     },
     methods: {
         onEdit() {
             window.location = safePath(this.editUrl);
         },
-        onSetRating(newRating) {
-            const url = `/page/rate_async?id=${this.pageId}&rating=${newRating}`;
-            urlData({ url });
-        },
         stsUrl(config) {
             return `${this.dataUrl}/prepare_download`;
         },
diff --git a/client/src/components/Visualizations/VisualizationPublished.vue b/client/src/components/Visualizations/VisualizationPublished.vue
index b2bd32cf44d..fa9a6590090 100644
--- a/client/src/components/Visualizations/VisualizationPublished.vue
+++ b/client/src/components/Visualizations/VisualizationPublished.vue
@@ -1,5 +1,5 @@
 <template>
-    <Published :item="visualization" @set-rating="onSetRating">
+    <Published :item="visualization">
         <template v-slot>
             <CenterFrame :src="getUrl" />
         </template>
@@ -33,16 +33,9 @@ export default {
     },
     created() {
         const url = `/api/visualizations/${this.id}`;
-        const query = this.$route.query;
         urlData({ url }).then((data) => {
-            this.visualization = { ...data, ...query };
+            this.visualization = data;
         });
     },
-    methods: {
-        onSetRating(newRating) {
-            const url = `/visualization/rate_async?id=${this.id}&rating=${newRating}`;
-            urlData({ url });
-        },
-    },
 };
 </script>
diff --git a/client/src/components/Workflow/WorkflowPublished.vue b/client/src/components/Workflow/WorkflowPublished.vue
index 2340d0a7083..43506cb194f 100644
--- a/client/src/components/Workflow/WorkflowPublished.vue
+++ b/client/src/components/Workflow/WorkflowPublished.vue
@@ -1,5 +1,5 @@
 <template>
-    <Published :item="workflow" @set-rating="onSetRating">
+    <Published :item="workflow">
         <template v-slot>
             <WorkflowDisplay :args="{ workflow_id: id }" :workflow="workflow" :expanded="true" />
         </template>
@@ -29,16 +29,9 @@ export default {
     },
     created() {
         const url = `/api/workflows/${this.id}`;
-        const query = this.$route.query;
         urlData({ url }).then((data) => {
-            this.workflow = { ...data, ...query };
+            this.workflow = data;
         });
     },
-    methods: {
-        onSetRating(newRating) {
-            const url = `/workflow/rate_async?id=${this.id}&rating=${newRating}`;
-            urlData({ url });
-        },
-    },
 };
 </script>
diff --git a/client/yarn.lock b/client/yarn.lock
index 5e543d7f1ab..df24b3709ee 100644
--- a/client/yarn.lock
+++ b/client/yarn.lock
@@ -10003,13 +10003,6 @@ vue-rx@^6.2.0:
   resolved "https://registry.yarnpkg.com/vue-rx/-/vue-rx-6.2.0.tgz#c3b86462b252626edd16417385bb9054a3a23acb"
   integrity sha512-tpKUcqS5IUYS+HsdbR5TlE5LL9PK4zwlplEtmMeydnbqaUTa9ciLMplJXAtFSiQw1vuURoyEJmCXqMxaVEIloQ==
 
-vue-star-rating@^1.7.0:
-  version "1.7.0"
-  resolved "https://registry.yarnpkg.com/vue-star-rating/-/vue-star-rating-1.7.0.tgz#07576aa1a83d54d4e9ea80eebccf058ff3c868b6"
-  integrity sha512-2qdONqnIlvFzJoallfr7VuFRB6hBzYgBVEbXC3yYZz2TfMhx36WtYQ5p7JRjW+pzZx2PGQYZ75UFoBE0jwIANw==
-  dependencies:
-    vue "^2.3.3"
-
 vue-style-loader@^4.1.0:
   version "4.1.3"
   resolved "https://registry.yarnpkg.com/vue-style-loader/-/vue-style-loader-4.1.3.tgz#6d55863a51fa757ab24e89d9371465072aa7bc35"
@@ -10036,7 +10029,7 @@ vue-virtual-scroll-list@^2.3.3:
   resolved "https://registry.yarnpkg.com/vue-virtual-scroll-list/-/vue-virtual-scroll-list-2.3.3.tgz#290f0d04be814b93585f2ef232097b671141e5b1"
   integrity sha512-heuwlZ+lEdcVSp66CpVXnyNdsiHl/XU1cvqJb0yUHGlLv6DGK8OX16FH8I3w/wKNXtHSPF9itdBBGJFrXkyHeg==
 
-vue@2.7.8, vue@^2.3.3, vue@^2.6.10, vue@^2.7.8:
+vue@2.7.8, vue@^2.6.10, vue@^2.7.8:
   version "2.7.8"
   resolved "https://registry.yarnpkg.com/vue/-/vue-2.7.8.tgz#34e06553137611d8cecc4b0cd78b7a59f80b1299"
   integrity sha512-ncwlZx5qOcn754bCu5/tS/IWPhXHopfit79cx+uIlLMyt3vCMGcXai5yCG5y+I6cDmEj4ukRYyZail9FTQh7lQ==

From f646f9a7e135c9ae2a8b308e6229795b9371da9b Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Fri, 4 Nov 2022 11:09:14 -0400
Subject: [PATCH 09/10] Strip item rating helper from controller endpoints

---
 .../webapps/galaxy/controllers/dataset.py     | 30 -------------------
 .../webapps/galaxy/controllers/history.py     | 25 ----------------
 lib/galaxy/webapps/galaxy/controllers/page.py | 28 -----------------
 .../galaxy/controllers/visualization.py       | 28 -----------------
 .../webapps/galaxy/controllers/workflow.py    | 28 -----------------
 5 files changed, 139 deletions(-)

diff --git a/lib/galaxy/webapps/galaxy/controllers/dataset.py b/lib/galaxy/webapps/galaxy/controllers/dataset.py
index 3aad8714803..765305a80ea 100644
--- a/lib/galaxy/webapps/galaxy/controllers/dataset.py
+++ b/lib/galaxy/webapps/galaxy/controllers/dataset.py
@@ -555,23 +555,6 @@ class DatasetInterface(BaseUIController, UsesAnnotations, UsesItemRatings, UsesE
         """Does nothing because datasets do not have an importable/accessible attribute. This method could potentially set another attribute."""
         return
 
-    @web.expose
-    @web.require_login("rate items")
-    @web.json
-    def rate_async(self, trans, id, rating):
-        """Rate a dataset asynchronously and return updated community data."""
-
-        decoded_id = self.decode_id(id)
-        dataset = self.hda_manager.get_accessible(decoded_id, trans.user)
-        dataset = self.hda_manager.error_if_uploading(dataset)
-        if not dataset:
-            return trans.show_error_message("The specified dataset does not exist.")
-
-        # Rate dataset.
-        self.rate_item(trans.sa_session, trans.get_user(), dataset, rating)
-
-        return self.get_ave_item_rating_data(trans.sa_session, dataset)
-
     @web.expose
     def display_by_username_and_slug(self, trans, username, slug, filename=None, preview=True):
         """Display dataset by username and slug; because datasets do not yet have slugs, the slug is the dataset's id."""
@@ -600,24 +583,11 @@ class DatasetInterface(BaseUIController, UsesAnnotations, UsesItemRatings, UsesE
             trans.response.set_content_type(dataset.get_mime())
             return open(dataset.file_name, "rb")
         else:
-            # Get rating data.
-            user_item_rating = 0
-            if trans.get_user():
-                user_item_rating = self.get_user_item_rating(trans.sa_session, trans.get_user(), dataset)
-                if user_item_rating:
-                    user_item_rating = user_item_rating.rating
-                else:
-                    user_item_rating = 0
-            ave_item_rating, num_ratings = self.get_ave_item_rating_data(trans.sa_session, dataset)
-
             return trans.fill_template_mako(
                 "/dataset/display.mako",
                 item=dataset,
                 item_data=dataset_data,
                 truncated=truncated,
-                user_item_rating=user_item_rating,
-                ave_item_rating=ave_item_rating,
-                num_ratings=num_ratings,
                 first_chunk=first_chunk,
             )
 
diff --git a/lib/galaxy/webapps/galaxy/controllers/history.py b/lib/galaxy/webapps/galaxy/controllers/history.py
index 5b0e273aea1..1b2de3d6c55 100644
--- a/lib/galaxy/webapps/galaxy/controllers/history.py
+++ b/lib/galaxy/webapps/galaxy/controllers/history.py
@@ -571,16 +571,6 @@ class HistoryController(BaseUIController, SharableMixin, UsesAnnotations, UsesIt
         # Security check raises error if user cannot access history.
         self.history_manager.error_unless_accessible(history, trans.user, current_history=trans.history)
 
-        # Get rating data.
-        user_item_rating = 0
-        if trans.get_user():
-            user_item_rating = self.get_user_item_rating(trans.sa_session, trans.get_user(), history)
-            if user_item_rating:
-                user_item_rating = user_item_rating.rating
-            else:
-                user_item_rating = 0
-        ave_item_rating, num_ratings = self.get_ave_item_rating_data(trans.sa_session, history)
-
         # Encode history id.
         history_id = trans.security.encode_id(history.id)
 
@@ -590,9 +580,6 @@ class HistoryController(BaseUIController, SharableMixin, UsesAnnotations, UsesIt
                 controller="published",
                 action="history",
                 id=history_id,
-                user_item_rating=user_item_rating,
-                ave_item_rating=ave_item_rating,
-                num_ratings=num_ratings,
             )
         )
 
@@ -741,18 +728,6 @@ class HistoryController(BaseUIController, SharableMixin, UsesAnnotations, UsesIt
         return trans.show_ok_message("Your jobs have been resumed.", refresh_frames=refresh_frames)
         # TODO: used in index.mako
 
-    @web.expose
-    @web.require_login("rate items")
-    @web.json
-    def rate_async(self, trans, id, rating):
-        """Rate a history asynchronously and return updated community data."""
-        history = self.history_manager.get_accessible(self.decode_id(id), trans.user, current_history=trans.history)
-        if not history:
-            return trans.show_error_message("The specified history does not exist.")
-        # Rate history.
-        self.rate_item(trans.sa_session, trans.get_user(), history, rating)
-        return self.get_ave_item_rating_data(trans.sa_session, history)
-
     @web.expose
     @web.json
     @web.require_login("get history name and link")
diff --git a/lib/galaxy/webapps/galaxy/controllers/page.py b/lib/galaxy/webapps/galaxy/controllers/page.py
index 30970043dba..180d782dea9 100644
--- a/lib/galaxy/webapps/galaxy/controllers/page.py
+++ b/lib/galaxy/webapps/galaxy/controllers/page.py
@@ -549,25 +549,12 @@ class PageController(BaseUIController, SharableMixin, UsesStoredWorkflowMixin, U
         # Encode page identifier.
         page_id = trans.security.encode_id(page.id)
 
-        # Get rating data.
-        user_item_rating = 0
-        if trans.get_user():
-            user_item_rating = self.get_user_item_rating(trans.sa_session, trans.get_user(), page)
-            if user_item_rating:
-                user_item_rating = user_item_rating.rating
-            else:
-                user_item_rating = 0
-        ave_item_rating, num_ratings = self.get_ave_item_rating_data(trans.sa_session, page)
-
         # Redirect to client.
         return trans.response.send_redirect(
             web.url_for(
                 controller="published",
                 action="page",
                 id=page_id,
-                user_item_rating=user_item_rating,
-                ave_item_rating=ave_item_rating,
-                num_ratings=num_ratings,
             )
         )
 
@@ -587,21 +574,6 @@ class PageController(BaseUIController, SharableMixin, UsesStoredWorkflowMixin, U
             trans.sa_session.flush()
         return
 
-    @web.expose
-    @web.require_login("rate items")
-    @web.json
-    def rate_async(self, trans, id, rating):
-        """Rate a page asynchronously and return updated community data."""
-
-        page = self.get_page(trans, id, check_ownership=False, check_accessible=True)
-        if not page:
-            return trans.show_error_message("The specified page does not exist.")
-
-        # Rate page.
-        self.rate_item(trans.sa_session, trans.get_user(), page, rating)
-
-        return self.get_ave_item_rating_data(trans.sa_session, page)
-
     @web.expose
     def get_embed_html_async(self, trans, id):
         """Returns HTML for embedding a workflow in a page."""
diff --git a/lib/galaxy/webapps/galaxy/controllers/visualization.py b/lib/galaxy/webapps/galaxy/controllers/visualization.py
index fa6534a7939..e56b249d8a6 100644
--- a/lib/galaxy/webapps/galaxy/controllers/visualization.py
+++ b/lib/galaxy/webapps/galaxy/controllers/visualization.py
@@ -435,21 +435,6 @@ class VisualizationController(
 
         return
 
-    @web.expose
-    @web.require_login("rate items")
-    @web.json
-    def rate_async(self, trans, id, rating):
-        """Rate a visualization asynchronously and return updated community data."""
-
-        visualization = self.get_visualization(trans, id, check_ownership=False, check_accessible=True)
-        if not visualization:
-            return trans.show_error_message("The specified visualization does not exist.")
-
-        # Rate visualization.
-        self.rate_item(trans.sa_session, trans.get_user(), visualization, rating)
-
-        return self.get_ave_item_rating_data(trans.sa_session, visualization)
-
     @web.expose
     @web.require_login("share Galaxy visualizations")
     def imp(self, trans, id):
@@ -511,25 +496,12 @@ class VisualizationController(
         # Encode page identifier.
         visualization_id = trans.security.encode_id(visualization.id)
 
-        # Get rating data.
-        user_item_rating = 0
-        if trans.get_user():
-            user_item_rating = self.get_user_item_rating(trans.sa_session, trans.get_user(), visualization)
-            if user_item_rating:
-                user_item_rating = user_item_rating.rating
-            else:
-                user_item_rating = 0
-        ave_item_rating, num_ratings = self.get_ave_item_rating_data(trans.sa_session, visualization)
-
         # Redirect to client.
         return trans.response.send_redirect(
             web.url_for(
                 controller="published",
                 action="visualization",
                 id=visualization_id,
-                user_item_rating=user_item_rating,
-                ave_item_rating=ave_item_rating,
-                num_ratings=num_ratings,
             )
         )
 
diff --git a/lib/galaxy/webapps/galaxy/controllers/workflow.py b/lib/galaxy/webapps/galaxy/controllers/workflow.py
index a3cf20c3e39..63a8d666985 100644
--- a/lib/galaxy/webapps/galaxy/controllers/workflow.py
+++ b/lib/galaxy/webapps/galaxy/controllers/workflow.py
@@ -268,16 +268,6 @@ class WorkflowController(BaseUIController, SharableMixin, UsesStoredWorkflowMixi
         for step in stored_workflow.latest_workflow.steps:
             step.annotation = self.get_item_annotation_str(trans.sa_session, stored_workflow.user, step)
 
-        # Get rating data.
-        user_item_rating = 0
-        if trans.get_user():
-            user_item_rating = self.get_user_item_rating(trans.sa_session, trans.get_user(), stored_workflow)
-            if user_item_rating:
-                user_item_rating = user_item_rating.rating
-            else:
-                user_item_rating = 0
-        ave_item_rating, num_ratings = self.get_ave_item_rating_data(trans.sa_session, stored_workflow)
-
         # Encode page identifier.
         workflow_id = trans.security.encode_id(stored_workflow.id)
 
@@ -287,9 +277,6 @@ class WorkflowController(BaseUIController, SharableMixin, UsesStoredWorkflowMixi
                 controller="published",
                 action="workflow",
                 id=workflow_id,
-                user_item_rating=user_item_rating,
-                ave_item_rating=ave_item_rating,
-                num_ratings=num_ratings,
             )
         )
 
@@ -346,21 +333,6 @@ class WorkflowController(BaseUIController, SharableMixin, UsesStoredWorkflowMixi
             trans.sa_session.flush()
             return new_annotation
 
-    @web.expose
-    @web.require_login("rate items")
-    @web.json
-    def rate_async(self, trans, id, rating):
-        """Rate a workflow asynchronously and return updated community data."""
-
-        stored = self.get_stored_workflow(trans, id, check_ownership=False, check_accessible=True)
-        if not stored:
-            return trans.show_error_message("The specified workflow does not exist.")
-
-        # Rate workflow.
-        self.rate_item(trans.sa_session, trans.get_user(), stored, rating)
-
-        return self.get_ave_item_rating_data(trans.sa_session, stored)
-
     @web.expose
     def get_embed_html_async(self, trans, id):
         """Returns HTML for embedding a workflow in a page."""

From 2659a9ec30b380794c5736d176093dfbedce9577 Mon Sep 17 00:00:00 2001
From: guerler <aysam.guerler@gmail.com>
Date: Fri, 4 Nov 2022 11:16:41 -0400
Subject: [PATCH 10/10] Remove item rating from legacy mako

---
 templates/display_base.mako | 75 +------------------------------------
 1 file changed, 1 insertion(+), 74 deletions(-)

diff --git a/templates/display_base.mako b/templates/display_base.mako
index 597d49abcf6..e0a6d5e2403 100644
--- a/templates/display_base.mako
+++ b/templates/display_base.mako
@@ -138,80 +138,7 @@
                 <p>
                     <a href="${href_to_all_items}">All published ${item_plural.lower()}</a><br>
                     <a href="${href_to_user_items}">Published ${item_plural.lower()} by ${item.user.username | h}</a>
-                ## Rating.
-                <h4>Rating</h4>
-                <%
-                    label = "ratings"
-                    if num_ratings == 1:
-                        label = "rating"
-                %>
-                <div style="padding-bottom: 0.75em; float: left">
-                    Community<br>
-                    <span style="font-size:80%">
-                        (<span id="num_ratings">${num_ratings}</span> ${label},
-                         <span id="ave_rating">${"%.1f" % ave_item_rating}</span> average)
-                    <span>
-                </div>
-                <div style="float: right">
-                    <input name="star1" type="radio" class="community_rating_star star" disabled="disabled" value="1"
-                    %if ave_item_rating > 0 and ave_item_rating <= 1.5:
-                        checked="checked"
-                    %endif
-                    />
-                    <input name="star1" type="radio" class="community_rating_star star" disabled="disabled" value="2"
-                    %if ave_item_rating > 1.5 and ave_item_rating <= 2.5:
-                        checked="checked"
-                    %endif
-                    />
-                    <input name="star1" type="radio" class="community_rating_star star" disabled="disabled" value="3"
-                    %if ave_item_rating > 2.5 and ave_item_rating <= 3.5:
-                        checked="checked"
-                    %endif
-                    />
-                    <input name="star1" type="radio" class="community_rating_star star" disabled="disabled" value="4"
-                    %if ave_item_rating > 3.5 and ave_item_rating <= 4.5:
-                        checked="checked"
-                    %endif
-                    />
-                    <input name="star1" type="radio" class="community_rating_star star" disabled="disabled" value="5"
-                    %if ave_item_rating > 4.5:
-                        checked="checked"
-                    %endif
-                    />
-                </div>
-                <div style="clear: both;"></div>
-                %if trans.get_user():
-                    <div style="float: left">
-                        Yours<br><span id="rating_feedback" style="font-size:80%; display: none">(thanks!)</span>
-                    </div>
-                    <div style="float: right">
-                        <input name="star2" type="radio" class="user_rating_star" value="1"
-                        %if user_item_rating == 1:
-                            checked="checked"
-                        %endif
-                        />
-                        <input name="star2" type="radio" class="user_rating_star" value="2"
-                        %if user_item_rating == 2:
-                            checked="checked"
-                        %endif
-                        />
-                        <input name="star2" type="radio" class="user_rating_star" value="3"
-                        %if user_item_rating == 3:
-                            checked="checked"
-                        %endif
-                        />
-                        <input name="star2" type="radio" class="user_rating_star" value="4"
-                        %if user_item_rating == 4:
-                            checked="checked"
-                        %endif
-                        />
-                        <input name="star2" type="radio" class="user_rating_star" value="5"
-                        %if user_item_rating == 5:
-                            checked="checked"
-                        %endif
-                        />
-                    </div>
-                %endif
+                </p>
                 <div style="clear: both;"></div>
                 ## Tags.
                 <h4>Tags</h4>