You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by bt...@apache.org on 2017/11/03 02:33:28 UTC

[11/15] james-project git commit: MAILBOX-317 Bonus: Avoid useless mailboxId -> path -> mailboxId lookup

MAILBOX-317 Bonus: Avoid useless mailboxId -> path -> mailboxId lookup


Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/d078c5e6
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/d078c5e6
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/d078c5e6

Branch: refs/heads/master
Commit: d078c5e6050fc905e43b639f414780e3292ac5df
Parents: 0930817
Author: benwa <bt...@linagora.com>
Authored: Thu Nov 2 09:58:42 2017 +0700
Committer: benwa <bt...@linagora.com>
Committed: Fri Nov 3 09:32:32 2017 +0700

----------------------------------------------------------------------
 .../java/org/apache/james/mailbox/store/StoreMailboxManager.java | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/james-project/blob/d078c5e6/mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMailboxManager.java
----------------------------------------------------------------------
diff --git a/mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMailboxManager.java b/mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMailboxManager.java
index 3cd66b0..8387ced 100644
--- a/mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMailboxManager.java
+++ b/mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMailboxManager.java
@@ -807,9 +807,7 @@ public class StoreMailboxManager implements MailboxManager {
 
     @Override
     public void setRights(MailboxId mailboxId, MailboxACL mailboxACL, MailboxSession session) throws MailboxException {
-        MailboxMapper mapper = mailboxSessionMapperFactory.getMailboxMapper(session);
-        Mailbox mailbox = mapper.findMailboxById(mailboxId);
-        storeRightManager.setRights(mailbox.generateAssociatedPath(), mailboxACL, session);
+        storeRightManager.setRights(mailboxId, mailboxACL, session);
     }
 
     @Override


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