Closed Bug 748090 Opened 12 years ago Closed 12 years ago

pop3 filters that move new message to imap folders mostly won't work


(MailNews Core :: Filters, defect)

Windows 7
Not set


(thunderbird12 fixed, thunderbird13 fixed, thunderbird14 fixed)

Thunderbird 15.0
Tracking Status
thunderbird12 --- fixed
thunderbird13 --- fixed
thunderbird14 --- fixed


(Reporter: Bienvenu, Assigned: Bienvenu)



(Keywords: regression, Whiteboard: [fixed in 12.0.1])


(1 file)

pop3 incoming mail filters that move messages to imap folders mostly won't work. This is a regression from the pluggable store work. Prior to pluggable stores, we used a single new mail parser for all the messages in a pop3 download session. The moves to imap folders were coalesced by a imap move coalescer owned by the parser. Now, we use one new mail parser per message, which means the imap move coalescing doesn't work, except if the filter happens to apply to the last message downloaded in a download session.
The relnote would be

pop3 filters that move/copy incoming messages to imap folders will not work, for the most part, except for the last message downloaded in get new mail operation. For this reason, it is probably best to disable such filters until this bug is fixed. Filters that are run manually should work.
Keywords: relnote
sadly, there's no way to do an automated test for this, so we'd have to fallback onto a litmus test.
Flags: in-litmus?
should this not block bug 402392, rather than be blocked by bug 402392?
Keywords: regression
(In reply to Wayne Mery (:wsmwk) from comment #3)
> should this not block bug 402392, rather than be blocked by bug 402392?

yeah, sorry, forgot to switch that.
Blocks: 402392
No longer depends on: 402392
Attached patch proposed fixSplinter Review
This should fix it...
Flags: in-litmus? → in-litmus?(ludovic)
Comment on attachment 617740 [details] [diff] [review]
proposed fix

this makes pop3 to imap filters work for me, and xpcshell tests all still pass.
Attachment #617740 - Flags: review?(mbanner)
Attachment #617740 - Flags: review?(mbanner) → review+
Comment on attachment 617740 [details] [diff] [review]
proposed fix

[Triage Comment]
a=me for landing regression fix on branches.
Attachment #617740 - Flags: approval-comm-release+
Attachment #617740 - Flags: approval-comm-beta+
Attachment #617740 - Flags: approval-comm-aurora+
We relnoted this for 12.0 and its fixed in 12.0.1 & later so no need for relnote anymore.
Keywords: relnote
Flags: in-litmus?(ludovic)
You need to log in before you can comment on or make changes to this bug.