135538 - No transliteration fallback for message headers and plain text body.

Status: VERIFIED FIXED

(MailNews Core :: Internationalization, defect)

Description

[nhottanscp]

http://www.mozilla.org/quality/intl/tests/data/mailsmoketest.zip

1) Reply to the last message of the intl smoketest data.
2) Two alerts will come up hit OK for both.
3) Received the mail, Euro sign in the header and the body turn to a question mark.

Expected result: the Euro sign is transliterated to "EUR".

Comment 1

[nhottanscp]

This is caused by setting an wrong option to nsISaveAsCharset caused by my check
in for bug 125077.

Nominating for nsbeta1, we need transliteration for message headers and plain
text body.

Comment 2

[nhottanscp]

Attachment: Use the correct attribute for transliteration. 'transliterate' is an attribute of nsIEntityConverter, so attr_EntityNone will skip transliteration at all.

Comment 3

[Frank Tang]

Impact Platform: ALL
Impact language users: ALL (people who will send out euro sign ) 560M 100% of
internet users
Probability of hitting the problem: HIGH, when users send out euro sign in mail
subject or plain text mail body.
Severity if hit the problem in the worst case: see ? mark instead EUR for euro
sign and make cause bad interpretation of fincial decision
Risk of the fix: LOW, only impact sending mail of message header and plain text
body. the patch basically back up what we used to use in n6.2.
Potential benefit: Fix other characters like TM sign will be send out as "TM"
also smart quote will convert right 

Comment 4

[Jean-Francois Ducarroz]

does this fix has any effect on bug 109342?

Comment 5

[nhottanscp]

>does this fix has any effect on bug 109342?
No, that bug is about changing a charset to something else for Euro sign (i.e.
not a workaround of this bug).
This bug, the problem is that it is sent as a question mark. And this bug is not
for Euro only, other symbols like trade mark, smart quotes also turn to question
marks.

Comment 6

[Frank Tang]

this is a adt2 bug

Comment 7

[Jean-Francois Ducarroz]

Comment on attachment 77737 [details] [diff] [review]
Use the correct attribute for transliteration. 'transliterate' is an attribute of nsIEntityConverter, so attr_EntityNone will skip transliteration at all.

R=ducarroz

Comment 8

[(not reading, please use [e-mail] instead)]

Comment on attachment 77737 [details] [diff] [review]
Use the correct attribute for transliteration. 'transliterate' is an attribute of nsIEntityConverter, so attr_EntityNone will skip transliteration at all.

sr=sspitzer

Comment 9

[Frank Tang]

adt1.0.0
low risk fix for high sensentive issue. 

Comment 10

[Asa Dotzler [:asa]]

Comment on attachment 77737 [details] [diff] [review]
Use the correct attribute for transliteration. 'transliterate' is an attribute of nsIEntityConverter, so attr_EntityNone will skip transliteration at all.

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 11

[Frank Tang]

the suggest change was in the tree before feb 19, 2002

Comment 12

[Jaime Rodriguez, Jr.]

adt1.0.0+ (on ADT's behalf) approval for checkin to 1.0.

Comment 13

[nhottanscp]

checked in to the trunk

Comment 14

[ji]

Verified with 06/18 trunk build.