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

Julian Reschke commented on JCR-3439:
-------------------------------------

Proposed changes (thanks Angela):

PrincipalManagerImpl line 214:
    - private class CheckedGroup implements Group {
    + private class CheckedGroup implements Group, JackrabbitPrincipal {

Also add tests:

a) principal from ACL equals same principal from principalMgr
b) principal from Authorizable equals same principal from principalMgr
c) ...and vice versa 
                
> PrincipalManagerImpl.CheckedGroup should implement JackrabbitPrincipal
> ----------------------------------------------------------------------
>
>                 Key: JCR-3439
>                 URL: https://issues.apache.org/jira/browse/JCR-3439
>             Project: Jackrabbit Content Repository
>          Issue Type: Bug
>          Components: jackrabbit-core
>            Reporter: Manfred Baedke
>
> otherwise org.apache.jackrabbit.api.security.principal.JackrabbitPrincipal 
> won't do the right thing on equals().

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to