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

Marco Piovesana edited comment on OAK-3328 at 11/19/16 6:25 PM:
----------------------------------------------------------------

Hi Marcel,
thanks for the hint. I did try to look at the patch and modify 
_checkPrecondition_ method. I was able to modify the property but i still get 
an error when trying to save the session.

It fails inside the class _VersionEditor_ because of this check:
{code:java}
    public void propertyChanged(PropertyState before, PropertyState after) 
throws CommitFailedException {
        if(!this.isVersionable()) {
            if(!this.isVersionProperty(after) && this.isReadOnly) {
                throwCheckedIn("Cannot change property " + after.getName() + " 
on checked in node");
            }
{code}

Is possible to extract the NodeDefinition from the NodeState (to check weather 
the node has the "IGNORE" property or not)? I tried but I don't seem to find a 
way to do it.
This way i can modify the method that defines the value for the _readOnly_ 
property. By the way, is that the right thing to do?

Marco.


was (Author: iosonomarco):
Hi Marcel,
thanks for the hint. I did try to look at the patch and modify 
_checkPrecondition_ method. I was able to modify the property but i still get 
an error when trying to save the session.

It fails inside the class _VersionEditor_ because of this check:
{code:java}
    public void propertyChanged(PropertyState before, PropertyState after) 
throws CommitFailedException {
        if(!this.isVersionable()) {
            if(!this.isVersionProperty(after) && this.isReadOnly) {
                throwCheckedIn("Cannot change property " + after.getName() + " 
on checked in node");
            }
{code}

Is possible to extract the NodeDefinition from the NodeState (to check weather 
the node has the "IGNORE" property or not)?  Is that the right thing to do?

Marco.

> checked-in state should only affect properties with OPV!=IGNORE
> ---------------------------------------------------------------
>
>                 Key: OAK-3328
>                 URL: https://issues.apache.org/jira/browse/OAK-3328
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: jcr
>            Reporter: Julian Reschke
>
> We currently (as of OAK-3310) protect all properties from changes when the 
> node is checked-in. According to the spec 
> (http://www.day.com/specs/jcr/2.0/15_Versioning.html#15.2.2%20Read-Only%20on%20Check-In),
>  this should only be the case when on-parent-version is != "IGNORE".
> (It seems this doesn't have TCK coverage)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to