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 2019/11/20 01:56:43 UTC

[james-project] 15/49: [Refactoring] JPAMailbox inline generateAssociatedPath

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 92a00ae315b93ab623beaa0f90afffe785742d64
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Sat Nov 16 10:43:04 2019 +0700

    [Refactoring] JPAMailbox inline generateAssociatedPath
---
 .../java/org/apache/james/mailbox/jpa/mail/model/JPAMailbox.java   | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/mailbox/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/JPAMailbox.java b/mailbox/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/JPAMailbox.java
index 45922af..347a2c5 100644
--- a/mailbox/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/JPAMailbox.java
+++ b/mailbox/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/JPAMailbox.java
@@ -134,11 +134,8 @@ public class JPAMailbox {
     }
 
     public Mailbox toMailbox() {
-        return new Mailbox(generateAssociatedPath(), uidValidity, new JPAId(mailboxId));
-    }
-
-    public MailboxPath generateAssociatedPath() {
-        return new MailboxPath(namespace, Username.of(user), name);
+        MailboxPath path = new MailboxPath(namespace, Username.of(user), name);
+        return new Mailbox(path, uidValidity, new JPAId(mailboxId));
     }
 
     public void setMailboxId(long mailboxId) {


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