Move edit/delete share hints into interpolate-translation functions, …

…fix mapGetters
owncloud · Apr 28, 2021 · 1b3d4b9 · 1b3d4b9
1 parent 4e9e13d
commit 1b3d4b9
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/packages/web-app-files/src/components/Collaborators/Collaborator.vue b/packages/web-app-files/src/components/Collaborators/Collaborator.vue
@@ -141,8 +141,8 @@
         <div class="uk-flex uk-flex-nowrap uk-flex-middle">
           <oc-button
             v-if="$_editButtonVisible"
-            :aria-label="$gettext(`Edit share with ${ collaborator.collaborator.displayName }`)"
-            :uk-tooltip="$gettext(`Edit share with ${ collaborator.collaborator.displayName }`)"
+            :aria-label="editShareHint"
+            :uk-tooltip="editShareHint"
             appearance="raw"
             class="files-collaborators-collaborator-edit"
             @click="$emit('onEdit', collaborator)"
@@ -152,8 +152,8 @@
           <div>
             <oc-button
               v-if="$_deleteButtonVisible"
-              :aria-label="$gettext(`Delete share with ${ collaborator.collaborator.displayName }`)"
-              :uk-tooltip="$gettext(`Delete share with ${ collaborator.collaborator.displayName }`)"
+              :aria-label="deleteShareHint"
+              :uk-tooltip="deleteShareHint"
               appearance="raw"
               class="files-collaborators-collaborator-delete"
               @click="$_removeShare"
@@ -223,6 +223,16 @@ export default {
       return this.modifiable && !this.removalInProgress
     },
 
+    editShareHint() {
+      const translated = this.$gettext('Edit share with ${ currentCollaborator }')
+      return this.$gettextInterpolate(translated, { currentCollaborator: this.collaborator.collaborator.displayName }, true)
+    },
+
+    deleteShareHint() {
+      const translated = this.$gettext('Delete share with ${ currentCollaborator }')
+      return this.$gettextInterpolate(translated, { currentCollaborator: this.collaborator.collaborator.displayName }, true)
+    },
+
     isIndirectShare() {
       // it is assumed that the "incoming" attribute only exists
       // on shares coming from this.collaborator.sTree which are all indirect

diff --git a/packages/web-app-files/src/components/PublicLinksSidebar/PrivateLinkItem.vue b/packages/web-app-files/src/components/PublicLinksSidebar/PrivateLinkItem.vue
@@ -37,7 +37,7 @@ export default {
   }),
 
   computed: {
-    ...mapGetters('Files', ['highlightedFile'], ['capabilities']),
+    ...mapGetters('Files', ['highlightedFile', 'capabilities']),
 
     copyLabel() {
       return this.$gettext('Copy private link url')