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 th...@apache.org on 2014/10/01 16:11:16 UTC

svn commit: r1628706 - /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/NodeImpl.java

Author: thomasm
Date: Wed Oct  1 14:11:16 2014
New Revision: 1628706

URL: http://svn.apache.org/r1628706
Log:
OAK-2157 Better error message for getPrimaryItem

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

Modified: jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/NodeImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/NodeImpl.java?rev=1628706&r1=1628705&r2=1628706&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/NodeImpl.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/NodeImpl.java Wed Oct  1 14:11:16 2014
@@ -689,14 +689,17 @@ public class NodeImpl<T extends NodeDele
                 // TODO: avoid nested calls
                 String name = getPrimaryNodeType().getPrimaryItemName();
                 if (name == null) {
-                    throw new ItemNotFoundException("No primary item present on node " + this);
+                    throw new ItemNotFoundException(
+                            "No primary item present on node " + NodeImpl.this);
                 }
                 if (hasProperty(name)) {
                     return getProperty(name);
                 } else if (hasNode(name)) {
                     return getNode(name);
                 } else {
-                    throw new ItemNotFoundException("Primary item " + name + " does not exist on node " + this);
+                    throw new ItemNotFoundException(
+                            "Primary item " + name + 
+                            " does not exist on node " + NodeImpl.this);
                 }
             }
         });