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 2012/11/30 15:55:26 UTC
svn commit: r1415669 -
/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/nodetype/ReadOnlyNodeTypeManager.java
Author: mduerig
Date: Fri Nov 30 14:55:25 2012
New Revision: 1415669
URL: http://svn.apache.org/viewvc?rev=1415669&view=rev
Log:
OAK-479: Adding a node to a node that doesn't accept children doesn't fail with ConstraintViolationException
Correct error message
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/nodetype/ReadOnlyNodeTypeManager.java
Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/nodetype/ReadOnlyNodeTypeManager.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/nodetype/ReadOnlyNodeTypeManager.java?rev=1415669&r1=1415668&r2=1415669&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/nodetype/ReadOnlyNodeTypeManager.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/nodetype/ReadOnlyNodeTypeManager.java Fri Nov 30 14:55:25 2012
@@ -316,7 +316,7 @@ public abstract class ReadOnlyNodeTypeMa
}
}
- throw new RepositoryException("No matching node definition found for " + this);
+ throw new RepositoryException("No matching node definition found for " + nodeName);
}
@Override
@@ -358,7 +358,7 @@ public abstract class ReadOnlyNodeTypeMa
}
}
- throw new RepositoryException("No matching node definition found for " + this);
+ throw new RepositoryException("No matching node definition found for " + targetNode.getName());
}
@Override
@@ -367,7 +367,7 @@ public abstract class ReadOnlyNodeTypeMa
NodeDefinition def = getDefinitionOrNull(parentNodeTypes, nodeName, nodeType);
if (def == null) {
- throw new RepositoryException("No matching node definition found for " + this);
+ throw new RepositoryException("No matching node definition found for " + nodeName);
}
return def;
@@ -474,7 +474,7 @@ public abstract class ReadOnlyNodeTypeMa
}
}
- throw new RepositoryException("No matching property definition found for " + this);
+ throw new RepositoryException("No matching property definition found for " + targetProperty.getName());
}
@Override