[jira] [Commented] (JAMES-3654) Remove GroupMembershipResolver and related implement

["Benoit Tellier (Jira)" <se...@james.apache.org>]

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

Benoit Tellier commented on JAMES-3654:
---------------------------------------

PR: https://github.com/apache/james-project/pull/670

Discussion on the mailing list: https://www.mail-archive.com/server-dev@james.apache.org/msg71095.html

> Remove GroupMembershipResolver and related implement
> ----------------------------------------------------
>
>                 Key: JAMES-3654
>                 URL: https://issues.apache.org/jira/browse/JAMES-3654
>             Project: James Server
>          Issue Type: Improvement
>          Components: mailbox
>            Reporter: Tung TRAN
>            Priority: Trivial
>
> Remove `org.apache.james.mailbox.acl.GroupMembershipResolver` , `SimpleGroupMembershipResolver` and related implement.
> Reason:
>  - Interface GroupMembershipResolver has only one implement is SimpleGroupMembershipResolver. As the author's comment: "There is no persistence. You will get only what you add".
> And nowhere in James call to `.addMemberShip` method => `.isMember` always return FALSE.
> Remove the exuberancy code to make it simpler.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org