From 74f3ea2108bba187c03e78299d13fca2aa0927c5 Mon Sep 17 00:00:00 2001
From: Martin Artnik <martin@scinote.net>
Date: Wed, 2 Oct 2024 14:22:51 +0200
Subject: [PATCH] Add file editing & versioning to inventories [SCI-11043]

---
 .../content/attachments/open_locally_menu.vue | 34 +++++++++++++++----
 .../shared/file_preview/_content.html.erb     |  4 +--
 config/locales/en.yml                         |  2 +-
 3 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/app/javascript/vue/shared/content/attachments/open_locally_menu.vue b/app/javascript/vue/shared/content/attachments/open_locally_menu.vue
index 8ea7048cc..9d40e11cf 100644
--- a/app/javascript/vue/shared/content/attachments/open_locally_menu.vue
+++ b/app/javascript/vue/shared/content/attachments/open_locally_menu.vue
@@ -1,8 +1,8 @@
 <template>
-  <div class="sn-open-locally-menu" @mouseenter="fetchLocalAppInfo">
-    <div v-if="(!canOpenLocally || disableLocalOpen) && (attachment.attributes.wopi && attachment.attributes.urls.edit_asset)">
+  <div class="sn-open-locally-menu flex" @mouseenter="fetchLocalAppInfo">
+    <div v-if="(!canOpenLocally) && (attachment.attributes.wopi && attachment.attributes.urls.edit_asset)">
       <a :href="editWopiSupported ? attachment.attributes.urls.edit_asset : null" target="_blank"
-         class="block whitespace-nowrap rounded px-3 py-2.5 hover:!text-sn-blue hover:no-underline cursor-pointer hover:bg-sn-super-light-grey"
+         class="btn btn-light"
          :class="{ 'disabled': !editWopiSupported }"
          :title="editWopiSupported ? null : attachment.attributes.wopi_context.title"
          style="pointer-events: all"
@@ -26,6 +26,7 @@
           {{ menu[0].text }}
         </a>
     </div>
+    <a @click="fileVersionsModal = true" class="btn btn-light"><i class="sn-icon sn-icon-history-search"></i>{{ i18n.t('assets.context_menu.versions') }}</a>
 
     <Teleport to="body">
       <NoPredefinedAppModal
@@ -47,6 +48,13 @@
         v-if="showUpdateVersionModal"
         @close="showUpdateVersionModal = false"
       />
+       <FileVersionsModal
+        v-if="fileVersionsModal"
+        :versionsUrl="attachment.attributes.urls.versions"
+        :restoreVersionUrl="attachment.attributes.urls.restore_version"
+        @close="fileVersionsModal = false"
+        @fileVersionRestored="refreshPreview"
+      />
     </Teleport>
   </div>
 </template>
@@ -55,14 +63,19 @@
 import OpenLocallyMixin from './mixins/open_locally.js';
 import MenuDropdown from '../../menu_dropdown.vue';
 import UpdateVersionModal from '../modal/update_version_modal.vue';
+import FileVersionsModal from '../../file_versions_modal.vue';
 
 export default {
   name: 'OpenLocallyMenu',
   mixins: [OpenLocallyMixin],
-  components: { MenuDropdown, UpdateVersionModal },
+  components: { MenuDropdown, UpdateVersionModal, FileVersionsModal },
   props: {
-    attachment: { type: Object, required: true },
-    disableLocalOpen: { type: Boolean, default: false }
+    attachment: { type: Object, required: true }
+  },
+  data() {
+    return {
+      fileVersionsModal: false
+    };
   },
   created() {
     this.fetchLocalAppInfo();
@@ -92,7 +105,7 @@ export default {
         });
       }
 
-      if (this.canOpenLocally && !this.disableLocalOpen) {
+      if (this.canOpenLocally) {
         const text = this.localAppName
           ? this.i18n.t('attachments.open_locally_in', { application: this.localAppName })
           : this.i18n.t('attachments.open_locally');
@@ -116,6 +129,13 @@ export default {
   methods: {
     openImageEditor() {
       document.getElementById('editImageButton').click();
+    },
+    refreshPreview() {
+      const imageElement = document.querySelector('.file-preview-container .asset-image');
+
+      if (!imageElement) return;
+
+      window.ActiveStoragePreviews.reCheckPreview({ target: imageElement });
     }
   }
 };
diff --git a/app/views/shared/file_preview/_content.html.erb b/app/views/shared/file_preview/_content.html.erb
index b99944c80..00d259dee 100644
--- a/app/views/shared/file_preview/_content.html.erb
+++ b/app/views/shared/file_preview/_content.html.erb
@@ -4,7 +4,7 @@
     <% if can_edit && !preview %>
       <% if wopi_enabled? && wopi_file?(asset) %>
         <div id="openLocallyMenu" data-behaviour="vue">
-          <open-locally-menu :disable-local-open="<%= asset.repository_asset_value.present? %>" :attachment="<%= { attributes: AssetSerializer.new(asset, scope: { user: current_user }).as_json }.to_json %>" />
+          <open-locally-menu :attachment="<%= { attributes: AssetSerializer.new(asset, scope: { user: current_user }).as_json }.to_json %>" />
         </div>
       <% elsif asset.file.metadata[:asset_type] == 'marvinjs' %>
         <button class="btn btn-light marvinjs-edit-button"
@@ -40,7 +40,7 @@
         </button>
       <% end %>
       <div id="openLocallyMenu" data-behaviour="vue">
-        <open-locally-menu :disable-local-open="<%= asset.repository_asset_value.present? %>" :attachment="<%= { attributes: AssetSerializer.new(asset, scope: { user: current_user }).as_json }.to_json %>" />
+        <open-locally-menu :attachment="<%= { attributes: AssetSerializer.new(asset, scope: { user: current_user }).as_json }.to_json %>" />
       </div>
     <% end %>
     <a class="btn btn-light file-download-link" href="<%= rails_blob_path(asset.file, disposition: 'attachment') %>" data-turbolinks="false">
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 570263530..1d5fc4813 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -4013,7 +4013,7 @@ en:
       inline_html: "Large"
       thumbnail_html: "Thumbnail"
       list_html: "List"
-      versions: "Versions"
+      versions: "Version history"
     file_versions_modal:
       title: "Version history"
       download: "Download"