[
https://issues.apache.org/jira/browse/JCR-3350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13403785#comment-13403785
]
angela commented on JCR-3350:
-----------------------------
hi jeff,
i just applied the patch a realized that the utility has an adobe license
header and adobe package names. that's obviously not
optimal for apache jackrabbit. furthermore the utility methods throw
SlingException... there is however not dependency to sling
in jackrabbit. finally the methods implicitly call sesssion.save... while
that's useful in certain cases it isn't when it comes to
request processing that should be atomic. in order to cover the latter i would
suggest that the method should have an autosave
flag (or something similar).
would it be possible for you to revise the patch and maybe create it against
the jackrabbit code base? thanks a lot!
> Easy-to-use utility class for adding ACEs to nodes
> --------------------------------------------------
>
> Key: JCR-3350
> URL: https://issues.apache.org/jira/browse/JCR-3350
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: jackrabbit-jcr-commons
> Reporter: Jeff Young
> Assignee: angela
> Labels: newbie, patch
> Attachments:
> JCR-3350_-_Easy-to-use_utility_class_for_adding_ACEs_to_nodes.patch
>
>
> There should be any easy (one-line) method for adding an ACE to a node in a
> repo supporting resource-based ACLs.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
