Merge pull request #610 from IBM/fix/delete-library

Update configuration for deleting a library
IBM · Oct 22, 2024 · d0c11ed · d0c11ed
2 parents 6aebdac + 6a3931f
commit d0c11ed
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/l10n/bundle.l10n.json b/l10n/bundle.l10n.json
@@ -169,7 +169,7 @@
   "Cancel": "Cancel",
   "Cleared {0} *{1}.": "Cleared {0} *{1}.",
   "Error clearing library! {0}": "Error clearing library! {0}",
-  "Are you sure you want to delete {0} {1}?": "Are you sure you want to delete {0} {1}?",
+  "This option will delete library {0} (and all of its contents) from the IBM i system. Are you sure you want to do this (this action cannot be undone)?": "This option will delete library {0} (and all of its contents) from the IBM i system. Are you sure you want to do this (this action cannot be undone)?",
   "Deleting object {0} {1}...": "Deleting object {0} {1}...",
   "Deleted {0} {1}": "Deleted {0} {1}",
   "Error deleting object! {0}": "Error deleting object! {0}",
@@ -178,6 +178,7 @@
   "Source file name must be 10 characters or less": "Source file name must be 10 characters or less",
   "Creating source file {0}.": "Creating source file {0}.",
   "Error creating source file! {0}": "Error creating source file! {0}",
+  "Are you sure you want to delete {0} {1}?": "Are you sure you want to delete {0} {1}?",
   "Are you sure you want to delete {0}?": "Are you sure you want to delete {0}?",
   "Deleting member {0}...": "Deleting member {0}...",
   "Deleted {0}": "Deleted {0}",

diff --git a/src/views/projectExplorer/index.ts b/src/views/projectExplorer/index.ts
@@ -1002,7 +1002,8 @@ export default class ProjectExplorer implements TreeDataProvider<ProjectExplorer
         if (element) {
           const objectBrowseObject = toObjectBrowserObject(element);
 
-          let result = await window.showWarningMessage(l10n.t('Are you sure you want to delete {0} {1}?', objectBrowseObject.path, objectBrowseObject.object.type.toUpperCase()), l10n.t(`Yes`), l10n.t(`Cancel`));
+          const library = element.label!.toString();
+          let result = await window.showWarningMessage(l10n.t('This option will delete library {0} (and all of its contents) from the IBM i system. Are you sure you want to do this (this action cannot be undone)?', library), l10n.t(`Yes`), l10n.t(`Cancel`));
 
           if (result === l10n.t(`Yes`)) {
             const ibmi = getInstance();
@@ -1366,7 +1367,7 @@ export default class ProjectExplorer implements TreeDataProvider<ProjectExplorer
     }
   }
 
-  async resolveTreeItem(item: TreeItem, element: ProjectExplorerTreeItem, token: CancellationToken): Promise<ProjectExplorerTreeItem>  {
+  async resolveTreeItem(item: TreeItem, element: ProjectExplorerTreeItem, token: CancellationToken): Promise<ProjectExplorerTreeItem> {
     if (element.getToolTip) {
       element.tooltip = await element.getToolTip();
     }