[
https://issues.apache.org/jira/browse/OAK-411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13488037#comment-13488037
]
angela commented on OAK-411:
----------------------------
just one additional comment:
currently there is some validation done in ReadWriteNodeTypeManager before
registration or unregistration of nodetypes. IMHO that needs to be done
in the RegistrationValidator (or however that validator is called in the
end) as we also have to make sure that no invalid node type definitions are
created or modified by means of using oak-api (circumventing jcr node type
management API)... where or not some preliminary validations
should be done in the NodeTypeManager methods upfront is most probably a
question of preference... however, i think it is mandatory upon commit.
> Validator for node type management
> ----------------------------------
>
> Key: OAK-411
> URL: https://issues.apache.org/jira/browse/OAK-411
> Project: Jackrabbit Oak
> Issue Type: Sub-task
> Components: core
> Reporter: angela
>
> while working on the protected item definitions i realized that
> we don't have any validation for node type management operations
> on the oak-level.
> IMO there should be a separate validator for content changes underneath
> /jcr:system/jcr:nodeTypes with the following characteristics:
> - prevent removal of built-in node types
> - assert validity of node type definition
> - prevent changes to registered node types that would render
> existing nodes invalid
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira