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)