[ https://jira.jboss.org/jira/browse/DNA-591?page=com.atlassian.jira.plugin... ] Barry LaFond updated DNA-591: ----------------------------- Comment: was deleted -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-591?page=com.atlassian.jira.plugin... ] Randall Hauch commented on DNA-591: ----------------------------------- If if it turns out that this is particular case has an incorrect CND, we need to figure out is why we're not identifying errors/warnings in the CND importer, and why those errors are not being reported when the JcrRepository loads them into the node type manager. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-591?page=com.atlassian.jira.plugin... ] Barry LaFond updated DNA-591: ----------------------------- Attachment: (was: test_cnd.cnd) -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-591?page=com.atlassian.jira.plugin... ] Brian Carothers updated DNA-591: -------------------------------- Attachment: DNA-591.patch Here's a first-cut patch that provides a meaningful error message when this situation occurs. The error occurs because the 3rd type in the .cnd file (ddl:schemaDefinition) declares a residual child node and extends another type in the file (ddl:statement) that also declares a residual child node. This is legal in the current type system if the child node definition in the subclass is "narrower" than the child node definition in the superclass. That is, the subclass's residual child node must be defined such that any node that matches the subclass definition must always match the superclass's definition as well. This all means that the RNTM must check that (among other things) the required primary types for the subclass are the same as the required primary types for the superclass (or that they are even more restrictive). To do this, the RTNM must load the required primary types of the superclass and subclass and compare the two. But the required primary type of the residual child node definition in the superclass (ddl:statement) is ddl:problem, which is the last type declared in the file. Since ddl:problem hasn't been loaded yet, the attempt to load the required primary types for the residual child node of ddl:statement fails and you get an NPE (or a semi-useful error message if you apply this patch). Now, the immediate issue can be fixed by moving the definition for ddl:problem to be first in the list. Then you get this slightly more helpful error message: org.jboss.dna.jcr.nodetype.InvalidNodeTypeDefinitionException: Cannot redefine child node '{}*' with required type 'ddl:ddlProblem' with new child node that does not required that type or a subtype of that type. To my way of thinking, this is a correct message. ddl:schemaDefinition is attempting to redefine the required priamry type of its residual node from ddl:problem (in the supertype ddl:statement) to ddl:otherStatement. If ddl:otherStatement extended ddl:problem, this would be ok with me. Having said that, not everyone agrees with me on this and the point is still being debated. I'm not going to try to commit this patch unless/until a consensus is reached on the issue of overriding child node definitions. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-591?page=com.atlassian.jira.plugin... ] Brian Carothers updated DNA-591: -------------------------------- Attachment: DNA-591_clean.patch Attached patch that cleanly (i.e., with proper internationalization) corrects the NPE and provides a reasonably useful error message instead. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira