Non-ascii name is garbled in the Reply-to field on the Account Manager mail panel

RESOLVED FIXED in Thunderbird 10.0

Status

MailNews Core
Internationalization
RESOLVED FIXED
16 years ago
6 years ago

People

(Reporter: marina, Assigned: m_kato)

Tracking

({intl})

Trunk
Thunderbird 10.0

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [ish2+])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

16 years ago
Stesp to repro:
- open Account Wizard, Edit|Account setting;
- enter a non-ascii name into the Replyt- field ( cyrillic name
<marina@netscape.com>
 - close Mail, reopen;
//note: the non-ascii name is garbled on the Reply-to field

Comment 1

16 years ago
There is a regression bug 77967 today, is this also a regression?

Comment 2

16 years ago
After I enter some Japanese characters on Reply-to Address field, save it and go 
back to the same screen again, the Ja chars are saved as ASCII characters. For 
example, hiragana a is save as ascii "B". It happens on 04/26 build too. 
It doesn't look like a regression. Is that we can only be allowed to enter ascii 
in Reply-to Address field?
QA Contact: ji → marina
(Reporter)

Comment 3

16 years ago
i don't think it is a regression. I remember that we could not enter
"nonasciiname<marina@netscape.com>" before.
(Reporter)

Comment 4

16 years ago
in 2001-04-18 build the non-ascii string entered in the Reply-to field on thhe
account wizard is not remembred after you close/reopen Mail, it is just lost. So
in a way it is not a regression but a progress :-)

Comment 5

16 years ago
Reassign to putterman.
Assignee: nhotta → putterman
Keywords: intl

Comment 6

16 years ago
reassigning to racham.
Assignee: putterman → racham

Comment 7

16 years ago
Same problem with BCC antoher address field on copy and folder settings window.

Comment 8

16 years ago
ji, 

Is this confirmed with today's build. Lately, fix to the similar problem has
been checked in. It was fixed at prefs level and that should ideally take care
of these kind of problems too. Please let me know. thanks.

bhuvan
(Reporter)

Comment 9

16 years ago
can not reproduce in 2001-05-09 ; non-ascii chars are saved correctly

Comment 10

16 years ago
Just checked Mac 2001-05-10-04 mtrunk build. It has no problem with Latin-1
accented characters, but Japanese characters are garbled. And this happens both
on Reply-to and BCC to other address field. I'll check win32 and linux build
when testable builds are available.
(Reporter)

Comment 11

16 years ago
with Win 2001-05-09-04 build JA chars can not be saved correctly in any field
(Reporter)

Comment 12

15 years ago
this problem still exists in mozilla builds , charset that is non-latin1 can not
be correctly saved on the Account manager. For .ex would i use a cyrillic name
for the Reply-to field on english system it would be saved incorrectly. If the
user doesn't know that because the name displays correctly while being typed he
would not understand why he sees garbage in the composition window on the
Reply-to field (and CC and Bcc). This type of mail would not be sent in current
builds because the recipient is not recognized a valid.This is a serious
problem, and needs to be fixed. Nominating 
Keywords: nsbeta1
(Reporter)

Comment 13

15 years ago
Created attachment 72814 [details]
here is a screen shot to illustrate the account display problem (cyrillic charset is used)
(Reporter)

Comment 14

15 years ago
Created attachment 72816 [details]
here is an error you'll get in case you will send the mail with the incorrect display of the recipient name

Comment 15

15 years ago
The Reply-to field is on AccountManager's main panel. Changing the summary to
reflect the same.
Summary: Non-ascii name is garbled in the Reply-to field on the Account Wizard → Non-ascii name is garbled in the Reply-to field on the Account Manager mail panel

Comment 16

15 years ago
Discussed in Mail News bug meeting with Engineering, QA, Mktng and PjM. 
Decision was to plus this bug.
Keywords: nsbeta1 → nsbeta1+
Target Milestone: --- → mozilla1.0

Comment 17

15 years ago
I'm going to minus this because if the user just types their email address in
then it will work, ie <marina@netscape.com>

Scott
Keywords: nsbeta1+ → nsbeta1-
Target Milestone: mozilla1.0 → Future

Updated

15 years ago
Blocks: 157673

Comment 18

