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

Cannot import contacts without FNs. #3980

Closed
RokeJulianLockhart opened this issue Jun 14, 2024 · 3 comments
Closed

Cannot import contacts without FNs. #3980

RokeJulianLockhart opened this issue Jun 14, 2024 · 3 comments
Labels
0. Needs triage bug Something isn't working feature: import Importing (and exporting) wontfix This will not be worked on

Comments

@RokeJulianLockhart
Copy link

RokeJulianLockhart commented Jun 14, 2024

Describe the bug

Like Kozea/Radicale#752 (comment) describes, I'm unable to import contact entries without FNs (in a .VCF which contains multiple entries, with and without FNs).

Steps to reproduce

  1. Create an applicable contact, like:

    BEGIN:VCARD
    VERSION:3.0
    PRODID;VALUE=TEXT:ez-vcard 0.11.3
    UID:47759cae-7de1-4ae5-82c1-b3bc560e90da
    REV;VALUE=DATE-AND-OR-TIME:20240224T161008Z
    END:VCARD
    

    ...(with FN: absent).

  2. Export all contacts (using https://wim.nl.tab.digital./remote.php/dav/addressbooks/users/rokejulianlockhart/sf2zs2/?export# at https://wim.nl.tab.digital./apps/contacts/All%20contacts/06f18e31-590e-4635-8309-98ff9e8ccfe9~sf2znd#).

  3. Delete all contacts.

  4. Attempt to import all contacts.

Expected behavior

I should be able to import them.

Actual behavior

I'm unable to import them, because those without FNs are considered to be erroneous.

Contact version

Unknown, but at most, https://github.com/nextcloud/contacts/releases/tag/v5.0.1.

Operating system

cpe:/o:canonical:ubuntu

PHP engine version

None

Web server

nginx/1.18.0

Database

{"needsDbUpgrade": false} // https://wim.nl.tab.digital/status.php

Additional info

As https://poe.com./s/ncCrUStKKizgXNtuPrUw# explains, BEGIN:VCARD((?:(?!END:VCARD|FN:).|\n)*)END:VCARD can be used to identify faulty entries at https://regex101.com./# and in code-insiders-1.91.0-1718259485.el8.rpm.

@hamza221
Copy link
Contributor

hamza221 commented Jul 1, 2024

Can you please provide screenshots/screencasts or code links instead of links from instances

@hamza221 hamza221 added the wontfix This will not be worked on label Jul 1, 2024
@hamza221
Copy link
Contributor

hamza221 commented Jul 1, 2024

We support VCard V3 and V4 Where FN is mandatory

@hamza221 hamza221 closed this as not planned Won't fix, can't repro, duplicate, stale Jul 1, 2024
@RokeJulianLockhart
Copy link
Author

RokeJulianLockhart commented Jul 1, 2024

#3980 (comment)

I did provide code. It's in #3980 (comment):

BEGIN:VCARD
VERSION:3.0
PRODID;VALUE=TEXT:ez-vcard 0.11.3
UID:47759cae-7de1-4ae5-82c1-b3bc560e90da
REV;VALUE=DATE-AND-OR-TIME:20240224T161008Z
END:VCARD

Irrespective, I don't see how the URIs relate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage bug Something isn't working feature: import Importing (and exporting) wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants