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

Missing UID in contact data #175

Closed
milgner opened this issue Mar 21, 2017 · 8 comments
Closed

Missing UID in contact data #175

milgner opened this issue Mar 21, 2017 · 8 comments
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working medium Medium priority

Comments

@milgner
Copy link

milgner commented Mar 21, 2017

Steps to reproduce

  1. Upgraded to Nextcloud 11, contacts app 1.5.3
  2. Open contacts app
  3. Javascript console shows error:
contact_model.js:30 Uncaught (in promise) TypeError: Cannot read property 'value' of undefined
    at Contact.uid (contact_model.js:30)
    at contact_service.js:37
  1. Spinning wheel, no contacts shown.

Expected behaviour

My contacts should be displayed :)

Actual behaviour

It looks like the app is trying to access the UID property and there isn't any. I debugged into the code in question which says

return model.getProperty('uid').value;

Accessing other data like email() works just fine. So now I'm wondering why it suddenly complains about a missing UID and whether there might have been a migration that was supposed to run but didn't during upgrade?
Is there any way to add new UID to those contacts missing them?

I know my way around MySQL so if you have an idea what should go where, I'll gladly write a migration script myself 😉

@skjnldsv
Copy link
Member

I don't see why a contact won't have any uid set!
importing create uid for each before storing it to the db and the dav server will refuse any vcard without uid. Did you edit your database?

@skjnldsv skjnldsv added the needs info Not enough information provided label Jul 26, 2017
@milgner
Copy link
Author

milgner commented Jul 26, 2017

Yes, it's quite baffling. I did not edit the database. The CardDAV sync on my phone works just fine, too.

@skjnldsv
Copy link
Member

Hum, could come from an external sync indeed. We need to prevent this from happening.

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of bug Something isn't working medium Medium priority and removed needs info Not enough information provided labels Jul 26, 2017
@irgendwie
Copy link
Member

Aren't cards without an UID rejected by the carddav backend in nextcloud/server?

@skjnldsv
Copy link
Member

Apparently not! 🤔

@skjnldsv skjnldsv marked this as a duplicate of #145 Jul 26, 2017
@skjnldsv
Copy link
Member

UIDs are automatically created on import and create in the contacts app. This needs to be created on https://github.com/nextcloud/server

@wirepatch
Copy link

wirepatch commented Sep 22, 2017

Since my issue may not be uid related I've created a new issue at #358

@skjnldsv
Copy link
Member

Look at your nextcloud.log file into the data folder! :)

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 medium Medium priority
Projects
None yet
Development

No branches or pull requests

4 participants