You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Michael Dürig (JIRA)" <ji...@apache.org> on 2013/12/18 18:12:07 UTC

[jira] [Commented] (OAK-710) PermissionValidator: Proper permission evaluation for moving/renaming nodes

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

Michael Dürig commented on OAK-710:
-----------------------------------

At http://svn.apache.org/r1552023 I added a couple of additional tests to {{MoveTrackerTest}} by porting the relevant cases from http://svn.apache.org/viewvc/jackrabbit/sandbox/jackrabbit-microkernel/src/main/java/org/apache/jackrabbit/state/ChangeLog.java?view=markup (line 222 forward). The numbering of the tests corresponds to that of the cases in {{ChangeLog.java}}. 

Out of these tests {{MoveTrackerTest#test12}} fails: after {{>/a:/b/c >/b:/d}} the original source path of {{/d/c}} should be {{/a}} but is {{null}}.
 


> PermissionValidator: Proper permission evaluation for moving/renaming nodes
> ---------------------------------------------------------------------------
>
>                 Key: OAK-710
>                 URL: https://issues.apache.org/jira/browse/OAK-710
>             Project: Jackrabbit Oak
>          Issue Type: Sub-task
>          Components: core
>            Reporter: angela
>            Assignee: angela
>             Fix For: 0.15
>
>         Attachments: OAK-710.patch
>
>
> In jr-core renaming/moving nodes just this just requires MODIFY_CHILD_NODE_COLLECTION permission instead of remove + add node.
> However, the Validator interface currently doesn't allow for easy detection
> of move/rename operations. For backwards compatibility it was desirable
> to find a solution for this open issue.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)