[dna-issues] [JBoss JIRA] Updated: (DNA-357) Hide dna:nodeDefinition Property from Users

[Brian Carothers (JIRA)]

     [ https://jira.jboss.org/jira/browse/DNA-357?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Carothers updated DNA-357:
--------------------------------

    Attachment: DNA-357_new_mixin.patch


I like the thought of directly extending nt:base, but it caused us to fail PredefinedNodeTypeTest.testBase.

The spec does give us the option to have nt:base have a supertype (e.g., referenceable).  Here's a patch that adds a new mixin type (dna:defined) with the dna:definition property and is a supertype of nt:base.  This patch would be applied in addition to the original patch.  I made the property protected tp keep users from accidentally corrupting it but not autocreated or mandatory as I did not want to add any guarantees about its availability in the API.

> Hide dna:nodeDefinition Property from Users
> -------------------------------------------
>
>                 Key: DNA-357
>                 URL: https://jira.jboss.org/jira/browse/DNA-357
>             Project: DNA
>          Issue Type: Feature Request
>          Components: JCR
>            Reporter: Brian Carothers
>            Priority: Minor
>         Attachments: DNA-357.patch, DNA-357_new_mixin.patch
>
>
> dna:nodeDefinition is a bit of an odd duck.  It is set on every property but doesn't always have a valid JCR definition.  This breaks an export-import round trip as the property cannot be set during import.  It also breaks a few TCK tests that assume a jcr:xmltext node only has two properties (jcr:primaryType and jcr:xmlcharacters), but that's really a TCK defect.
> Can we just suppress this property at the JCR layer?  It would still be available for internal use but would not appear to end users.  Alternatively, we should modify our definition for nt:base to include it as a protected property - this solution may violate the spec.

-- 
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