[ 
https://issues.apache.org/jira/browse/OAK-823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13660553#comment-13660553
 ] 

angela commented on OAK-823:
----------------------------

i tend to disagree... as this will allow someone to modify the jcr:uuid 
property which not
legal in JCR.

IMO the problem were are actually having is that we see a modification of an 
existing
property while in fact the original property was was removed and a new one was 
added. 
this should be reflected during the commit.

so, i am not in favor of the changes made in r1483721.

                
> Missing validation for jcr:uuid property
> ----------------------------------------
>
>                 Key: OAK-823
>                 URL: https://issues.apache.org/jira/browse/OAK-823
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: core
>            Reporter: angela
>            Assignee: angela
>
> A jcr:uuid property (defined by mix:referenceable) is defined to be
> an autocreated/mandatory/protected property.
> Currently the commit hooks don't make sure that in this case 
> - the value is a valid UUID (propertyAdded)
> - nor does it assert that this property can never be modified 
> (propertyChanged)
> note however, that i might be removed in case the mixin type is removed or
> the primary type gets modified and there is no valid definition left for
> that property. since the type-validator must assert that the mandatory
> property exists (if the node is referenceable) there is not extra verification
> needed in the case of removal.

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

Reply via email to