[full-ci] Refactor webdav listFiles

changelog/unreleased/enhancement-webdav-client

@@ -5,3 +5,4 @@ We've added webdav support to the `web-client` package. This wraps the existing
 handles the differentiation of public link and user-specific webdav requests internally.
 
 https://github.com/owncloud/web/pull/7430
+https://github.com/owncloud/web/pull/7880

packages/web-app-files/src/helpers/resource/actions/transfer.ts

@@ -107,10 +107,9 @@ export class ResourceTransfer extends ConflictDialog {
     }
 
     const errors = []
-    const targetFolderResources = await this.clientService.webdav.listFiles(
-      this.targetSpace,
-      this.targetFolder
-    )
+    const targetFolderResources = (
+      await this.clientService.webdav.listFiles(this.targetSpace, this.targetFolder)
+    ).children
 
     const resolvedConflicts = await this.resolveAllConflicts(
       this.resourcesToMove,

packages/web-app-files/src/mixins/actions/rename.ts

@@ -73,10 +73,11 @@ export default {
       let parentResources
       if (isSameResource(resources[0], this.currentFolder)) {
         const parentPath = dirname(this.currentFolder.path)
-        parentResources = await (this.$clientService.webdav as WebDAV).listFiles(
-          space || this.space,
-          { path: parentPath }
-        )
+        parentResources = (
+          await (this.$clientService.webdav as WebDAV).listFiles(space || this.space, {
+            path: parentPath
+          })
+        ).children
       }
 
       const confirmAction = (newName) => {

packages/web-app-files/src/mixins/actions/restore.ts

@@ -68,10 +68,11 @@ export default {
           existingResources = existingResourcesCache[parentPath]
         } else {
           try {
-            existingResources = await this.$clientService.webdav.listFiles(this.space, {
-              path: parentPath
-            })
-            existingResources = existingResources.slice(1)
+            existingResources = (
+              await this.$clientService.webdav.listFiles(this.space, {
+                path: parentPath
+              })
+            ).children
           } catch (error) {
             missingFolderPaths.push(parentPath)
           }

packages/web-app-files/src/services/folder/loaderSpace.ts

@@ -53,8 +53,9 @@ export class FolderLoaderSpace implements FolderLoader {
       try {
         store.commit('Files/CLEAR_CURRENT_FILES_LIST')
 
-        const resources = yield webdav.listFiles(space, { path, fileId })
-        let currentFolder = resources.shift()
+        const listFilesResult = yield webdav.listFiles(space, { path, fileId })
+        const resources = listFilesResult.children
+        let currentFolder = listFilesResult.resource
         replaceInvalidFileRoute({ space, resource: currentFolder, path, fileId })
 
         if (path === '/') {

packages/web-app-files/tests/unit/helpers/resource/resourcesTransfer.spec.ts

@@ -7,6 +7,7 @@ import {
 } from '../../../../src/helpers/resource'
 import { mockDeep, mockReset } from 'jest-mock-extended'
 import { buildSpace, Resource } from 'web-client/src/helpers'
+import { ListFilesResult } from 'web-client/src/webdav/listFiles'
 
 const clientServiceMock = mockDeep<ClientService>()
 let resourcesToMove
@@ -74,8 +75,12 @@ describe('resourcesTransfer', () => {
     it.each([TransferType.COPY, TransferType.MOVE])(
       'should copy / move files without renaming them if no conflicts exist',
       async (action: TransferType) => {
+        const listFilesResult = {
+          resource: {} as Resource,
+          children: [] as Resource[]
+        } as ListFilesResult
         clientServiceMock.webdav.listFiles.mockReturnValueOnce(
-          new Promise((resolve) => resolve([] as Resource[]))
+          new Promise((resolve) => resolve(listFilesResult))
         )
         const resourcesTransfer = new ResourceTransfer(
           sourceSpace,

packages/web-app-files/tests/unit/mixins/actions/restore.spec.js

@@ -117,7 +117,7 @@ function getWrapper({
       $clientService: {
         webdav: {
           listFiles: jest.fn().mockImplementation(() => {
-            return []
+            return { resource: {}, children: [] }
           }),
           restoreFile: jest.fn().mockImplementation(() => {
             if (resolveRestore) {

packages/web-client/src/webdav/getFileInfo.ts

@@ -17,7 +17,7 @@ export const GetFileInfoFactory = (
           depth: 0,
           ...options
         })
-      )[0]
+      ).resource
     }
   }
 }

packages/web-client/src/webdav/listFiles.ts

@@ -1,11 +1,6 @@
-import { buildResource } from '../helpers/resource'
+import { buildResource, Resource } from '../helpers/resource'
 import { DavProperties, DavProperty } from './constants'
-import {
-  buildPublicSpaceResource,
-  isPublicSpaceResource,
-  Resource,
-  SpaceResource
-} from '../helpers'
+import { buildPublicSpaceResource, isPublicSpaceResource, SpaceResource } from '../helpers'
 import { WebDavOptions } from './types'
 import { urlJoin } from '../utils'
 
@@ -20,7 +15,7 @@ export const ListFilesFactory = ({ sdk }: WebDavOptions) => {
       space: SpaceResource,
       { path, fileId }: { path?: string; fileId?: string | number } = {},
       { depth = 1, davProperties }: ListFilesOptions = {}
-    ): Promise<Resource[]> {
+    ): Promise<ListFilesResult> {
       let webDavResources: any[]
       if (isPublicSpaceResource(space)) {
         webDavResources = await sdk.publicFiles.list(
@@ -44,9 +39,13 @@ export const ListFilesFactory = ({ sdk }: WebDavOptions) => {
         })
         if (!path) {
           const [rootFolder, ...children] = webDavResources
-          return [buildPublicSpaceResource(rootFolder), ...children.map(buildResource)]
+          return {
+            resource: buildPublicSpaceResource(rootFolder),
+            children: children.map(buildResource)
+          } as ListFilesResult
         }
-        return webDavResources.map(buildResource)
+        const resources = webDavResources.map(buildResource)
+        return { resource: resources[0], children: resources.slice(1) } as ListFilesResult
       }
 
       const listFilesCorrectedPath = async () => {
@@ -64,7 +63,7 @@ export const ListFilesFactory = ({ sdk }: WebDavOptions) => {
         if (fileId && fileId !== resources[0].fileId) {
           return listFilesCorrectedPath()
         }
-        return resources
+        return { resource: resources[0], children: resources.slice(1) } as ListFilesResult
       } catch (e) {
         if (e.statusCode === 404 && fileId) {
           return listFilesCorrectedPath()
@@ -74,3 +73,7 @@ export const ListFilesFactory = ({ sdk }: WebDavOptions) => {
     }
   }
 }
+export interface ListFilesResult {
+  resource: Resource
+  children?: Resource[]
+}

packages/web-pkg/src/composables/appDefaults/useAppFolderHandling.ts

@@ -63,9 +63,10 @@ export function useAppFolderHandling({
       })
 
       const path = dirname(pathResource.path)
-      const resources = await webdav.listFiles(space, {
+      const listFilesResult = await webdav.listFiles(space, {
         path
       })
+      const resources = [listFilesResult.resource, ...listFilesResult.children]
 
       if (resources[0].type === 'file') {
         store.commit('Files/LOAD_FILES', {