Merge pull request #7488 from owncloud/prevent-unnecessary-propfind-d…

…uring-upload
owncloud · Aug 19, 2022 · 25b34b0 · 25b34b0
2 parents b8e5df4 + c5e363c
commit 25b34b0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 13 deletions.
diff --git a/changelog/unreleased/bugfix-prevent-unnecessary-propfind-during-upload b/changelog/unreleased/bugfix-prevent-unnecessary-propfind-during-upload
@@ -0,0 +1,6 @@
+Bugfix: Prevent unnecessary PROPFIND request during upload
+
+We've removed the unnecessary PROPFIND request at the start of each upload, increasing upload performance especially in larger folders.
+
+https://github.com/owncloud/web/issues/7486
+https://github.com/owncloud/web/pull/7488
diff --git a/packages/web-app-files/src/components/AppBar/CreateAndUpload.vue b/packages/web-app-files/src/components/AppBar/CreateAndUpload.vue
@@ -345,13 +345,7 @@ export default defineComponent({
           }
         }
 
-        let pathFileWasUploadedTo = file.meta.currentFolder
-        if (file.meta.relativeFolder) {
-          pathFileWasUploadedTo += file.meta.relativeFolder
-        }
-
-        const fileIsInCurrentPath = pathFileWasUploadedTo === this.currentPath
-
+        const fileIsInCurrentPath = file.meta.currentFolder === this.currentPath
         if (fileIsInCurrentPath) {
           bus.publish('app.files.list.load')
         }
@@ -788,12 +782,6 @@ export default defineComponent({
       this.$uppyService.publish('uploadStarted')
       await this.createDirectoryTree(files)
       this.$uppyService.publish('addedForUpload', files)
-
-      const reloadRequired = !!files.find((f) => f.meta.currentFolder === this.currentPath)
-      if (reloadRequired) {
-        bus.publish('app.files.list.load')
-      }
-
       this.$uppyService.uploadFiles(files)
     },