1242676 - "Reply with template" filter should add References: and In-Reply-To: headers to the reply mail.

Status: RESOLVED FIXED

(MailNews Core :: Filters, defect)

Description

[Magnus Melin [:mkmelin]]

Attachment: bugXXX_auto-reply-refs.patch

"Reply with template" filter should add References: and In-Reply-To: headers to the reply mail so that you can see what mail it was a reply to, and threading will work.

Comment 1

[Kent James (:rkent)]

Comment on attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

Review of attachment 8711819 [details] [diff] [review]:
-----------------------------------------------------------------

OK looks good with one nit.

::: mailnews/compose/src/nsMsgCompose.cpp
@@ +1800,5 @@
>  
>    nsCOMPtr<nsIPrefBranch> prefs (do_GetService(NS_PREFSERVICE_CONTRACTID, &rv));
>    NS_ENSURE_SUCCESS(rv, rv);
>  
>    // If we are forwarding inline, mime did already setup the compose fields therefore we should stop now

Can you correct this comment since you have expanded the meaning of this section?

Comment 2

[Magnus Melin [:mkmelin]]

https://hg.mozilla.org/comm-central/rev/63ee59a8e786 -> FIXED

Comment 3

[Magnus Melin [:mkmelin]]

Comment on attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

[Approval Request Comment]
User impact if declined: auto-replies not threaded
Testing completed (on c-c, etc.): landed on trunk with test
Risk to taking this patch (and alternatives if risky): not risky, should land after trunk baking

Comment 4

[Jorg K (CEST = GMT+2)]

Aurora (TB 46):
https://hg.mozilla.org/releases/comm-aurora/rev/8057891037ba

Comment 5

[Kent James (:rkent)]

Comment on attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

http://hg.mozilla.org/releases/comm-esr45/rev/024ed64816aa