The default bug view has changed. See this FAQ.

Make MDN (return receipts) work for non-standard headers too, and make the MDN confirmation message say which addresses the receipt will be sent to.

RESOLVED FIXED in seamonkey2.13

Status

SeaMonkey
MailNews: General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.13

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

5 years ago
Planned Fixes in this bug:
1. Port parts of Bug 568447:
1.1 Make it work correctly with the non-standard Return-Receipt-To.
1.2 The MDN bar does not need to be hidden for deleted IMAP messages.
2. Ports Bug 360800:
2.1 Make the MDN confirmation dialog say which addresses the receipt will be sent to (can be multiple).

From Bug 568447:

> Fix the nits, and also i made it work correctly with the non-standard
> Return-Receipt-To. (Strangely enough i had a mail from an outlook user, in
> one mail its the standard header, in the next the non-standard, go figure...)

> Also i don't think the mdn bar needs to be hidden for deleted imap messages
> if they are showing. It probably made sense when it was a dialog...
(Assignee)

Updated

5 years ago
Summary: Make MDN (return receipts) work for non-standard headers too, and make the MDN confirmation dialog say which addresses the receipt will be sent to. → Make MDN (return receipts) work for non-standard headers too, and make the MDN confirmation message say which addresses the receipt will be sent to.
(Assignee)

Comment 1

5 years ago
Created attachment 634102 [details] [diff] [review]
Patch v1.0 Proposed fix.
Attachment #634102 - Flags: review?(iann_bugzilla)
Attachment #634102 - Flags: feedback?(neil)

Comment 2

5 years ago
Comment on attachment 634102 [details] [diff] [review]
Patch v1.0 Proposed fix.

>-<!ENTITY mdnBarIgnoreButton.label "Ignore Request">
>-<!ENTITY mdnBarSendButton.label "Send Receipt">
>+<!ENTITY mdnBarIgnoreButton2.label "Ignore">
>+<!ENTITY mdnBarIgnoreButton2.accesskey "I">
>+<!ENTITY mdnBarSendButton2.label "Send Return Receipt">
>+<!ENTITY mdnBarSendButton2.accesskey "S">
Why are we changing the labels?

>+                 aMimeHdr.extractHeader("Return-Receipt-To", false); // not
not what?

>                  class="msgNotificationBarText">&mdnBarMessage.label;</description>
Didn't you get rid of this entity?
(Assignee)

Comment 3

5 years ago
> Why are we changing the labels?
No idea. I'll revert this labels

> not what?
or not RFC 3798.

>>                  class="msgNotificationBarText">&mdnBarMessage.label;</description>
> Didn't you get rid of this entity?
Sigh. insufficient qrefresh. New patch RSN.
(Assignee)

Comment 4

5 years ago
Created attachment 635769 [details] [diff] [review]
Patch v1.1

>>-<!ENTITY mdnBarIgnoreButton.label "Ignore Request">
>>-<!ENTITY mdnBarSendButton.label "Send Receipt">
>>+<!ENTITY mdnBarIgnoreButton2.label "Ignore">
>>+<!ENTITY mdnBarIgnoreButton2.accesskey "I">
>>+<!ENTITY mdnBarSendButton2.label "Send Return Receipt">
>>+<!ENTITY mdnBarSendButton2.accesskey "S">
> Why are we changing the labels?
Reverted labels.

>>+                 aMimeHdr.extractHeader("Return-Receipt-To", false); // not
> not what?
Fixed comment.

>>                  class="msgNotificationBarText">&mdnBarMessage.label;</description>
> Didn't you get rid of this entity?
Removed for sure this time.
Attachment #634102 - Attachment is obsolete: true
Attachment #634102 - Flags: review?(iann_bugzilla)
Attachment #634102 - Flags: feedback?(neil)
Attachment #635769 - Flags: review?(iann_bugzilla)

Updated

5 years ago
Attachment #635769 - Attachment is patch: true

Comment 5

5 years ago
Comment on attachment 635769 [details] [diff] [review]
Patch v1.1

>+++ b/suite/locales/en-US/chrome/mailnews/messenger.dtd
>+<!ENTITY mdnBarIgnoreButton.label "Ignore">
>+<!ENTITY mdnBarIgnoreButton.accesskey "I">
>+<!ENTITY mdnBarSendButton.label "Send Return Receipt">
>+<!ENTITY mdnBarSendButton.accesskey "S">
I guess the meaning of these labels is not changing so probably no need to change the entity name, but maybe Neil was suggesting you didn't need to change the labels at all, just add the accesskeys.

>+++ b/suite/mailnews/mailWindowOverlay.js

>+    var mdnBarMsg = document.getElementById("mdnBarMessage");
>+    var barMsg;
>+    // If the return receipt doesn't go to the sender address, note that in the
>+    // notification.
>+    if (mdnAddr != fromAddr)
>+      barMsg = gMessengerBundle.getFormattedString("mdnBarMessageAddressDiffers",
>+                                                   [authorName, mdnAddr]);
>+    else
>+      barMsg = gMessengerBundle.getFormattedString("mdnBarMessageNormal",
>+                                                   [authorName]);
>+    mdnBarMsg.textContent = barMsg;
Nit: You could just inline mdnBarMsg.
r=me as long as you double check with Neil.
Attachment #635769 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 6

5 years ago
Created attachment 640066 [details] [diff] [review]
Patch v1.2 fix labels. r=IanN.

> I guess the meaning of these labels is not changing so probably no need to 
> change the entity name, but maybe Neil was suggesting you didn't need to change 
> the labels at all, just add the accesskeys.
Fixed.

> Nit: You could just inline mdnBarMsg.
Fixed.

> r=me as long as you double check with Neil.
Neil confirms that he doesn't want the label text changed, just add the accesskeys.

New patch attached.
Attachment #635769 - Attachment is obsolete: true
Attachment #640066 - Flags: superreview?(mnyromyr)
Attachment #640066 - Flags: review+

Comment 7

5 years ago
Comment on attachment 640066 [details] [diff] [review]
Patch v1.2 fix labels. r=IanN.

>+/*
>+ * This function handles all mdn response generation (ie, imap and pop).
>+ * For pop the msg uid can be 0 (ie, 1st msg in a local folder) so no
>+ * need to check uid here. No one seems to set mimeHeaders to null so
>+ * no need to check it either.
>+*/

I don't actually see the point here, especially if the last line is misaligned. :-P
Attachment #640066 - Flags: superreview?(mnyromyr) → superreview+
(Assignee)

Comment 8

5 years ago
> I don't actually see the point here, especially if the last line is misaligned. :-P
Alignment fixed on checkin.

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/06898b5bcf86
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.13
You need to log in before you can comment on or make changes to this bug.