212678 - New variable in "newchangedmail": %re%

Status: RESOLVED WONTFIX

(Bugzilla :: Email Notifications, enhancement, P4)

Description

[SineSwiper]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0)
Build Identifier: 

I wanted to add a variable that would be similar to %neworchanged%, but it 
would report "Re:" if it's NOT new and nothing if it is new.  This way, the e-
mail messages would show up correctly in a mailing list or message thread.

The amount of code require to make this change is maybe 5 lines, and I could do 
it easily, but I just wanted some input on whether it was better to not add 
extra **** into the scripts, or go ahead and add the variable to the 
processmail and defparams.pl scripts.

Reproducible: Always

Steps to Reproduce:

Comment 1

[J. Paul Reed [:preed]]

I wouldn't bother worrying about this until bug 84876 lands.

When it does, the proper way to do this is to handle it in the email template;
pass along the current "isnew" flag, and let the template decide what to do
about it.

Having said that, I don't think a new Param is necessarily warranted (or even
wanted); there was some discussion on IRC about combining a "enable mail
threading" param that would turn on a feature like this as well as whatever bug
31314 turns out to be when it goes in.

Comment 2

[SineSwiper]

Attachment: Patch fix

Attached the patch if anybody wants it...

Bug 31314 looks like something I could fix in 5 minutes (I'm familar with
In-Reply-To very well...), but I'll have to take a look at it.	(Like you
implied, bug subjects can be deceiving.)

Comment 3

[J. Paul Reed [:preed]]

Kick all my Bugzilla bugs back to the component default owner.

Comment 4

[Frédéric Buclin]

The newchangedmail parameter is gone.