Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ITEM2.xxx properties - the phone number label is not correctly set #1028

Closed
j-ed opened this issue Mar 29, 2019 · 4 comments
Closed

ITEM2.xxx properties - the phone number label is not correctly set #1028

j-ed opened this issue Mar 29, 2019 · 4 comments
Labels
bug Something isn't working

Comments

@j-ed
Copy link

j-ed commented Mar 29, 2019

The field label of the following phone number is not correctly set:

Example 1:

4L06RU.TEL;TYPE="WORK,VOICE":+49 1234 556677
4L06RU.X-ABLABEL:VOICE

Example 2:

9JN9AI.TEL;TYPE=pref:+49 1234 556677
9JN9AI.X-ABLABEL:WORK\;VOICE\;PREF

Example 3:

FACL6F.TEL;TYPE=CELL,PREF:+49 1234 556677
FACL6F.X-ABLABEL:PREF\;CELL

Expected result:

The type WORK or CELL should be recognized correctly and the corresponding label Arbeit or Mobil (German language) should be displayed.

Current result:

Example 1:
20190329_nc-kontakte_wrong_phone_label-1
Example 2:
20190329_nc-kontakte_wrong_phone_label-2
Example 3:
20190329_nc-kontakte_wrong_phone_label-3

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #1026 (ITEM2.xxx properties - the url label is not correctly set), #1027 (ITEM2.xxx properties - the url label is not set at all), #360 (Incorrect phone number labels), #42 (Support X-ABLabel and ITEM2.xxx properties), and #224 (Phone numbers ignored when importing vCards).

@skjnldsv
Copy link
Member

4L06RU.X-ABLABEL:VOICE
9JN9AI.X-ABLABEL:WORK\;VOICE\;PREF
FACL6F.X-ABLABEL:PREF\;CELL

All of those are not valid.
X-ABLABEL is a text entry and supports text only. That is why the semi-colon are escaped with \;

How did you get those data? Is this coming from the nextcloud app?
How did those fields get created?

@skjnldsv skjnldsv added the needs info Not enough information provided label Mar 29, 2019
@j-ed
Copy link
Author

j-ed commented Mar 29, 2019

Unfortunately I cannot tell you how these entries have originally been created but I have in mind that the Calendar app had some trouble with incorrectly formatted entries in vcards in the past. Especially the escaped separator was an issue.
Based on the PRODID the records were synced from my Android phone some time ago:

PRODID:-//NTBAB//Android//ContactSync//9.0//164
REV:20171008T115403Z

PRODID:-//NTBAB//Android//ContactSync//9.0//164

PRODID:-//NTBAB//Android//ContactSync//9.0//166

@skjnldsv
Copy link
Member

skjnldsv commented Apr 1, 2019

@j-ed so unfortunately, I cannot really find any thing to do here. I guess we cannot really determine if the ABLABEL have been misused as a type or not :(

@skjnldsv skjnldsv closed this as completed Apr 1, 2019
@skjnldsv skjnldsv removed the needs info Not enough information provided label Apr 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants