From 5a516284261c5acd1e1b352730cb2148c061c414 Mon Sep 17 00:00:00 2001
From: Raphael Odini <raphodn@users.noreply.github.com>
Date: Thu, 5 Sep 2024 16:21:09 +0200
Subject: [PATCH] refactor: homogenise getShareLinkUrl. Add missing data-name

---
 src/components/LocationActionMenuButton.vue | 7 ++++++-
 src/components/LocationCard.vue             | 1 +
 src/components/PriceActionMenuButton.vue    | 5 ++++-
 src/components/ProductActionMenuButton.vue  | 7 ++++++-
 src/components/ProductCard.vue              | 2 +-
 src/components/ProofCard.vue                | 2 +-
 src/components/UserActionMenuButton.vue     | 7 ++++++-
 src/components/UserCard.vue                 | 1 +
 src/views/UserDashboardPriceList.vue        | 5 ++++-
 9 files changed, 30 insertions(+), 7 deletions(-)

diff --git a/src/components/LocationActionMenuButton.vue b/src/components/LocationActionMenuButton.vue
index 4f8fb01d2b6..9acddaa4f5a 100644
--- a/src/components/LocationActionMenuButton.vue
+++ b/src/components/LocationActionMenuButton.vue
@@ -3,7 +3,7 @@
     <v-icon>mdi-dots-vertical</v-icon>
     <v-menu activator="parent" scroll-strategy="close" transition="slide-y-transition">
       <v-list>
-        <ShareLink :overrideUrl="'/locations/' + location.id" display="list-item" />
+        <ShareLink :overrideUrl="getShareLinkUrl" display="list-item" />
         <v-divider />
         <OpenStreetMapLink :location="location" display="list-item" />
       </v-list>
@@ -28,6 +28,11 @@ export default {
       type: String,
       default: 'position:absolute;bottom:6px;right:0;'
     }
+  },
+  computed: {
+    getShareLinkUrl() {
+      return `/locations/${this.location.id}`
+    }
   }
 }
 </script>
diff --git a/src/components/LocationCard.vue b/src/components/LocationCard.vue
index d89b3493d45..845ceb41125 100644
--- a/src/components/LocationCard.vue
+++ b/src/components/LocationCard.vue
@@ -3,6 +3,7 @@
     :title="getLocationTitle(location)"
     :subtitle="location ? location.osm_display_name : ''"
     :prepend-icon="location ? 'mdi-map-marker-outline' : 'mdi-map-marker-remove-variant'"
+    data-name="location-card"
     @click="goToLocation(location)"
   >
     <v-card-text v-if="location">
diff --git a/src/components/PriceActionMenuButton.vue b/src/components/PriceActionMenuButton.vue
index ec2748957d7..aa880a6acbd 100644
--- a/src/components/PriceActionMenuButton.vue
+++ b/src/components/PriceActionMenuButton.vue
@@ -18,7 +18,7 @@
             {{ $t('Common.Price') }}
           </v-list-subheader>
           <v-divider />
-          <ShareLink v-if="showPriceShare" :overrideUrl="getPriceDetailUrl" display="list-item" />
+          <ShareLink v-if="showPriceShare" :overrideUrl="getShareLinkUrl" display="list-item" />
           <v-list-item :slim="true" prepend-icon="mdi-eye-outline" :to="getPriceDetailUrl">
             {{ $t('Common.Details') }}
           </v-list-item>
@@ -115,6 +115,9 @@ export default {
     showPriceShare() {
       return this.$route.path === this.getPriceDetailUrl
     },
+    getShareLinkUrl() {
+      return this.getPriceDetailUrl
+    },
     userIsPriceOwner() {
       return this.username && (this.price.owner === this.username)
     }
diff --git a/src/components/ProductActionMenuButton.vue b/src/components/ProductActionMenuButton.vue
index a1170fb8f91..43ed9d29a31 100644
--- a/src/components/ProductActionMenuButton.vue
+++ b/src/components/ProductActionMenuButton.vue
@@ -4,7 +4,7 @@
     <v-menu activator="parent" scroll-strategy="close" transition="slide-y-transition">
       <v-list>
         <PriceAddLink :productCode="product.code" display="list-item" />
-        <ShareLink :overrideUrl="'/products/' + product.code" display="list-item" />
+        <ShareLink :overrideUrl="getShareLinkUrl" display="list-item" />
         <v-divider />
         <OpenFoodFactsLink :source="product.source" facet="product" :value="product.code" display="list-item" />
       </v-list>
@@ -30,6 +30,11 @@ export default {
       type: String,
       default: 'position:absolute;bottom:6px;right:0;'
     }
+  },
+  computed: {
+    getShareLinkUrl() {
+      return `/products/${this.product.code}`
+    }
   }
 }
 </script>
diff --git a/src/components/ProductCard.vue b/src/components/ProductCard.vue
index 0ab069400f0..97233b046c7 100644
--- a/src/components/ProductCard.vue
+++ b/src/components/ProductCard.vue
@@ -1,5 +1,5 @@
 <template>
-  <v-card data-name="product-card">
+  <v-card :id="'product_' + product.code" data-name="product-card">
     <v-container class="pa-2" :style="latestPrice ? 'position:relative;' : ''">
       <v-row v-if="product">
         <v-col class="pr-0" style="max-width:20%">
diff --git a/src/components/ProofCard.vue b/src/components/ProofCard.vue
index 1fca20b4a84..2198b567127 100644
--- a/src/components/ProofCard.vue
+++ b/src/components/ProofCard.vue
@@ -1,5 +1,5 @@
 <template>
-  <v-card :id="'proof_' + proof.id" @click="selectProof">
+  <v-card :id="'proof_' + proof.id" data-name="proof-card" @click="selectProof">
     <v-card-title v-if="!hideProofHeader">
       {{ $t('Common.Proof') }} <v-btn style="float:right;" variant="text" density="compact" icon="mdi-close" @click="close" />
     </v-card-title>
diff --git a/src/components/UserActionMenuButton.vue b/src/components/UserActionMenuButton.vue
index b6cd223c2ec..0270251c6f4 100644
--- a/src/components/UserActionMenuButton.vue
+++ b/src/components/UserActionMenuButton.vue
@@ -3,7 +3,7 @@
     <v-icon>mdi-dots-vertical</v-icon>
     <v-menu activator="parent" scroll-strategy="close" transition="slide-y-transition">
       <v-list>
-        <ShareLink :overrideUrl="'/users/' + user.user_id" display="list-item" />
+        <ShareLink :overrideUrl="getShareLinkUrl" display="list-item" />
         <v-divider />
         <OpenFoodFactsLink facet="editor" :value="user.user_id" display="list-item" />
       </v-list>
@@ -28,6 +28,11 @@ export default {
       type: String,
       default: 'position:absolute;bottom:6px;right:0;'
     }
+  },
+  computed: {
+    getShareLinkUrl() {
+      return `/users/${this.user.user_id}`
+    }
   }
 }
 </script>
diff --git a/src/components/UserCard.vue b/src/components/UserCard.vue
index 4039280eee3..16bd4a7f5a7 100644
--- a/src/components/UserCard.vue
+++ b/src/components/UserCard.vue
@@ -2,6 +2,7 @@
   <v-card
     :title="user.user_id"
     prepend-icon="mdi-account"
+    data-name="user-card"
     @click="goToUser(user)"
   >
     <v-card-text>
diff --git a/src/views/UserDashboardPriceList.vue b/src/views/UserDashboardPriceList.vue
index b23a73df9cd..6eef68f5100 100644
--- a/src/views/UserDashboardPriceList.vue
+++ b/src/views/UserDashboardPriceList.vue
@@ -11,7 +11,7 @@
       <v-btn size="small" prepend-icon="mdi-arrow-left" to="/dashboard">
         {{ $t('UserDashboard.Title') }}
       </v-btn>
-      <ShareLink :overrideUrl="'/users/' + username" display="button" />
+      <ShareLink :overrideUrl="getShareLinkUrl" display="button" />
     </v-col>
   </v-row>
 
@@ -65,6 +65,9 @@ export default {
     username() {
       return this.appStore.user.username
     },
+    getShareLinkUrl() {
+      return `/users/${this.username}`
+    }
   },
   mounted() {
     this.getUserPrices()