Code cleaning / documentation

[lgpl/argeo-commons.git] / server / plugins / org.argeo.jcr.ui.explorer / src / main / java / org / argeo / jcr / ui / explorer / commands / ImportFileSystem.java

diff --git a/server/plugins/org.argeo.jcr.ui.explorer/src/main/java/org/argeo/jcr/ui/explorer/commands/ImportFileSystem.java b/server/plugins/org.argeo.jcr.ui.explorer/src/main/java/org/argeo/jcr/ui/explorer/commands/ImportFileSystem.java
index 12880bad0de4d51e9b92c14c8280d349aa86af2a..e52694a1ca738736dc4b73404c0efd4a6d3d1ee5 100644 (file)
--- a/server/plugins/org.argeo.jcr.ui.explorer/src/main/java/org/argeo/jcr/ui/explorer/commands/ImportFileSystem.java
+++ b/server/plugins/org.argeo.jcr.ui.explorer/src/main/java/org/argeo/jcr/ui/explorer/commands/ImportFileSystem.java
@@ -2,9 +2,7 @@ package org.argeo.jcr.ui.explorer.commands;
 
 import javax.jcr.Node;
 
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
-import org.argeo.eclipse.ui.dialogs.Error;
+import org.argeo.eclipse.ui.ErrorFeedback;
 import org.argeo.eclipse.ui.jcr.views.AbstractJcrBrowser;
 import org.argeo.jcr.ui.explorer.wizards.ImportFileSystemWizard;
 import org.eclipse.core.commands.AbstractHandler;
@@ -15,9 +13,8 @@ import org.eclipse.jface.viewers.IStructuredSelection;
 import org.eclipse.jface.wizard.WizardDialog;
 import org.eclipse.ui.handlers.HandlerUtil;
 
+/** Import a local file system directory tree. */
 public class ImportFileSystem extends AbstractHandler {
-       private static Log log = LogFactory.getLog(ImportFileSystem.class);
-
        public Object execute(ExecutionEvent event) throws ExecutionException {
                ISelection selection = HandlerUtil.getActiveWorkbenchWindow(event)
                                .getActivePage().getSelection();
@@ -30,11 +27,6 @@ public class ImportFileSystem extends AbstractHandler {
                        try {
                                if (obj instanceof Node) {
                                        Node folder = (Node) obj;
-                                       // if (!folder.getPrimaryNodeType().getName()
-                                       // .equals(NodeType.NT_FOLDER)) {
-                                       // Error.show("Can only import to a folder node");
-                                       // return null;
-                                       // }
                                        ImportFileSystemWizard wizard = new ImportFileSystemWizard(
                                                        folder);
                                        WizardDialog dialog = new WizardDialog(
@@ -42,10 +34,10 @@ public class ImportFileSystem extends AbstractHandler {
                                        dialog.open();
                                        view.refresh(folder);
                                } else {
-                                       Error.show("Can only import to a node");
+                                       ErrorFeedback.show("Can only import to a node");
                                }
                        } catch (Exception e) {
-                               Error.show("Cannot import files to " + obj, e);
+                               ErrorFeedback.show("Cannot import files to " + obj, e);
                        }
                }
                return null;