YDA-5439: check if collection exists in destination before copy / move

UtrechtUniversity · Oct 4, 2023 · 84723c7 · 84723c7
1 parent f3f164a
commit 84723c7
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/research.py b/research.py
@@ -138,6 +138,10 @@ def api_research_folder_copy(ctx, folder_path, new_folder_path):
     if not collection.exists(ctx, folder_path):
         return api.Error('invalid_source', 'The original folder ' + folder_path + ' can not be found')
 
+    # Collection exists in destination?
+    if collection.exists(ctx, new_folder_path):
+        return api.Error('invalid_destination', 'Folder with this name already exists in destination. Please choose another destination')
+
     # All requirements OK
     try:
         collection.copy(ctx, folder_path, new_folder_path)
@@ -195,6 +199,10 @@ def api_research_folder_move(ctx, folder_path, new_folder_path):
     if not collection.exists(ctx, folder_path):
         return api.Error('invalid_source', 'The original folder ' + folder_path + ' can not be found')
 
+    # Collection exists in destination?
+    if collection.exists(ctx, new_folder_path):
+        return api.Error('invalid_destination', 'Folder with this name already exists in destination. Please choose another destination')
+
     # All requirements OK
     try:
         collection.move(ctx, folder_path, new_folder_path)