This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git
commit f4767c6bdf77d0a3cca35b1a447609a7f7a3a848 Author: Benoit Tellier <[email protected]> AuthorDate: Tue Feb 18 14:08:39 2020 +0700 [REFACTORING] CassandraMailboxMapper Inline misnamed method This method is called authorizedMailbox but do not act nor take decisions on a mailbox object. The inlined version seems easier to understand. --- .../james/mailbox/cassandra/mail/CassandraMailboxMapper.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMailboxMapper.java b/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMailboxMapper.java index ed2573c..916b842 100644 --- a/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMailboxMapper.java +++ b/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMailboxMapper.java @@ -260,15 +260,10 @@ public class CassandraMailboxMapper implements MailboxMapper { @Override public List<Mailbox> findNonPersonalMailboxes(Username userName, Right right) { return userMailboxRightsDAO.listRightsForUser(userName) - .filter(mailboxId -> authorizedMailbox(mailboxId.getRight(), right)) + .filter(mailboxId -> mailboxId.getRight().contains(right)) .map(Pair::getLeft) .flatMap(this::retrieveMailbox) .collectList() .block(); } - - private boolean authorizedMailbox(MailboxACL.Rfc4314Rights rights, Right right) { - return rights.contains(right); - } - } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