15 years ago
nominate for nsbeta1
Keywords: nsbeta1- → nsbeta1
(Reporter)

Comment 19

15 years ago
 The account wizard has three fields: "Account name" (this one accepts non-ascii
names), "Email address" ( this one is explicitly saying that only emails are
expected to be entered ) but in the third field we say :"Reply-to Address" and
the user could try to enter the pretty name here like "yourname@xyz.com". This
field doesn't accept pretty names , the non-ascii string enter here will be
saved as gibberish. So we have a choice to either change the code (in many
places) to be able to save the string in unicode (probably too late for this
release) or maybe change the wording for Reply-to address to make it clear that
ONLY email address (no pretty name) is going to be collected. Jennifer, what's
your take on this? Cc'ing Jennifer and Scott.  

Comment 20

15 years ago
>or maybe change the wording for Reply-to address to make it clear that
>ONLY email address (no pretty name) is going to be collected.

"Reply-to Email Address:" (sorta long) or "Reply-to Email:" (not sure about 
this one)?

Comment 21

15 years ago
I prefer "Reply-to Email Address:" if space allows. This is a more descriptive
label than "Reply-to Email:" . Including the phrase "Email Address" is the
clearest way to describe what we're asking for.
(Reporter)

Comment 22

15 years ago
The "Reply-to Email Address" sounds good to me... one question though: do we
want to permanently exclude the use of pretty name for the Reply? The "From"
field is made of a combination of two strings: "Your name" and " Email address".
The fact that "Your name" field accepts non-ascii chars allows us to enter a
Name in japanese, chinese etc... The Reply then will only include the email
address though the user could have a different Display name associated with a
reply address... I am thinking how consistent is this design..?

Comment 23

15 years ago
OK, i think i understand better now. "PrettyName <email@server.com>" works with 
ascii chars but "PrettyName" gets messed up when using non-ascii characters. I 
would think the best solution would be to allow non-ascii characters for 
PrettyName.

Updated

15 years ago
Whiteboard: [ish2+]
Greg, another bug for your consideration.

Comment 25

15 years ago
Mail triage team: nsbeta1-
Keywords: nsbeta1-

Updated

15 years ago
Keywords: nsbeta1
mass re-assign.
Assignee: racham → sspitzer
Product: MailNews → Core

Comment 27

13 years ago
Comment on attachment 72816 [details]
here is an error you'll get in case you will send the mail with the incorrect display of the recipient name

This attachment is not useful because the error message being shown is about
the lack of a To: address in the email; it's unrelated to the Reply-To problem.

It is true that Unicode characters (beyond 0xFF) 
are stored oddly, as shown in attachment 72814 [details].  In Mozilla 1.8a6, an illegal
Reply-To header is simply left off the new message; but in TB 1.0, such a
header is truncated at the first illegal character -- see bug 208577.
Attachment #72816 - Attachment is obsolete: true
(In reply to comment #27)
> In Mozilla 1.8a6, an illegal
> Reply-To header is simply left off the new message; but in TB 1.0, such a
> header is truncated at the first illegal character -- see bug 208577.

Being a little pedantic and pointing to comment #23 and the cone I made in bug 208577 (number 6), I would like to stress that this is not limited to illegal characters (such as those in the email address) but also to any non-ASCII character (such as those one might use when preceding the address with a PrettyName).

Updated

10 years ago
Assignee: sspitzer → smontagu
OS: Windows NT → All
QA Contact: marina → mailnews.i18n
Hardware: PC → All

Updated

10 years ago
Duplicate of this bug: 376444
Product: Core → MailNews Core
(Assignee)

Comment 30

6 years ago
Created attachment 571270 [details] [diff] [review]
fix
Assignee: smontagu → m_kato
Attachment #571270 - Flags: review?(dbienvenu)
(Assignee)

Comment 31

6 years ago
no easy way to write unit test because this is attribute.
(Assignee)

Updated

6 years ago
Duplicate of this bug: 632370

Comment 33

6 years ago
Comment on attachment 571270 [details] [diff] [review]
fix

thx for the patch
Attachment #571270 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 34

6 years ago
http://hg.mozilla.org/comm-central/rev/8bba60eda697
Target Milestone: Future → Thunderbird 10.0
(Assignee)

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.