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 an...@apache.org on 2012/08/14 10:34:55 UTC

svn commit: r1372771 - /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/SessionDelegate.java

Author: angela
Date: Tue Aug 14 08:34:55 2012
New Revision: 1372771

URL: http://svn.apache.org/viewvc?rev=1372771&view=rev
Log:
fix idention and add TODO regarding identifier handling

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

Modified: jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/SessionDelegate.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/SessionDelegate.java?rev=1372771&r1=1372770&r2=1372771&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/SessionDelegate.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/SessionDelegate.java Tue Aug 14 08:34:55 2012
@@ -192,11 +192,14 @@ public class SessionDelegate {
 
     @CheckForNull
     public NodeDelegate getNodeByIdentifier(String id) {
+        // TODO delegate identifier handling  to the OAK-API (or oak-utility)
+        // TODO as stated in NodeDelegate#getIdentifier() a non-uuid ID shoud
+        // TODO consisting of closest referenceable parent and a relative path
+        // TODO irrespective of the accessibility of the parent node(s)
         if (id.startsWith("/")) {
             Tree tree = getTree(id);
             return tree == null ? null : new NodeDelegate(this, tree);
-        }
-        else {
+        } else {
             // referenceable
             return findByJcrUuid(id);
         }