Closed Bug 77967 Opened 23 years ago Closed 23 years ago

Non-ASCII account name is displayed garbled.

Categories

(MailNews Core :: Internationalization, defect, P2)

Product:
MailNews Core
Component:
Internationalization
Platform:
x86
All
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.1

People

(Reporter: ji, Assigned: racham)

References

Details

(Keywords: intl, regression, Whiteboard: [nsbeta1+]Have Fix)

Attachments

(2 files)

A screenshot.
140.13 KB, image/jpeg
Details
Possible patch for regression
898 bytes, patch
Details | Diff | Splinter Review 
****Observed with win32 04/27 trunk build*****

The non-ASCII account name is displayed garbled. This problem doesn't exist on
2001-04-26-06 trunk build.

Steps to reproduce:
1. Launch Mail.
2. Select Edit | Mail/News Account Settings.
3. Click on New Account.
4. Fill up all the info and process to the last screen of the account setup wizard.
5. Enter non-ASCII account name, click on Next to go to next screen and click on
Finish to complete the account setup.
On 3-pane, the account name is displayed garbled.
Attached image A screenshot. — 

    
  
Keywords: intl, 
          
            regression

    
    

    
  
How about account names in the account dialog?


    
    

    
  
Reassign to putterman, regression starting from today's build.
Assignee: nhotta → putterman

    
    

    
  
It's fine on the dialog. Once you complete the wizard, and go back to it again, 
it's garbled. 

    
    

    
  
reassigning to racham.
Assignee: putterman → racham

    
    

    
  
Nominating for nsbeta1. It also garbled the From line display on mail compose
window.
Keywords: nsbeta1

    
    

    
  
*** Bug 76860 has been marked as a duplicate of this bug. ***

    
    

    
  
marking nsbeta1+
Priority: -- → P2
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.1

    
    

    
  
*** Bug 78252 has been marked as a duplicate of this bug. ***

    
    

    
  
Please correct the summary of this bug, spelling "acount" correctly; as it is
now it can't be found through Bugzilla queries.

    
  
Summary: Non-ASCII acount name is displayed garbled. → Non-ASCII account name is displayed garbled.

    
    

    
  

*** This bug has been marked as a duplicate of 77990 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE

    
    

    
  
This one has right component, more keywords and status info, and this is filed 
first.
I'd like to keep this bug open. If you think this one needs more info, please 
add your ccomments. Thanks.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---

    
    

    
  
*** Bug 77990 has been marked as a duplicate of this bug. ***

    
    

    
  
As I commented in my version of this bug (76860) it is saving the UTF-8 encoded 
version of the character, instead of the character itself. Which would be fine 
if it decoded from UTF-8 when it read it back in...

    
    

    
  
I have a workaround.

1. Search on your harddrive for prefs.js.
2. Open your prefs.js file with a text-editor
3. Change it like this:

user_pref("mail.identity.id2.fullName", "HÃ¥kan Waara");
to
user_pref("mail.identity.id2.fullName", "Håkan Waara");

and do that for all your identities (id1, id2, id3 etc.)

So this seems to be a one-time thing. But it was really painful till I found the
workaround.


    
    

    
  
The display problem also shows on account central and the pulldown menu by
clicking the small arrow besides Get Msg button.

    
    

    
  
yep. As Eric mentioned this is a conversion problem and is a regression. It is
going to be garbled all the places which uses the name. I will debug to findout
the source of the problem.

    
    

    
  
See if this patch helps. I don't know if this is it or not, but the timing is 
about right for the prefs landing.


    
    

    
  
I have been suspecting recent prefs changes only. Thanks for the patch. Will
apply and see if that fixes the problem. thanks.

bhuvan

    
    

    
  
Thank timeless, he was the one wise enough to cc me. I just made (hopefully) the 
right connection.


    
    

    
  
looks good, yeah, i just occurred to me because of one of the other bugs you 
fixed...  now if only i can get you to help me w/ my pref refactor bug.

    
  
Whiteboard: [nsbeta1+] → [nsbeta1+]Have Fix

    
    

    
  
Naoki, 

The patch does the right thing. We do need to make conversions from utf8-to-ucs2
here. Please review the patch. thanks.

Will request Seth for super-review.

    
    

    
  
r=nhotta

In PrefBranch.cpp, I see one more NS_ConvertASCIItoUCS2 in NotifyObserver. Is
that for pref name or pref value?


    
    

    
  
That one is ok. It's for the preference name. The observer doesn't pass the
value, only the name of the changed preference.

    
    

    
  
Any progress on this? This is killing me!
OS: Windows 95 → All

    
    

    
  
Sent a sr= request to Seth via email now. Will get this in soon.

    
    

    
  
Fixed. thanks timeless and all.
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED

    
    

    
  
Verified with win32, mac and linux 05/07 mtrunk builds. It's fixed.
Status: RESOLVED → VERIFIED

    
    

    
  
*** Bug 79089 has been marked as a duplicate of this bug. ***
Product: MailNews → Core
Product: Core → MailNews Core

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: