You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by st...@apache.org on 2008/06/16 15:04:31 UTC

svn commit: r668147 - /jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/NodeImpl.java

Author: stefan
Date: Mon Jun 16 06:04:31 2008
New Revision: 668147

URL: http://svn.apache.org/viewvc?rev=668147&view=rev
Log:
JCR-1651: Node.addNode(String, String) doesn't prevent use of mixin types as primary type

Modified:
    jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/NodeImpl.java

Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/NodeImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/NodeImpl.java?rev=668147&r1=668146&r2=668147&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/NodeImpl.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/NodeImpl.java Mon Jun 16 06:04:31 2008
@@ -2121,6 +2121,9 @@
         sanityCheck();
 
         NodeTypeImpl nt = (NodeTypeImpl) session.getNodeTypeManager().getNodeType(nodeTypeName);
+        if (nt.isMixin()) {
+            throw new RepositoryException(nodeTypeName + ": not a primary node type");
+        }
         return internalAddNode(relPath, nt);
     }