Closed Bug 747743 Opened 12 years ago Closed 12 years ago

Mojibake in context menus on the chat conversation

Categories

(Thunderbird :: Instant Messaging, defect)

13 Branch
x86
Windows 7
defect
Not set
normal

Tracking

(thunderbird13 fixed)

RESOLVED FIXED
Thunderbird 14.0
Tracking Status
thunderbird13 --- fixed

People

(Reporter: marsf, Assigned: florian)

Details

(Keywords: jp-critical)

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; ja; rv:1.9.2.28) Gecko/20120306 Firefox/3.6.28
Build ID: 20120306064154

Steps to reproduce:

1. Open chat tab.
2. Connect twitter account.
3. Right click individual tweet to show its context menu.


Actual results:

Only the menus of twitter actions (retweet, reply, follow, ... ) has mojibake.
It seems that the localized UTF-8 strings are not properly handled.
Keywords: jp-critical
I'm a bit confused by this bug report, as:
- I've tried to replace the en-US strings with strings containing UTF8 characters in my debug build, and these strings were displayed correctly.
- I've tried to find the ja translation of these strings, but http://hg.mozilla.org/l10n-central/ja/file/tip/chat/twitter.properties doesn't exist (I've checked the URL with the French locale, and http://hg.mozilla.org/l10n-central/fr/file/tip/chat/twitter.properties#l20 does exist).
I've tested on Aurora build. (We haven't updated l10n-central.)
Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20120422 Thunderbird/13.0a2

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ja/file/47eeb04b0b3c/chat/twitter.properties

Tooltips and options are displayed correctly.
Attached patch FixSplinter Review
I couldn't reproduce with a string containing French non ASCII characters, but I can reproduce using the strings in the ja localized file.

It's caused by a trivial type error in the idl interface.

Thanks for reporting this!
Assignee: nobody → florian
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #617456 - Flags: review?(clokep)
Attachment #617456 - Flags: approval-comm-aurora?
Comment on attachment 617456 [details] [diff] [review]
Fix

This change looks fine, thanks for fixing it Florian.
Attachment #617456 - Flags: review?(clokep) → review+
http://hg.mozilla.org/comm-central/rev/9869097a829e
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0
Attachment #617456 - Flags: approval-comm-aurora? → approval-comm-aurora+
Drive-by comment here - should the UUID for that IDL have been changed?
(In reply to Mike Conley (:mconley) from comment #6)
> Drive-by comment here - should the UUID for that IDL have been changed?

I don't think Thunderbird cares since we haven't turned this on, but Instantbird might...
(In reply to David :Bienvenu from comment #7)
> (In reply to Mike Conley (:mconley) from comment #6)
> > Drive-by comment here - should the UUID for that IDL have been changed?
> 
> I don't think Thunderbird cares since we haven't turned this on, but
> Instantbird might...

I've never applied that rule for Instantbird as I'm not aware of any binary add-on using our xpcom interfaces.
Looks like this was pushed to aurora earlier today:

http://hg.mozilla.org/releases/comm-aurora/rev/88ca33153d6a
You need to log in before you can comment on or make changes to this bug.