From f4be48163e15eee171e9b31749887acddb727e62 Mon Sep 17 00:00:00 2001
From: Dominik Schmidt <dev@dominik-schmidt.de>
Date: Wed, 16 Feb 2022 17:57:42 +0100
Subject: [PATCH 1/5] Add changelog

---
 changelog/unreleased/enhancement-search-context-menu | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 changelog/unreleased/enhancement-search-context-menu

diff --git a/changelog/unreleased/enhancement-search-context-menu b/changelog/unreleased/enhancement-search-context-menu
new file mode 100644
index 00000000000..0965175ba9f
--- /dev/null
+++ b/changelog/unreleased/enhancement-search-context-menu
@@ -0,0 +1,5 @@
+Enhancement: Enable context menu for search results
+
+We've enabled a rudimentary context menu for search results.
+
+https://github.com/owncloud/web/pull/6445

From 5fffd7acaa51281c02c29ccb7f01700e682ccf70 Mon Sep 17 00:00:00 2001
From: Dominik Schmidt <dev@dominik-schmidt.de>
Date: Wed, 16 Feb 2022 15:31:11 +0100
Subject: [PATCH 2/5] Enable context menu for search results

---
 packages/web-app-files/src/components/Search/List.vue | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/packages/web-app-files/src/components/Search/List.vue b/packages/web-app-files/src/components/Search/List.vue
index d2b6e567638..a6d73f5e0c5 100644
--- a/packages/web-app-files/src/components/Search/List.vue
+++ b/packages/web-app-files/src/components/Search/List.vue
@@ -16,11 +16,14 @@
       :target-route="resourceTargetLocation"
       :are-paths-displayed="true"
       :are-thumbnails-displayed="displayThumbnails"
-      :has-actions="false"
+      :has-actions="true"
       :is-selectable="false"
       @fileClick="$_fileActions_triggerDefaultAction"
       @rowMounted="rowMounted"
     >
+      <template #contextMenu="{ resource }">
+        <context-actions :items="[resource]" />
+      </template>
       <template #footer>
         <pagination :pages="paginationPages" :current-page="paginationPage" />
         <list-info
@@ -43,6 +46,7 @@ import { ImageType, ImageDimension } from '../../constants'
 import { createLocationSpaces } from '../../router'
 import NoContentMessage from '../FilesList/NoContentMessage.vue'
 import ResourceTable from '../FilesList/ResourceTable.vue'
+import ContextActions from '../FilesList/ContextActions.vue'
 import debounce from 'lodash-es/debounce'
 import { mapMutations, mapGetters, mapActions } from 'vuex'
 import { computed } from '@vue/composition-api'
@@ -55,7 +59,7 @@ import MixinFilesListScrolling from '../../mixins/filesListScrolling'
 const visibilityObserver = new VisibilityObserver()
 
 export default {
-  components: { ListInfo, Pagination, NoContentMessage, ResourceTable },
+  components: { ContextActions, ListInfo, Pagination, NoContentMessage, ResourceTable },
   mixins: [MixinFileActions, MixinFilesListFilter, MixinFilesListScrolling],
   props: {
     searchResults: {

From 09f154b766cc7d039099817f84330b50a1531b97 Mon Sep 17 00:00:00 2001
From: Dominik Schmidt <dev@dominik-schmidt.de>
Date: Wed, 16 Feb 2022 17:47:30 +0100
Subject: [PATCH 3/5] Enable or disable several file actions for usage outside
 of the files app

---
 .../AppBar/SelectedResources/BatchActions.vue |  6 ++++++
 .../components/FilesList/ContextActions.vue   |  6 ++++++
 .../src/composables/useIsFilesAppActive.ts    | 17 ++++++++++++++++
 .../src/mixins/actions/downloadArchive.js     |  1 +
 .../src/mixins/actions/downloadFile.js        |  1 +
 .../src/mixins/actions/favorite.js            |  1 +
 .../src/mixins/actions/navigate.js            | 10 ++++------
 .../src/mixins/actions/showActions.js         |  5 +++++
 .../src/mixins/actions/showDetails.js         |  5 +++++
 .../src/mixins/actions/showShares.js          |  5 +++++
 .../tests/unit/components/Search/List.spec.js | 20 +++++++++++++++++--
 11 files changed, 69 insertions(+), 8 deletions(-)
 create mode 100644 packages/web-app-files/src/composables/useIsFilesAppActive.ts

diff --git a/packages/web-app-files/src/components/AppBar/SelectedResources/BatchActions.vue b/packages/web-app-files/src/components/AppBar/SelectedResources/BatchActions.vue
index 72fd9adcf27..a8fedcb5c2a 100644
--- a/packages/web-app-files/src/components/AppBar/SelectedResources/BatchActions.vue
+++ b/packages/web-app-files/src/components/AppBar/SelectedResources/BatchActions.vue
@@ -23,8 +23,14 @@ import DownloadFile from '../../../mixins/actions/downloadFile'
 import EmptyTrashBin from '../../../mixins/actions/emptyTrashBin'
 import Move from '../../../mixins/actions/move'
 import Restore from '../../../mixins/actions/restore'
+import { useIsFilesAppActive } from '../../../composables/useIsFilesAppActive'
 
 export default {
+  setup() {
+    return {
+      isFilesAppActive: useIsFilesAppActive()
+    }
+  },
   name: 'BatchActions',
   components: { ActionMenuItem },
   mixins: [
diff --git a/packages/web-app-files/src/components/FilesList/ContextActions.vue b/packages/web-app-files/src/components/FilesList/ContextActions.vue
index 6661f71889f..ed60b7d2682 100644
--- a/packages/web-app-files/src/components/FilesList/ContextActions.vue
+++ b/packages/web-app-files/src/components/FilesList/ContextActions.vue
@@ -39,8 +39,14 @@ import Restore from '../../mixins/actions/restore'
 import ShowActions from '../../mixins/actions/showActions'
 import ShowDetails from '../../mixins/actions/showDetails'
 import ShowShares from '../../mixins/actions/showShares'
+import { useIsFilesAppActive } from '../../composables/useIsFilesAppActive'
 
 export default {
+  setup() {
+    return {
+      isFilesAppActive: useIsFilesAppActive()
+    }
+  },
   name: 'ContextActions',
   components: { ActionMenuItem },
   mixins: [
diff --git a/packages/web-app-files/src/composables/useIsFilesAppActive.ts b/packages/web-app-files/src/composables/useIsFilesAppActive.ts
new file mode 100644
index 00000000000..bef36d31308
--- /dev/null
+++ b/packages/web-app-files/src/composables/useIsFilesAppActive.ts
@@ -0,0 +1,17 @@
+import { computed, ComputedRef, unref } from '@vue/composition-api'
+import { useActiveApp } from 'web-pkg/src/composables'
+import FilesApp from '../index'
+
+/*
+ * This composable can be used to check whether the current route
+ * belongs to the file app. It's used to determine whether certain
+ * file actions should be enabled in other apps too, for instance
+ * in the search app. This way we avoid having cross references to
+ * other apps in the files app.
+ */
+export const useIsFilesAppActive = (): ComputedRef<boolean> => {
+  const activeApp = useActiveApp()
+  return computed(() => {
+    return unref(activeApp) === FilesApp.appInfo.id
+  })
+}
diff --git a/packages/web-app-files/src/mixins/actions/downloadArchive.js b/packages/web-app-files/src/mixins/actions/downloadArchive.js
index aaf1e80f767..cd57d8723ec 100644
--- a/packages/web-app-files/src/mixins/actions/downloadArchive.js
+++ b/packages/web-app-files/src/mixins/actions/downloadArchive.js
@@ -21,6 +21,7 @@ export default {
           },
           isEnabled: ({ resources }) => {
             if (
+              this.isFilesAppActive &&
               !isLocationSpacesActive(this.$router, 'files-spaces-personal-home') &&
               !isLocationSpacesActive(this.$router, 'files-spaces-project') &&
               !isLocationPublicActive(this.$router, 'files-public-files') &&
diff --git a/packages/web-app-files/src/mixins/actions/downloadFile.js b/packages/web-app-files/src/mixins/actions/downloadFile.js
index a9d72d459ee..cfcb47f8597 100644
--- a/packages/web-app-files/src/mixins/actions/downloadFile.js
+++ b/packages/web-app-files/src/mixins/actions/downloadFile.js
@@ -17,6 +17,7 @@ export default {
           },
           isEnabled: ({ resources }) => {
             if (
+              this.isFilesAppActive &&
               !isLocationSpacesActive(this.$router, 'files-spaces-personal-home') &&
               !isLocationSpacesActive(this.$router, 'files-spaces-project') &&
               !isLocationPublicActive(this.$router, 'files-public-files') &&
diff --git a/packages/web-app-files/src/mixins/actions/favorite.js b/packages/web-app-files/src/mixins/actions/favorite.js
index 839c9fcf7bd..85a59edc345 100644
--- a/packages/web-app-files/src/mixins/actions/favorite.js
+++ b/packages/web-app-files/src/mixins/actions/favorite.js
@@ -18,6 +18,7 @@ export default {
           },
           isEnabled: ({ resources }) => {
             if (
+              this.isFilesAppActive &&
               !isLocationSpacesActive(this.$router, 'files-spaces-personal-home') &&
               !isLocationCommonActive(this.$router, 'files-common-favorites')
             ) {
diff --git a/packages/web-app-files/src/mixins/actions/navigate.js b/packages/web-app-files/src/mixins/actions/navigate.js
index 9ccacf79f3e..c8290860051 100644
--- a/packages/web-app-files/src/mixins/actions/navigate.js
+++ b/packages/web-app-files/src/mixins/actions/navigate.js
@@ -29,7 +29,7 @@ export default {
               return false
             }
 
-            if (isSameResource(resources[0], this.currentFolder)) {
+            if (this.currentFolder !== null && isSameResource(resources[0], this.currentFolder)) {
               return false
             }
 
@@ -61,16 +61,14 @@ export default {
     },
     routeName() {
       if (isLocationPublicActive(this.$router, 'files-public-files')) {
-        createLocationPublic('files-public-files')
-        return
+        return createLocationPublic('files-public-files')
       }
 
       if (isLocationSpacesActive(this.$router, 'files-spaces-project')) {
-        createLocationPublic('files-spaces-project')
-        return
+        return createLocationPublic('files-spaces-project')
       }
 
-      createLocationSpaces('files-spaces-personal-home')
+      return createLocationSpaces('files-spaces-personal-home')
     }
   }
 }
diff --git a/packages/web-app-files/src/mixins/actions/showActions.js b/packages/web-app-files/src/mixins/actions/showActions.js
index c280cb21f95..08d545ebbaf 100644
--- a/packages/web-app-files/src/mixins/actions/showActions.js
+++ b/packages/web-app-files/src/mixins/actions/showActions.js
@@ -12,6 +12,11 @@ export default {
           label: () => this.$gettext('All Actions'),
           handler: this.$_showActions_trigger,
           isEnabled: ({ resources }) => {
+            // sidebar is currently only available inside files app
+            if (!this.isFilesAppActive) {
+              return false
+            }
+
             // we don't have batch actions in the right sidebar, yet.
             // return hardcoded `true` in all cases once we have them.
             return resources.length === 1
diff --git a/packages/web-app-files/src/mixins/actions/showDetails.js b/packages/web-app-files/src/mixins/actions/showDetails.js
index b0f121e4cdb..17c76730055 100644
--- a/packages/web-app-files/src/mixins/actions/showDetails.js
+++ b/packages/web-app-files/src/mixins/actions/showDetails.js
@@ -14,6 +14,11 @@ export default {
           // we don't have details in the trashbin, yet.
           // remove trashbin route rule once we have them.
           isEnabled: ({ resources }) => {
+            // sidebar is currently only available inside files app
+            if (!this.isFilesAppActive) {
+              return false
+            }
+
             if (isLocationCommonActive(this.$router, 'files-common-trash')) {
               return false
             }
diff --git a/packages/web-app-files/src/mixins/actions/showShares.js b/packages/web-app-files/src/mixins/actions/showShares.js
index d55f402148c..8965a448246 100644
--- a/packages/web-app-files/src/mixins/actions/showShares.js
+++ b/packages/web-app-files/src/mixins/actions/showShares.js
@@ -13,6 +13,11 @@ export default {
           label: () => this.$gettext('Share'),
           handler: this.$_showShares_trigger,
           isEnabled: ({ resources }) => {
+            // sidebar is currently only available inside files app
+            if (!this.isFilesAppActive) {
+              return false
+            }
+
             if (isLocationCommonActive(this.$router, 'files-common-trash')) {
               return false
             }
diff --git a/packages/web-app-files/tests/unit/components/Search/List.spec.js b/packages/web-app-files/tests/unit/components/Search/List.spec.js
index 313daf2da8f..f8c7f2f11fc 100644
--- a/packages/web-app-files/tests/unit/components/Search/List.spec.js
+++ b/packages/web-app-files/tests/unit/components/Search/List.spec.js
@@ -42,7 +42,15 @@ const files = [
     path: 'lorem.pdf',
     size: 50
   }
-]
+].map((file) => {
+  return {
+    ...file,
+    canDownload: () => true,
+    canBeDeleted: () => true,
+    isReceivedShare: () => false,
+    isMounted: () => false
+  }
+})
 
 describe('List component', () => {
   afterEach(() => {
@@ -140,7 +148,15 @@ function createStore(activeFiles) {
         getters: {
           activeFiles: () => activeFiles,
           totalFilesCount: () => ({ files: activeFiles.length, folders: 0 }),
-          totalFilesSize: () => getTotalSize(activeFiles)
+          totalFilesSize: () => getTotalSize(activeFiles),
+          currentFolder: () => {
+            return {
+              path: '',
+              canCreate() {
+                return false
+              }
+            }
+          }
         },
         mutations: {
           CLEAR_CURRENT_FILES_LIST: jest.fn(),

From 33ef769ddd7b57ffdb695c00b9498e1b90bd4d0f Mon Sep 17 00:00:00 2001
From: Dominik Schmidt <dev@dominik-schmidt.de>
Date: Thu, 17 Feb 2022 00:06:16 +0100
Subject: [PATCH 4/5] Create context menu only for selected items in search

---
 packages/web-app-files/src/components/Search/List.vue | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/packages/web-app-files/src/components/Search/List.vue b/packages/web-app-files/src/components/Search/List.vue
index a6d73f5e0c5..60534da59dd 100644
--- a/packages/web-app-files/src/components/Search/List.vue
+++ b/packages/web-app-files/src/components/Search/List.vue
@@ -10,6 +10,7 @@
     </no-content-message>
     <resource-table
       v-else
+      v-model="selected"
       class="files-table"
       :class="{ 'files-table-squashed': false }"
       :resources="paginatedResources"
@@ -22,7 +23,7 @@
       @rowMounted="rowMounted"
     >
       <template #contextMenu="{ resource }">
-        <context-actions :items="[resource]" />
+        <context-actions v-if="isResourceInSelection(resource)" :items="selected" />
       </template>
       <template #footer>
         <pagination :pages="paginationPages" :current-page="paginationPage" />
@@ -49,7 +50,7 @@ import ResourceTable from '../FilesList/ResourceTable.vue'
 import ContextActions from '../FilesList/ContextActions.vue'
 import debounce from 'lodash-es/debounce'
 import { mapMutations, mapGetters, mapActions } from 'vuex'
-import { computed } from '@vue/composition-api'
+import { computed, ref } from '@vue/composition-api'
 import ListInfo from '../FilesList/ListInfo.vue'
 import Pagination from '../FilesList/Pagination.vue'
 import MixinFileActions from '../../mixins/fileActions'
@@ -78,7 +79,9 @@ export default {
       items: computed(() => store.getters['Files/activeFiles'])
     })
 
+    const selected = ref([])
     return {
+      selected,
       paginatedResources,
       paginationPages,
       paginationPage,
@@ -128,6 +131,9 @@ export default {
       }, 250)
 
       visibilityObserver.observe(component.$el, { onEnter: debounced, onExit: debounced.cancel })
+    },
+    isResourceInSelection(resource) {
+      return this.selected?.includes(resource)
     }
   }
 }

From 64336c28040f97c013e36b1ca95d78d401364d77 Mon Sep 17 00:00:00 2001
From: Dominik Schmidt <dev@dominik-schmidt.de>
Date: Thu, 17 Feb 2022 00:17:51 +0100
Subject: [PATCH 5/5] Make isFilesAppActive available to sidebar actions

---
 .../src/components/SideBar/Actions/FileActions.vue          | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/packages/web-app-files/src/components/SideBar/Actions/FileActions.vue b/packages/web-app-files/src/components/SideBar/Actions/FileActions.vue
index 6455d379bdd..eb019f1ad3e 100644
--- a/packages/web-app-files/src/components/SideBar/Actions/FileActions.vue
+++ b/packages/web-app-files/src/components/SideBar/Actions/FileActions.vue
@@ -15,8 +15,14 @@ import { mapGetters } from 'vuex'
 import ActionMenuItem from '../../ActionMenuItem.vue'
 
 import FileActions from '../../../mixins/fileActions'
+import { useIsFilesAppActive } from '../../../composables/useIsFilesAppActive'
 
 export default {
+  setup() {
+    return {
+      isFilesAppActive: useIsFilesAppActive()
+    }
+  },
   name: 'FileActions',
   title: ($gettext) => {
     return $gettext('Actions')