[ https://issues.apache.org/jira/browse/SLING-9090?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17040815#comment-17040815 ]
Bertrand Delacretaz commented on SLING-9090: -------------------------------------------- bq. ...either the parser should remove the support for Action.REMOVE and Action.REMOVE_ALL or the jcr implementation part should respect it... I would suggest first implementing stub versions of the required operations in the jcr-repoinit module, which fail loudly with a message indicating that these features are not supported. And later implement these features in that jcr-repoinit module. Unless nobody needs these features, in which case I agree with removing them from the parser grammar. > AclLine.Action.REMOVE and AclLine.Action.REMOVE_ALL not handled in jcr > implementation > ------------------------------------------------------------------------------------- > > Key: SLING-9090 > URL: https://issues.apache.org/jira/browse/SLING-9090 > Project: Sling > Issue Type: Bug > Components: Repoinit > Reporter: Angela Schreiber > Priority: Major > > [~bdelacretaz], while the documentation and the parser code provides the > ability to remove an individual or all access control entries, it seems the > JCR implementation doesn't actually support it. > using it may lead to odd side effects or failures.... so, i think either the > parser should remove the support for Action.REMOVE and Action.REMOVE_ALL or > the jcr implementation part should respect it... at the very minimum it > should spot any usage of it and fail the repo-init if there is no way to > implement it properly. -- This message was sent by Atlassian Jira (v8.3.4#803005)