From bb3394885ca8b2e252ed94c86b1c0955f203fdd0 Mon Sep 17 00:00:00 2001 From: Simon Detheridge Date: Tue, 18 Feb 2020 11:37:02 +0000 Subject: [PATCH] Merge pull request #2604 from overleaf/cmg-duplicate-modal-root Stop duplicate file modal appearing when moving back to original folder - root edition GitOrigin-RevId: 11fe3b2457190f8bada010a344143a66b8967f34 --- .../FileTreeRootFolderController.js | 27 +++++++++++-------- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/services/web/frontend/js/ide/file-tree/controllers/FileTreeRootFolderController.js b/services/web/frontend/js/ide/file-tree/controllers/FileTreeRootFolderController.js index 7b8f29cc3d..ffc8c4ff86 100644 --- a/services/web/frontend/js/ide/file-tree/controllers/FileTreeRootFolderController.js +++ b/services/web/frontend/js/ide/file-tree/controllers/FileTreeRootFolderController.js @@ -22,19 +22,24 @@ define(['base'], App => } else { entities = [$(ui.draggable).scope().entity] } + + const ids = rootFolder.children.map(entity => entity.id) + for (let dropped_entity of Array.from(entities)) { - try { - ide.fileTreeManager.moveEntity(dropped_entity, rootFolder) - } catch (err) { - $modal.open({ - templateUrl: 'duplicateFileModalTemplate', - controller: 'DuplicateFileModalController', - resolve: { - fileName() { - return dropped_entity.name + if (!ids.includes(dropped_entity.id)) { + try { + ide.fileTreeManager.moveEntity(dropped_entity, rootFolder) + } catch (err) { + $modal.open({ + templateUrl: 'duplicateFileModalTemplate', + controller: 'DuplicateFileModalController', + resolve: { + fileName() { + return dropped_entity.name + } } - } - }) + }) + } } } $scope.$digest()