305498 - UTF-8 e-mail encoding is not working

Status: RESOLVED FIXED

(Bugzilla :: Email Notifications, defect)

Description

[Cédric Caron]

When the UTF-8 option is enabled and the text of the e-mail contains 8bit 
caracters the e-mail is not corectly formated and the non ascii catacters are 
displayed as =XX whwre XX is an hex number.

this is a litle sample:

dans le reports "fournisseurs", coordonn=C3=A9es d=C3=A9taill=C3=A9es de l'= 
agent : les 
commentaires ne s'affichent pas sur le document d'impression.

Comment 1

[Cédric Caron]

Attachment: Patch v.1

the replace function only seems to replace an existing entry.

Comment 2

[:glob ✱]

Comment on attachment 193657 [details] [diff] [review]
Patch v.1

> the replace function only seems to replace an existing entry.

weird, the docs for replace say "If the tag does not appear in the header then
a new line will be appended to the header."

i'd be interested to know what version of Mail::Header you have.


assuming you've testing it, r=glob

Comment 3

[Cédric Caron]

Mail::Header 1.67

Comment 4

[Cédric Caron]

I tested the code but I don't know if this code will corectly replace an 
existing field (the perl eeprom conected to my brain is a beta version ;-)

Comment 5

[Frédéric Buclin]

(In reply to comment #4)
> I tested the code but I don't know if this code will corectly replace an 
> existing field (the perl eeprom conected to my brain is a beta version ;-)

Does this mean this patch needs more testing? Or can we go with the checkin?

Comment 6

[Cédric Caron]

(In reply to comment #5)
> Does this mean this patch needs more testing? Or can we go with the checkin?

I made aditional testing. everything seems ok. the function replace any 
existing field with the new one.

Comment 7

[Frédéric Buclin]

Checking in Bugzilla/BugMail.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/BugMail.pm,v  <--  BugMail.pm
new revision: 1.50; previous revision: 1.49
done