[ 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:22 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)? 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} do you know how can i check the OPV settings of this property (or its parent node) to see if is set to "IGNORE"? 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)