james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Trần Tiến Đức (JIRA) <j...@apache.org>
Subject [jira] [Updated] (MAILBOX-378) Refactoring: push MessageManager::expunge into IMAP layer
Date Tue, 19 Feb 2019 10:34:00 GMT

     [ https://issues.apache.org/jira/browse/MAILBOX-378?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Trần Tiến Đức updated MAILBOX-378:
----------------------------------
    Description: 
With an expunged model, we get the deleted metadata **after** the delete operation proceeded,
which is not compatible with PreDeletionHooks(MAILBOX-379) design.

In this ticket, we propose ourselves to implement this expunge mechanism in the IMAP layer.

-1. Promote `List<MessageUid> retrieveDeletedMessage(MessageRange)` as a MessageManager
method.

-2. Implement a new `void delete(List<MessageUid>)` as a MessageManager method.

-3. Remove `MessageManager::expunge` & `MessageMapper::expungeMarkedForDeletionInMailbox`.

You will need to adapt MessageMapper tests and MessageManager tests accordingly.

  was:
With an expunged model, we get the deleted metadata **after** the delete operation proceeded,
which is not compatible with PreDeletionHooks design.

In this ticket, we propose ourselves to implement this expunge mechanism in the IMAP layer.

-1. Promote `List<MessageUid> retrieveDeletedMessage(MessageRange)` as a MessageManager
method.

-2. Implement a new `void delete(List<MessageUid>)` as a MessageManager method.

-3. Remove `MessageManager::expunge` & `MessageMapper::expungeMarkedForDeletionInMailbox`.

You will need to adapt MessageMapper tests and MessageManager tests accordingly.


> Refactoring: push MessageManager::expunge into IMAP layer
> ---------------------------------------------------------
>
>                 Key: MAILBOX-378
>                 URL: https://issues.apache.org/jira/browse/MAILBOX-378
>             Project: James Mailbox
>          Issue Type: Bug
>            Reporter: Trần Tiến Đức
>            Priority: Major
>
> With an expunged model, we get the deleted metadata **after** the delete operation proceeded,
which is not compatible with PreDeletionHooks(MAILBOX-379) design.
> In this ticket, we propose ourselves to implement this expunge mechanism in the IMAP
layer.
> -1. Promote `List<MessageUid> retrieveDeletedMessage(MessageRange)` as a MessageManager
method.
> -2. Implement a new `void delete(List<MessageUid>)` as a MessageManager method.
> -3. Remove `MessageManager::expunge` & `MessageMapper::expungeMarkedForDeletionInMailbox`.
> You will need to adapt MessageMapper tests and MessageManager tests accordingly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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


Mime
View raw message