[ https://issues.apache.org/jira/browse/MAILBOX-311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16198516#comment-16198516 ]
Tellier Benoit edited comment on MAILBOX-311 at 10/10/17 11:12 AM: ------------------------------------------------------------------- We should have an immutable definition of a mailbox. http://jqno.nl/equalsverifier/errormessages/mutability-equals-depends-on-mutable-field/ We should at least have the mailboxId immutable, which means changing a bit relation between StoreMailboxManager and its mappers. was (Author: btellier): We should have an immutable definition of a mailbox. http://jqno.nl/equalsverifier/errormessages/mutability-equals-depends-on-mutable-field/ We can not base it on mailboxId because it is back-end set. I propose to choose namespace, user, path and uidValidity as definition of a mailbox > SimpleMailbox does not match Bean contract > ------------------------------------------ > > Key: MAILBOX-311 > URL: https://issues.apache.org/jira/browse/MAILBOX-311 > Project: James Mailbox > Issue Type: Bug > Components: store > Affects Versions: master > Reporter: Tellier Benoit > Labels: easyfix > Fix For: master > > > Right now SimpleMailbox does not match bean contract: > - Equals relies on mailboxId > - hashCode relies on path -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org