Migrate SLC modules

[gpl/argeo-slc.git] / plugins / org.argeo.slc.client.ui.dist / src / main / java / org / argeo / slc / client / ui / dist / commands / DeleteArtifacts.java

diff --git a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/commands/DeleteArtifacts.java b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/commands/DeleteArtifacts.java
index 51143dc4e44f688da7acc2399676c136ac541a9f..b942ead9b3d77e3b14049458041babf9b9de922c 100644 (file)
--- a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/commands/DeleteArtifacts.java
+++ b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/commands/DeleteArtifacts.java
@@ -22,7 +22,7 @@ import javax.jcr.RepositoryException;
 
 import org.argeo.ArgeoException;
 import org.argeo.slc.client.ui.dist.DistPlugin;
-import org.argeo.slc.client.ui.dist.utils.CommandHelpers;
+import org.argeo.slc.jcr.SlcTypes;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
 import org.eclipse.core.commands.ExecutionException;
@@ -32,25 +32,22 @@ import org.eclipse.jface.viewers.ISelection;
 import org.eclipse.jface.viewers.IStructuredSelection;
 import org.eclipse.ui.IEditorPart;
 import org.eclipse.ui.IWorkbenchPart;
+import org.eclipse.ui.handlers.HandlerUtil;
 
 /**
  * Delete chosen artifacts from the current workspace.
  */
-
 public class DeleteArtifacts extends AbstractHandler {
        // private static final Log log = LogFactory.getLog(DeleteWorkspace.class);
+
        public final static String ID = DistPlugin.ID + ".deleteArtifacts";
        public final static String DEFAULT_LABEL = "Delete selected items";
-       // public final static String DEFAULT_ICON_PATH = "icons/removeItem.gif";
-
        public final static ImageDescriptor DEFAULT_ICON = DistPlugin
                        .getImageDescriptor("icons/removeItem.gif");
 
-       
        public Object execute(ExecutionEvent event) throws ExecutionException {
                try {
-                       IWorkbenchPart activePart = DistPlugin.getDefault().getWorkbench()
-                                       .getActiveWorkbenchWindow().getActivePage().getActivePart();
+                       IWorkbenchPart activePart = HandlerUtil.getActivePart(event);
 
                        if (activePart instanceof IEditorPart) {
                                ISelection selector = ((IEditorPart) activePart)
@@ -63,23 +60,25 @@ public class DeleteArtifacts extends AbstractHandler {
                                        String msg = "Your are about to definitively remove the "
                                                        + ((IStructuredSelection) selector).size()
                                                        + " selected artifacts.\n"
-                                                       + "Are you sure you want to proceed ?";
+                                                       + "Are you sure you want to proceed?";
 
                                        boolean result = MessageDialog.openConfirm(DistPlugin
                                                        .getDefault().getWorkbench().getDisplay()
-                                                       .getActiveShell(), "Confirm Delete", msg);
+                                                       .getActiveShell(), "Confirm Deletion", msg);
 
                                        if (result) {
                                                while (it.hasNext()) {
                                                        Node node = (Node) it.next();
-                                                       // we remove the artifactVersion, that is the parent
-                                                       node.getParent().remove();
-                                                       node.getSession().save();
+                                                       if (node.isNodeType(SlcTypes.SLC_ARTIFACT)) {
+                                                               // we remove the artifactVersion, that is the parent
+                                                               node.getParent().remove();
+                                                               node.getSession().save();
+                                                       }
                                                }
                                        }
                                }
                        }
-                       CommandHelpers.callCommand(RefreshDistributionOverviewPage.ID);
+                       // CommandHelpers.callCommand(RefreshDistributionOverviewPage.ID);
                } catch (RepositoryException re) {
                        throw new ArgeoException(
                                        "Unexpected error while deleting artifacts.", re);