The 'yomigana' (phonetic) input fields in vcard editing window are emptied when users are trying to save that.

RESOLVED FIXED in Thunderbird 3.0b3

Status

Thunderbird
Address Book
P3
normal
RESOLVED FIXED
10 years ago
8 years ago

People

(Reporter: Parag Nemade, Assigned: standard8)

Tracking

({dataloss, intl})

unspecified
Thunderbird 3.0b3
dataloss, intl
Bug Flags:
blocking-thunderbird3 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b1) Gecko/2007110903
Build Identifier: 

The 'yomigana' input fields in vcard editing window are emptied when users are
trying to save that.


Reproducible: Always

Steps to Reproduce:
0. Ensures Japanese support and thunderbird are installed.
1. Open virtual terminal in gnome: Alt-F2 > `gnome-terminal` > Enter.
2. Start thunderbird in locale Japanese: `LANG=ja_JP.UTF-8 thunderbird`.
3. Open vCard editing window: '編集' > 'アカウント設定' > (select an account, or
create a new one) > (click on account name) > 'vCardを編集'.
4. Input some English typing into two 'よみがな' input fields > 'OK'.
5. Reopen vCard editing window: 'vCardを編集'.

Actual Results:  
Data in 'yomigana' (よみがな) should be updated.

Expected Results:  
'yomigana' (よみがな) fields are empty.

Reproducible on Fedora 8 with thunderbird-2.0.0.6-6.fc8 rpm installed.

Comment 1

10 years ago
I can input よみがな just fine using the latest english linux trunk build.
(Assignee)

Comment 2

10 years ago
(In reply to comment #1)
> I can input よみがな just fine using the latest english linux trunk build.
> 

I've started having a look at this. 'yomigana' stands for phonetic. The phonetic address book fields can be enabled (in any language/locale) by setting the pref 'mail.addr_book.show_phonetic_fields'.

The problem is not in the address book, but in the account settings when you go to Account Settings for a specific account and "Edit Card" (next to attach my vcard to messages). Its there the phonetic fields don't get saved.

The problem is there we are saving the card as a vcard in prefs (not in an address book). I took a brief look at the code and the code doesn't save phonetic fields for vcards at the moment.

I still need to find out what the vcard spec says.

Comment 3

10 years ago
(In reply to comment #1)
> I can input よみがな just fine using the latest english linux trunk build.
> 

Do you mean inputting a string in those two よみがな fields and click on 'OK'?

Comment 4

10 years ago
My test was for name, display name. Didn't know about phonetic fields "yomigana"... so please just disregard comment 1.

Comment 5

10 years ago
Hi Magnus, for Fedora go to gnome-terminal:

$ LANG=ja_JP.UTF-8 thunderbird

should be able to get you there. - Rgds, Caius.
(Assignee)

Comment 6

9 years ago
The RFC (http://tools.ietf.org/html/rfc2426) for vcards, doesn't seem to support phonetic fields (or multiple names).

Maybe we should hide these fields from the vcard display?

Comment 7

9 years ago
Thanks Mark, I have two thoughts in my head:

- Is rfc2426 the latest and the most compatible specification available?
- Has this feature (fields) not been supported by other email client (e.g. OE, Euroda, etc) also?

Hope these questions help in clarification. Rgds Caius.
RFC 2426 is the latest actual spec (it's vCard 3.0).  Section 3.1.2 seems to suggest support for multiple names, but I just skimmed it, so I may have read it wrong.  

Additionally, there's an IETF working group working on updating that spec, so there may be newer drafts.  It would be interesting to know if they exist and say anything that would address this issue differently than 2426.

Support in other clients is a relevant question, as is support in Tb2.  I know that Tb 2 doesn't support all vCard 3.0 features, but I'm not sure if we support just vCard 2.0 or some superset of that.

Comment 9

9 years ago
Hi, if it had been confirmed that the field is not usable currently, IMHO we could hide it until all the supporting codes at the background are done. Rgds.
Keywords: intl
Confirming, as per comment 6.  Since this is dataloss bug, adding that keyword, and since there appears to be a trivial way to avoid that dataloss (hide the field and don't accept the data at all), I think this should block Thunderbird 3.  Marking as such and giving to Mark.
Assignee: nobody → bugzilla
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-thunderbird3+
Keywords: dataloss
(Assignee)

Updated

8 years ago
OS: Linux → All
Priority: -- → P3
Hardware: x86 → All
Summary: The 'yomigana' input fields in vcard editing window are emptied when users are trying to save that. → The 'yomigana' (phonetic) input fields in vcard editing window are emptied when users are trying to save that.
Target Milestone: --- → Thunderbird 3.0b3
(Assignee)

Comment 11

8 years ago
I've done some research, and I can't see any room for phonetic fields in the vcard standard at the moment. We'd certainly need to update our vcard support anyway. So I think it is best to disable those fields.
(Assignee)

Comment 12

8 years ago
Created attachment 363882 [details] [diff] [review]
Hide the phonetic fields as well for "vcards"

This hides the phonetic fields as well when we bring up the vcard for editing in the account manager (to test set mail.addr_book.show_phonetic_fields to "true").
Attachment #363882 - Flags: superreview?(neil)
Attachment #363882 - Flags: review?(bienvenu)
(Assignee)

Updated

8 years ago
Status: NEW → ASSIGNED
Whiteboard: [has patch][needs review neil,bienvenu]

Updated

8 years ago
Attachment #363882 - Flags: superreview?(neil) → superreview+

Comment 13

8 years ago
Comment on attachment 363882 [details] [diff] [review]
Hide the phonetic fields as well for "vcards"

won't this cause a redeclaration of "var i" warning with js warnings turned on? r=me, with that addressed.
Attachment #363882 - Flags: review?(bienvenu) → review+
(Assignee)

Comment 14

8 years ago
Checked in with David's comment fixed: http://hg.mozilla.org/comm-central/rev/c066ee5931e3
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][needs review neil,bienvenu]

Comment 15

8 years ago
Use |for (let i ...)| to limit the scope of the "i" variable to the for loop. This way you can safely reuse "i" in different block scopes.
You need to log in before you can comment on or make changes to this bug.