TB 102 - After editing contacts from certain CardDAV address books (e.g. Google Contacts), field type labels are lost (e.g. "work", "home", "cell", "fax" etc. for phone numbers/addresses)
Categories
(Thunderbird :: Address Book, defect, P1)
Tracking
(thunderbird_esr102+ fixed, thunderbird104? fixed)
People
(Reporter: wrhenshaw99, Assigned: u695164)
References
(Blocks 2 open bugs)
Details
(Keywords: dataloss, Whiteboard: [TM:102.1.2])
Attachments
(6 files)
(deleted),
image/jpeg
|
Details | |
(deleted),
image/jpeg
|
Details | |
(deleted),
image/jpeg
|
Details | |
(deleted),
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
wsmwk
:
approval-comm-esr102+
|
Details |
(deleted),
image/jpeg
|
Details | |
(deleted),
image/jpeg
|
Details |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/103.0.0.0 Safari/537.36
Steps to reproduce:
Edit a contact in a CALDAV address book with multiple phone numbers
Correct labels are not shown in edit window, all are shown as NONE
Make a change to something other than phone numbers
Save contact
All labels for contact phone numbers are set to NONE
Actual results:
All previous labels for contact phone numbers are removed.
Expected results:
Labels for contact phone numbers should be shown in the edit window and changed on the contact only if modified in the edit window.
Reporter | ||
Comment 1•2 years ago
|
||
Reporter | ||
Comment 2•2 years ago
|
||
Updated•2 years ago
|
Comment 3•2 years ago
|
||
TB103.0b2 on Linux.
I can confirm this behavior. Existing labels for a contact are not visible in Edit mode. Labels in Edit mode are just 'None'.
The address book is for Google Contacts via CardDAV.
Reporter | ||
Comment 4•2 years ago
|
||
I didn't notice this before. Labels on addresses are also being lost. Rather than displaying in the edit window as "None", they are just blank.
Thanks for reporting the issue. In the underlying data from a another vCard source the vCard types could be set to uppercase. The display of a vCard supports this but not yet the edit of a contact.
(In reply to wrhenshaw99 from comment #4)
I didn't notice this before. Labels on addresses are also being lost. Rather than displaying in the edit window as "None", they are just blank.
This behavior is intentional and the label "None" is no type and displayed as a blank type.
Reporter | ||
Comment 7•2 years ago
|
||
(In reply to Nicolai Kasper from comment #5)
Thanks for reporting the issue. In the underlying data from a another vCard source the vCard types could be set to uppercase. The display of a vCard supports this but not yet the edit of a contact.
My CARDDAV address book is linked to Google. None of the labels are in uppercase in Google. They only display as upper case in TB after adding the CARDDAV address book and syncing.
(In reply to wrhenshaw99 from comment #4)
I didn't notice this before. Labels on addresses are also being lost. Rather than displaying in the edit window as "None", they are just blank.
This behavior is intentional and the label "None" is no type and displayed as a blank type.
I must not have explained it very well. In Google all my contact addresses are labeled as Work or Home. When viewed in TB in the CARDDAV address book the labels all appear as Work or Home. When I edit one of those contacts the labels do not appear in the edit window... they are all blank. Shouldn't the labels that exist on the entry show in the edit window?
Bill
Reporter | ||
Comment 8•2 years ago
|
||
Reporter | ||
Comment 9•2 years ago
|
||
Assignee | ||
Comment 10•2 years ago
|
||
I must not have explained it very well either. Thanks for your clarification!
(In reply to wrhenshaw99 from comment #7)
My CARDDAV address book is linked to Google. None of the labels are in uppercase in Google. They only display as upper case in TB after adding the CARDDAV address book and syncing.
There is a difference in the vCard data and how they are represented in a contact.
Although the labels are shown not in uppercase in your Google Contacts, the underlying vCard data which is not shown to you in a UI is in uppercase when sent from google.
I must not have explained it very well. In Google all my contact addresses are labeled as Work or Home. When viewed in TB in the CARDDAV address book the labels all appear as Work or Home. When I edit one of those contacts the labels do not appear in the edit window... they are all blank. Shouldn't the labels that exist on the entry show in the edit window?
Yes the labels that exist on the entry should be shown in the edit window.
Reporter | ||
Comment 11•2 years ago
|
||
I see, thanks for the clarification.
Bill
Updated•2 years ago
|
Comment 12•2 years ago
|
||
Afasics, losing field type labels of users' contacts constitutes dataloss -> P1/S2. We should land this asap.
Geoff (reviewer), could you land this now as Nicolai hasn't been around since your r+?
Thanks to Nicolai for fixing this bug!
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Comment 14•2 years ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/dc7f84412cbd
Support uppercase types for contact edits. r=darktrojan
Assignee | ||
Comment 15•2 years ago
|
||
Comment on attachment 9284342 [details]
Bug 1778043 - Support uppercase types for contact edits. r=darktrojan
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: In the edit view types from CardDAV sources with uppercase types are not able to be set or send.
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): Low-Medium. This could lead to further problems of Address Books who only support uppercase types (there are no known for me until now). E.g. google supports only uppercase types but is able to convert lowercase types to uppercase for their usage. This patch is depended on the patch from Bug 1776214.
Updated•2 years ago
|
Comment 16•2 years ago
|
||
Comment on attachment 9284342 [details]
Bug 1778043 - Support uppercase types for contact edits. r=darktrojan
[Triage Comment]
Approved for beta
Comment 17•2 years ago
|
||
bugherder uplift |
Thunderbird 104.0b3:
https://hg.mozilla.org/releases/comm-beta/rev/ae9b54312a58
Comment 18•2 years ago
|
||
Comment on attachment 9284342 [details]
Bug 1778043 - Support uppercase types for contact edits. r=darktrojan
[Triage Comment]
Approved for esr102
Comment 19•2 years ago
|
||
bugherder uplift |
Thunderbird 102.2.0:
https://hg.mozilla.org/releases/comm-esr102/rev/4bcc92f11f24
Description
•