fix(ContentNavigator): added name checking for duplicates on drag and drop (#1665)

* feat(ContentNavigator): added name checking for duplicates on drag and drop

Signed-off-by: Kishan Patel <[email protected]>

* fix(ContentNavigator): changed to server side handling for drag and drop on restserveradapter

Signed-off-by: Kishan Patel <[email protected]>

---------

Signed-off-by: Kishan Patel <[email protected]>

Author: kpatl1

client/src/components/ContentNavigator/ContentDataProvider.ts

@@ -717,13 +717,14 @@ class ContentDataProvider
     path: string,
     displayErrorMessages: boolean = true,
   ): Promise<boolean> {
-    const folder = await this.model.createFolder(target, basename(path));
+    const folderName = basename(path);
+    const folder = await this.model.createFolder(target, folderName);
     let success = true;
     if (!folder) {
       displayErrorMessages &&
         window.showErrorMessage(
           l10n.t(Messages.FileDropError, {
-            name: basename(path),
+            name: folderName,
           }),
         );
 

client/src/connection/rest/RestServerAdapter.ts

@@ -447,12 +447,17 @@ class RestServerAdapter implements ContentAdapter {
       },
     };
 
-    const response =
-      await this.fileSystemApi.updateFileOrDirectoryOnSystem(params);
-    delete this.fileMetadataMap[currentFilePath];
-    this.updateFileMetadata(newFilePath, response);
+    try {
+      const response =
+        await this.fileSystemApi.updateFileOrDirectoryOnSystem(params);
+      delete this.fileMetadataMap[currentFilePath];
+      this.updateFileMetadata(newFilePath, response);
 
-    return this.filePropertiesToContentItem(response.data).vscUri;
+      return this.filePropertiesToContentItem(response.data).vscUri;
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    } catch (error) {
+      return;
+    }
   }
 
   public async renameItem(