You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by md...@apache.org on 2013/05/01 16:28:20 UTC

svn commit: r1478020 - /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java

Author: mduerig
Date: Wed May  1 14:28:20 2013
New Revision: 1478020

URL: http://svn.apache.org/r1478020
Log:
OAK-753: TreeImpl exposes hidden child trees
add fixme, use NodeStateUtils.isHidden instead of ad-hoc test

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java?rev=1478020&r1=1478019&r2=1478020&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java Wed May  1 14:28:20 2013
@@ -82,6 +82,7 @@ import org.apache.jackrabbit.oak.api.Typ
 import org.apache.jackrabbit.oak.commons.PathUtils;
 import org.apache.jackrabbit.oak.core.IdentifierManager;
 import org.apache.jackrabbit.oak.plugins.memory.PropertyStates;
+import org.apache.jackrabbit.oak.spi.state.NodeStateUtils;
 import org.apache.jackrabbit.oak.util.TreeUtil;
 
 /**
@@ -287,7 +288,8 @@ public class NodeDelegate extends ItemDe
         return transform(filter(getTree().getProperties().iterator(), new Predicate<PropertyState>() {
                 @Override
                 public boolean apply(PropertyState property) {
-                    return !property.getName().startsWith(":");
+                    // FIXME clarify handling of hidden items (OAK-753)
+                    return !NodeStateUtils.isHidden(property.getName());
                 }
                 }),
                 new Function<PropertyState, PropertyDelegate>() {