New notification bar introduced with bug 151244 is displayed every time I select a mail with a receipt request. This is great if I want to choose later to send or ignore the request, but if I already chose to ignore that request the notification should not be displayed.
This is working fine for me on trunk builds, is this still broken for you?
(In reply to comment #1) > is this still broken for you? Still broken on Gecko/20100613 Lanikai/3.1.1pre. Do I have to test it with a 3.2 nightly?
Ok, I've just done a bit more testing - it appears this is broken on Local Folders / POP only. IMAP appears to be working fine.
Confirming broken for 3.1RC2 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:22.214.171.124) Gecko/20100608 Lightning/1.0b2 Thunderbird/3.1 Return receipt notification bar is always showing again, no matter if you ignore or confirm the request (adjusting summary accordingly) Maybe it's related to bug 380339 - Return receipt status is only stored in .msf file and not in mbox file which I think should be fixed along with this one
David, can you take a look at this one?
This code only works for IMAP. I suppose the expectation is that some other code is supposed to clear the local hdr flag. http://mxr.mozilla.org/comm-central/source/mailnews/extensions/mdn/src/nsMsgMdnGenerator.cpp#180
Some code was removed here (middle section): https://bugzilla.mozilla.org/attachment.cgi?id=437900&action=diff#a/mail/base/content/mailWindowOverlay.js_sec6 That seems to remove the storing of the message flags. I just wonder why there is similar code in the c++ part, but apparently only for imap.
I have the same problem (multiple return receipt requests on same msg) with Thunderbird 3.1 and a message from my Yahoo mail account downloaded through POP. Using Mozilla/5.0 Gecko/20100608 Thunderbird/3.1.
Created attachment 454761 [details] [diff] [review] proposed fix fix is to make sure we set the mdn sent flag locally when the request is ignored or sent, as we did before.
Comment on attachment 454761 [details] [diff] [review] proposed fix >+ if (imapFolder) >+ return imapFolder->StoreImapFlags(kImapMsgMDNSentFlag, PR_TRUE, &key, 1, nsnull); >+} Not returning anything for local folders... >+ ClearMDNNeededFlag(m_folder, m_key); No assertion this time?
Created attachment 454865 [details] [diff] [review] fix addressing comments added assertion, fixed return, and combined common code.
Comment on attachment 454865 [details] [diff] [review] fix addressing comments >+ var askUser = mdnGenerator.process(MDN_DISPOSE_TYPE_DISPLAYED, msgWindow, msgFolder, >+ msgHdr.messageKey, mimeHdr, false); [Should we be able to predict this value?]
(In reply to comment #12) > (From update of attachment 454865 [details] [diff] [review]) > >+ var askUser = mdnGenerator.process(MDN_DISPOSE_TYPE_DISPLAYED, msgWindow, msgFolder, > >+ msgHdr.messageKey, mimeHdr, false); > [Should we be able to predict this value?] we can ignore this return value - what we're really interested in is the initialization of the mdnGenerator that happens when we call mdnGenerator.process, which enables UserDeclined() to act on the right message.
fixed on trunk.
Comment on attachment 454865 [details] [diff] [review] fix addressing comments very much want this for 3.1.1
Comment on attachment 454865 [details] [diff] [review] fix addressing comments a=Standard8 Please land this just on comm-1.9.2 default. I'll sort out what we're doing with relbranches later.
Checked into 3.1 branch and relbranch: http://hg.mozilla.org/releases/comm-1.9.2/rev/814e33ede3c1 http://hg.mozilla.org/releases/comm-1.9.2/rev/be6d5c421ed0