Closed
Bug 945635
Opened 11 years ago
Closed 6 years ago
[email] Implement trash emptying logic for IMAP and ActiveSync
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Firefox OS Graveyard
Gaia::E-Mail
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: asuth, Unassigned)
References
Details
Bug 945544 is introducing support for emptying the trash for POP3. Things are more complicated for IMAP and ActiveSync because there is an online component. The way Thunderbird does things for local folders is to just delete the existing mailbox and create a new one: http://mxr.mozilla.org/comm-central/source/mailnews/local/src/nsLocalMailFolder.cpp#646 For IMAP Thunderbird does locally destroy its stores, but the server operation is *not* implemented as a folder deletion and re-creation. Instead it does: - "STORE 1:* +FLAGS.SILENT (\Deleted)". (It also uses sequence numbers for some reason, I don't see why we would want to do that.) - Triggers an expunge - Uses DeleteSubFolders to perform a pattern list to find the sub-folders of the trash folder. The logic for this is surprisingly complex, probably because it isn't using or assuming an up-to-date folder list. Since our folder list is equally sketchy, it could make sense for us to imitate this, but on the other hand, if we didn't know about a folder to show it to the user, we probably don't want to delete it either. There's also some logic involving subscriptions, but we don't support/honor subscriptions, so that doesn't matter much to us. http://mxr.mozilla.org/comm-central/source/mailnews/imap/src/nsImapMailFolder.cpp#1440 http://mxr.mozilla.org/comm-central/source/mailnews/imap/src/nsImapProtocol.cpp#2787 http://mxr.mozilla.org/comm-central/source/mailnews/imap/src/nsImapProtocol.cpp#6735 ActiveSync's trash emptying needs more research in general. A quick search of protocol docs and the android exchange code doesn't show anything jumping out in terms of specialized support.
Comment 1•6 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•