Provide Display to ui event listener

[lgpl/argeo-commons.git] / eclipse / runtime / org.argeo.eclipse.ui.jcr / src / main / java / org / argeo / eclipse / ui / jcr / AbstractNodeContentProvider.java

diff --git a/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AbstractNodeContentProvider.java b/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AbstractNodeContentProvider.java
index 72a14baf850c8251b6ed04e64db89d04d736803a..10930a9ae312d3cf017bf8f042c432d7db1700d3 100644 (file)
--- a/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AbstractNodeContentProvider.java
+++ b/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AbstractNodeContentProvider.java
@@ -8,12 +8,17 @@ import javax.jcr.NodeIterator;
 import javax.jcr.RepositoryException;
 import javax.jcr.Session;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.argeo.ArgeoException;
 import org.argeo.eclipse.ui.AbstractTreeContentProvider;
 
 /** Canonic implementation of tree content provider manipulating JCR nodes. */
 public abstract class AbstractNodeContentProvider extends
                AbstractTreeContentProvider {
+       private final static Log log = LogFactory
+                       .getLog(AbstractNodeContentProvider.class);
+
        private Session session;
 
        public AbstractNodeContentProvider(Session session) {
@@ -63,10 +68,21 @@ public abstract class AbstractNodeContentProvider extends
                return children;
        }
 
+       /**
+        * To be overridden in order to filter out some nodes. Does nothing by
+        * default. The provided list is a temporary one and can thus be modified
+        * directly . (e.g. via an iterator)
+        */
+       protected List<Node> filterChildren(List<Node> children)
+                       throws RepositoryException {
+               return children;
+       }
+
        protected Object[] getChildren(Node node) throws RepositoryException {
                List<Node> nodes = new ArrayList<Node>();
                for (NodeIterator nit = node.getNodes(); nit.hasNext();)
                        nodes.add(nit.nextNode());
+               nodes = filterChildren(nodes);
                return nodes.toArray();
        }
 
@@ -81,7 +97,8 @@ public abstract class AbstractNodeContentProvider extends
                                else
                                        return node.getParent();
                        } catch (RepositoryException e) {
-                               throw new ArgeoException("Cannot get parent of " + element, e);
+                               log.warn("Cannot get parent of " + element + ": " + e);
+                               return null;
                        }
                } else if (element instanceof WrappedNode) {
                        WrappedNode wrappedNode = (WrappedNode) element;