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

Phone field missing on many contacts #3440

Closed
meichthys opened this issue Jun 5, 2023 · 14 comments · Fixed by #3644
Closed

Phone field missing on many contacts #3440

meichthys opened this issue Jun 5, 2023 · 14 comments · Fixed by #3644
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working feature: contacts regression Regression of a previous working feature

Comments

@meichthys
Copy link
Contributor

Describe the bug

Recently I've noticed that many of the contacts (when viewed from the nextcloud web ui) are missing phone numbers.
The numbers are however synced to my other devices via cardDav.

Steps to reproduce

  1. Go to Contacts app via web ui
  2. View some contacts that you know should have phone numbers

Expected behavior

All contacts that have phone numbers should show the phone numbers.

Actual behavior

Some contacts that have phone numbers when synced with a carddav client do not show the phone number.

Contact version

5.3.0-rc.3

Operating system

Ubuntu

PHP engine version

PHP 8.1

Web server

Apache (supported)

Database

PostgreSQL

Additional info

I'm not sure if this is also happening for other fields. I've only noticed it for the phone field.

@thanoskarras
Copy link

Just installed contacts version 5.3.0 on Nextcloud 25.0.7 (opensuse leap, apache, mysql, php 8.0) and have the same problem.

@Hainer1
Copy link

Hainer1 commented Jun 28, 2023

Same here NC26.0.2, Contact 5.3.2, apache, mariaDB, ubuntu VM, php8.1

@meichthys
Copy link
Contributor Author

^ Also seeing it on NC 27

@blausand
Copy link

blausand commented Jul 1, 2023

Also seeing it around the globe, folks. This is not a small issue. Almost killed some contacts b/c the web interface made me believe they contain no info but the name.

@cm-schl
Copy link

cm-schl commented Jul 3, 2023

Could someone give some example how this looks like and what the exported .vcf of such a contact contains?

@meichthys
Copy link
Contributor Author

meichthys commented Jul 3, 2023

Screenshot_20230703-063455.png

VCF minus photo blob:

'''
BEGIN:VCARD
VERSION:3.0
PRODID;VALUE=TEXT:-//Apple Inc.//iOS 15.2.1//EN
N:;;;;
FN:Alphanetunlimited
ORG:Alphanetunlimited;
ITEM1.TEL;TYPE=pref;VALUE=UNKNOWN:+18558693466
NOTE:They offer unlimited data hotspot plans on their website for about $80
a month.\n\nLater switched away from them in favor of Verizon 5G Home inte
rnet
X-ABSHOWAS:COMPANY
REV;VALUE=DATE-AND-OR-TIME:20220922T173041Z
UID:F994676C-4C5C-4121-9E10-1F53B22E63F5
PHOTO;ENCODING=b;TYPE=jpeg;VALUE=BINARY:
'''

One thing I noticed is the 'prodid' is iOS bit I am now using android.

@cm-schl
Copy link

cm-schl commented Jul 4, 2023

Thanks for the example!
Ok, so this seams to be connected with the usage of the custom properties like in #3447 - or have you the same problem if the phone number is saved with one of the typical categories like private, work etc?

@ChristophWurst ChristophWurst added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage labels Jul 4, 2023
@ChristophWurst ChristophWurst moved this to 📄 To do (~10 entries) in 💌 📅 👥 Groupware team Jul 4, 2023
@ChristophWurst ChristophWurst added the regression Regression of a previous working feature label Jul 4, 2023
@meichthys
Copy link
Contributor Author

@cm-schl Yes, same issue with typical categories.

@cm-schl
Copy link

cm-schl commented Jul 6, 2023

Just to give some more info - I think I can't reproduce this on v.25.0.8 with Contacts v5.3.2:
grafik

BEGIN:VCARD
VERSION:3.0
PRODID:-//Sabre//Sabre VObject 4.4.2//EN
UID:4a8f35b8-0106-4552-afbb-4cde0cfb6b39
FN:contact new
N:new;contact;;;
TEL;TYPE=HOME:123 456 789
REV:20230706T065252Z
END:VCARD

@karthikiyengar
Copy link

