[full-ci] Search all files with ocis backend

changelog/unreleased/enhancement-enable-search-all-files

@@ -0,0 +1,7 @@
+Enhancement: Enable search all files for ocis backend
+
+We've enabled the search all files feature for ocis backend:
+* Find files in sub directories
+* Find files in other places like project spaces
+
+https://github.com/owncloud/web/pull/6841

packages/web-app-files/src/components/FilesList/ResourceTable.vue

@@ -50,7 +50,7 @@
           :key="`${item.path}-${resourceDomSelector(item)}-${item.thumbnail}`"
           :resource="item"
           :is-path-displayed="arePathsDisplayed"
-          :parent-folder-name-default="defaultParentFolderName"
+          :parent-folder-name-default="getDefaultParentFolderName(item)"
           :is-thumbnail-displayed="areThumbnailsDisplayed"
           :is-extension-displayed="areFileExtensionsShown"
           :is-resource-clickable="isResourceClickable(item.id)"
@@ -176,6 +176,7 @@ import Rename from '../../mixins/actions/rename'
 import { defineComponent, PropType } from '@vue/composition-api'
 import { extractDomSelector, Resource } from '../../helpers/resource'
 import { ShareTypes } from '../../helpers/share'
+import { createLocationSpaces } from '../../router'
 
 export default defineComponent({
   mixins: [Rename],
@@ -353,7 +354,7 @@ export default defineComponent({
   },
   computed: {
     ...mapGetters(['configuration']),
-    ...mapState('Files', ['areFileExtensionsShown']),
+    ...mapState('Files', ['areFileExtensionsShown', 'spaces']),
     popperOptions() {
       return {
         modifiers: [
@@ -512,9 +513,6 @@ export default defineComponent({
     },
     currentLanguage() {
       return (this.$language?.current || '').split('_')[0]
-    },
-    defaultParentFolderName() {
-      return this.hasSpaces ? this.$gettext('Personal') : this.$gettext('All files and folders')
     }
   },
   methods: {
@@ -543,11 +541,20 @@ export default defineComponent({
       if (this.targetRoute === null) {
         return {}
       }
+
+      const matchingSpace = this.getMatchingSpace(storageId)
+
+      if (matchingSpace && matchingSpace?.driveType === 'project') {
+        return createLocationSpaces('files-spaces-project', {
+          params: { storageId, item: path.replace(/^\//, '') || undefined }
+        })
+      }
+
       return {
         name: this.targetRoute.name,
         query: this.targetRoute.query,
         params: {
-          item: path.replace(/^\//, ''),
+          item: path.replace(/^\//, '') || undefined,
           ...this.targetRoute.params,
           ...(storageId && { storageId })
         }
@@ -693,6 +700,21 @@ export default defineComponent({
         resourceType,
         ownerName: resource.owner[0].displayName
       })
+    },
+    getMatchingSpace(storageId) {
+      return this.spaces.find((space) => space.id === storageId)
+    },
+    getDefaultParentFolderName(resource) {
+      if (!this.hasSpaces) {
+        return this.$gettext('All files and folders')
+      }
+      const matchingSpace = this.getMatchingSpace(resource.storageId)
+
+      if (matchingSpace && matchingSpace?.driveType === 'project') {
+        return matchingSpace.name
+      }
+
+      return this.$gettext('Personal')
     }
   }
 })

packages/web-app-files/src/components/Search/Preview.vue

@@ -1,11 +1,12 @@
 <template>
-  <div class="files-search-preview" @click="$_fileActions_triggerDefaultAction(resource)">
+  <div class="files-search-preview">
     <oc-resource
       :resource="resource"
       :is-path-displayed="true"
       :folder-link="folderLink(resource)"
       :parent-folder-link="parentFolderLink(resource)"
       :parent-folder-name-default="defaultParentFolderName"
+      @click="$_fileActions_triggerDefaultAction(resource)"
     />
   </div>
 </template>
@@ -17,7 +18,7 @@ import { ImageDimension } from '../../constants'
 import { loadPreview } from '../../helpers/resource'
 import debounce from 'lodash-es/debounce'
 import Vue from 'vue'
-import { mapGetters } from 'vuex'
+import { mapGetters, mapState } from 'vuex'
 import { createLocationSpaces } from '../../router'
 import path from 'path'
 import { useCapabilitySpacesEnabled } from 'web-pkg/src/composables'
@@ -43,7 +44,8 @@ export default {
   setup() {
     return {
       hasSpaces: useCapabilitySpacesEnabled(),
-      resourceTargetLocation: createLocationSpaces('files-spaces-personal-home')
+      resourceTargetLocation: createLocationSpaces('files-spaces-personal-home'),
+      resourceTargetLocationSpace: createLocationSpaces('files-spaces-project')
     }
   },
   data() {
@@ -53,9 +55,21 @@ export default {
   },
   computed: {
     ...mapGetters(['configuration', 'user', 'getToken']),
+    ...mapState('Files', ['spaces']),
 
+    matchingSpace() {
+      return this.spaces.find((space) => space.id === this.resource.storageId)
+    },
     defaultParentFolderName() {
-      return this.hasSpaces ? this.$gettext('Personal') : this.$gettext('All files and folders')
+      if (!this.hasSpaces) {
+        return this.$gettext('All files and folders')
+      }
+
+      if (this.matchingSpace?.driveType === 'project') {
+        return this.matchingSpace.name
+      }
+
+      return this.$gettext('Personal')
     }
   },
   beforeMount() {
@@ -91,14 +105,20 @@ export default {
       return this.createFolderLink(path.dirname(file.path), file.storageId)
     },
     createFolderLink(path, storageId) {
-      if (this.resourceTargetLocation === null) {
+      if (this.resourceTargetLocation === null || this.resourceTargetLocationSpace === null) {
         return {}
       }
+
+      if (this.matchingSpace?.driveType === 'project') {
+        return createLocationSpaces('files-spaces-project', {
+          params: { storageId, item: path.replace(/^\//, '') || undefined }
+        })
+      }
+
       return {
         name: this.resourceTargetLocation.name,
-        query: this.resourceTargetLocation.query,
         params: {
-          item: path.replace(/^\//, ''),
+          item: path.replace(/^\//, '') || undefined,
           ...this.resourceTargetLocation.params,
           ...(storageId && { storageId })
         }

packages/web-app-files/src/helpers/resources.ts

@@ -122,7 +122,7 @@ export function buildSpace(space) {
     spaceReadmeData = space.special.find((el) => el.specialFolder.name === 'readme')
   }
 
-  if (space.root) {
+  if (space.root?.permissions) {
     for (const permission of space.root.permissions) {
       for (const role of SpacePeopleShareRoles.list()) {
         if (permission.roles.includes(role.name)) {
@@ -146,6 +146,7 @@ export function buildSpace(space) {
     extension: '',
     path: '',
     webDavPath: '',
+    driveType: space.driveType,
     type: 'space',
     isFolder: true,
     mdate: space.lastModifiedDateTime,
@@ -161,7 +162,7 @@ export function buildSpace(space) {
     privateLink: '',
     downloadURL: '',
     ownerDisplayName: '',
-    ownerId: '',
+    ownerId: space.owner?.user?.id,
     disabled,
     spaceQuota: space.quota,
     spaceRoles,

packages/web-app-files/src/index.js

@@ -22,6 +22,7 @@ import { archiverService, thumbnailService, Registry } from './services'
 import fileSideBars from './fileSideBars'
 import { buildRoutes } from './router'
 import get from 'lodash-es/get'
+import { clientService } from 'web-pkg/src/services'
 
 // dirty: importing view from other extension within project
 import SearchResults from '../../web-app-search/src/views/List.vue'
@@ -125,6 +126,9 @@ export default {
     bus.publish('app.search.register.provider', Registry.sdkSearch)
   },
   userReady({ store }) {
+    // Load spaces to make them available across the application
+    store.dispatch('Files/loadSpaces', { clientService })
+
     archiverService.initialize(
       store.getters.configuration.server || window.location.origin,
       get(store, 'getters.capabilities.files.archivers', [

packages/web-app-files/src/search/sdk/list.ts

@@ -24,7 +24,12 @@ export default class List implements SearchList {
     )
 
     return plainResources.map((plainResource) => {
-      const resource = buildResource(plainResource)
+      let resourceName = decodeURIComponent(plainResource.name)
+      if (resourceName.startsWith('/dav')) {
+        resourceName = resourceName.slice(4)
+      }
+
+      const resource = buildResource({ ...plainResource, name: resourceName })
       return { id: resource.id, data: resource }
     })
   }

packages/web-app-files/src/search/sdk/preview.ts

@@ -42,7 +42,12 @@ export default class Preview implements SearchPreview {
     return this.cache.set(
       term,
       plainResources.map((plainResource) => {
-        const resource = buildResource(plainResource)
+        let resourceName = decodeURIComponent(plainResource.name)
+        if (resourceName.startsWith('/dav')) {
+          resourceName = resourceName.slice(4)
+        }
+
+        const resource = buildResource({ ...plainResource, name: resourceName })
         return { id: resource.id, data: resource }
       })
     )

packages/web-app-files/src/store/actions.js

@@ -678,6 +678,21 @@ export default {
     })
   },
 
+  async loadSpaces(context, { clientService }) {
+    const graphClient = clientService.graphAuthenticated(
+      context.rootGetters.configuration.server,
+      context.rootGetters.getToken
+    )
+    const graphResponse = await graphClient.drives.listMyDrives()
+
+    if (!graphResponse.data) {
+      return
+    }
+
+    const spaces = graphResponse.data.value.map((space) => buildSpace(space))
+    context.commit('LOAD_SPACES', spaces)
+  },
+
   async loadPreview({ commit, rootGetters }, { resource, isPublic, dimensions, type }) {
     if (
       rootGetters.previewFileExtensions.length &&

packages/web-app-files/src/store/mutations.js

@@ -6,6 +6,12 @@ import { renameResource } from '../helpers/resources'
 import { ShareTypes } from '../helpers/share'
 
 export default {
+  LOAD_SPACES(state, spaces) {
+    state.spaces = spaces
+  },
+  CLEAR_SPACES(state) {
+    state.spaces = []
+  },
   LOAD_FILES(state, { currentFolder, files }) {
     state.currentFolder = currentFolder
     state.files = files

packages/web-app-files/src/store/state.js

@@ -6,6 +6,7 @@ export default {
   inProgress: [],
   shareOpen: null,
   versions: [],
+  spaces: [],
 
   /**
    * Outgoing shares and links from currently highlighted element

packages/web-app-files/tests/unit/components/FilesList/ResourceTable.spec.js

@@ -280,7 +280,10 @@ function getMountedWrapper(options = {}) {
           },
           modules: {
             Files: {
-              namespaced: true
+              namespaced: true,
+              state: {
+                spaces: []
+              }
             }
           }
         }),