fix(files): Fix having to resolve conflicts twice when dropping files

Signed-off-by: Christopher Ng <chrng8@gmail.com>
nextcloud · Sep 25, 2024 · 1507877 · 1507877
1 parent b33444c
commit 1507877
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 13 deletions.
diff --git a/apps/files/src/actions/moveOrCopyAction.ts b/apps/files/src/actions/moveOrCopyAction.ts
@@ -142,19 +142,21 @@ export const handleCopyMoveNodeTo = async (node: Node, destination: Folder, meth
 				}
 			} else {
 				// show conflict file popup if we do not allow overwriting
-				const otherNodes = await getContents(destination.path)
-				if (hasConflict([node], otherNodes.contents)) {
-					try {
-						// Let the user choose what to do with the conflicting files
-						const { selected, renamed } = await openConflictPicker(destination.path, [node], otherNodes.contents)
-						// two empty arrays: either only old files or conflict skipped -> no action required
-						if (!selected.length && !renamed.length) {
+				if (!overwrite) {
+					const otherNodes = await getContents(destination.path)
+					if (hasConflict([node], otherNodes.contents)) {
+						try {
+							// Let the user choose what to do with the conflicting files
+							const { selected, renamed } = await openConflictPicker(destination.path, [node], otherNodes.contents)
+							// two empty arrays: either only old files or conflict skipped -> no action required
+							if (!selected.length && !renamed.length) {
+								return
+							}
+						} catch (error) {
+							// User cancelled
+							showError(t('files', 'Move cancelled'))
 							return
 						}
-					} catch (error) {
-						// User cancelled
-						showError(t('files', 'Move cancelled'))
-						return
 					}
 				}
 				// getting here means either no conflict, file was renamed to keep both files

diff --git a/apps/files/src/services/DropService.ts b/apps/files/src/services/DropService.ts
@@ -196,8 +196,7 @@ export const onDropInternalFiles = async (nodes: Node[], destination: Folder, co
 
 	for (const node of nodes) {
 		Vue.set(node, 'status', NodeStatus.LOADING)
-		// TODO: resolve potential conflicts prior and force overwrite
-		queue.push(handleCopyMoveNodeTo(node, destination, isCopy ? MoveCopyAction.COPY : MoveCopyAction.MOVE))
+		queue.push(handleCopyMoveNodeTo(node, destination, isCopy ? MoveCopyAction.COPY : MoveCopyAction.MOVE, true))
 	}
 
 	// Wait for all promises to settle