BEGIN:VCARD
VERSION:3.0
PRODID;VALUE=TEXT:-//Apple Inc.//iOS 16.6//EN
N:;Fake Name;;;
FN:Fake Name
ITEM1.TEL;TYPE=pref;VALUE=UNKNOWN:+49 1111 11111
REV;VALUE=DATE-TIME:20230817T172123Z
CATEGORIES:myContacts
UID:bc8bcbbd-6199-4344-8b99-0c89840d8170
END:VCARD

Can't see the field. 3p software reports it correctly under label "Telephone".

@nyquist2205
Copy link

Same for me, did some experiments:

Existing old contact with "mobile" number did not show the number in NC, but in Android:

BEGIN:VCARD
VERSION:3.0
FN:Tobias %%REMOVED%%
N:Tobias;%%REMOVED%%;;;
ITEM1.TEL:+49 176 %%REMOVED%%
ITEM1.X-ABLABEL:Mobil
CATEGORIES:myContacts
UID:b6aa81f0-0bb8-4870-a424-327a441884ce
REV:20200210T203727Z
END:VCARD

Changed number to "private" on Android phone and synced to NC, now numbers shows in NC Contacts:

BEGIN:VCARD
VERSION:3.0
PRODID:-//Sabre//Sabre VObject 4.4.2//EN
UID:b6aa81f0-0bb8-4870-a424-327a441884ce
FN:Tobias %%REMOVED%%
N:Tobias;%%REMOVED%%;;;
CATEGORIES:myContacts
TEL;TYPE=home:+49 176 %%REMOVED%%
REV:20230822T141830Z
END:VCARD

Created a new contact in NC Contacts, this one also has the TEL;TYPE= syntax and therefore shows the number:

BEGIN:VCARD
VERSION:3.0
PRODID:-//Sabre//Sabre VObject 4.4.2//EN
UID:784b13cf-989d-4a65-950d-e349050f7e4b
FN:Mr Nobody
N:Nobody;Mr;;;
TEL;TYPE=cell:123456789
REV:20230822T142029Z
END:VCARD

I do not know where the old contact was created, but I am pretty sure it was either a Android Smartphone or NC contacts.

  • Nextcloud Contacts 5.3.0
  • Nextcloud 26.0.0
  • PHP 8.1.11

@pov91
Copy link

pov91 commented Sep 17, 2023

I have the same problem on an instance with Nextcloud 25. It is not possible to export the contacts so that a new instance with version 27 can read it. Some error messages occur, so that the contacts are not imported. Other contacts are imported but do not indicate all fields again, which occurs primarily for telephone numbers, but also for address fields. However, it does not always occur in all contacts. but with many. I have the impression it occurs especially if you add many additional fields to the contact. All fields exist in the exported vcf files. Synchonized on the phone, everything looks normal, but so you cannot change the contacts in the cloud or export/import them to another instance. This means that the contact app is not reliable.

@LittleAlf
Copy link

LittleAlf commented Sep 24, 2023

Unfortunately, I have to confirm this as a regression in version 5.4.2 of the Contacts App as well:

  • Nextcloud 26.0.7
  • Nextcloud Contacts 5.4.2
  • PHP 8.1.23

Any phone number with a custom label does not show up in the Contacts UI anymore!

To verify it, I did the following:

  • Added phone number with a custom label under Android 13 in the Google contacts up.
  • After synching this entry with my Nextcloud server the new entries did not show up in the NC Contacts UI.
  • In my Android phone I changed the custom labels of the numbers to standard labels 'Mobile'.
  • After doing so, the phone numbers showed up in the NC Contacts UI.
  • Now I changed back to my Android phone and changed the labels of the phone numbers back to custom labels.
  • Again, these numbers does not show up in the NC Contacts UI!
  • I can confirm, that the VCARD entry from NC contains all phone numbers including the new ones and including the correct custom labels!

I was used to accept that you cannot enter custom labels in the NC Contacts UI directly. But this is now at least the second time that this regression is happening in the NC Contacts App!

This casts a serious doubt on the usability of this App.

@LittleAlf
Copy link

@cm-schl:

You do not see it, because you use a standard label. Try changing the label of the mentioned phone number to a custom label ('benutzerdefiniert') on your Android or iOS phone. Then you will see, that the phone number is missing in the NC Contacts UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working feature: contacts regression Regression of a previous working feature
Projects
Development

Successfully merging a pull request may close this issue.

10 participants