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

Changing email type doesn't update X-ABLABEL #560

Closed
raoulbhatia opened this issue May 29, 2018 · 3 comments
Closed

Changing email type doesn't update X-ABLABEL #560

raoulbhatia opened this issue May 29, 2018 · 3 comments
Labels
1. to develop Accepted and waiting to be taken care of compatibility Compatibility with other services

Comments

@raoulbhatia
Copy link
Contributor

raoulbhatia commented May 29, 2018

Steps to reproduce

  1. Upload a contact with X-ABLABEL set for the email, i.e.
ITEM1.EMAIL;TYPE=OTHER;TYPE=pref:email@domain.tld
ITEM1.X-ABLABEL:_$!<Other>!$_
  1. The email address will be shown as type _$!<OTHER>!$_.
  2. Change the type to Work.
  3. Switch contacts from one to another contact and back.
  4. The email address still shows as type _$!<OTHER>!$_.

Expected behaviour

The vcard should be sanitized and simplified to contain something like:

EMAIL;TYPE=WORK:email@domain.tld

Actual behaviour

The updated vcard is PUT to the server. The request parameter shows:

ITEM1.EMAIL;TYPE=WORK;TYPE=pref:email@domain.tld
ITEM1.X-ABLABEL:_$!<Other>!$_

Server configuration

Operating system: Ubuntu 17.10

Web server: nginx from Ubuntu package nginx-extras version 1.12.1-0ubuntu2

Database: MySQL from Ubuntu version 5.7.22-0ubuntu0.17.10.1

PHP version: PHP7.1 from Ubuntu version 7.1.17-0ubuntu0.17.10.1

Nextcloud version: 13.0.2

Contacts version: 2.1.3

Updated from an older Nextcloud or fresh install: Update through various releases. Last update from 13.0.1.

Signing status:

No errors have been found.

List of activated apps:

Let me know if relevant to share.

Nextcloud configuration:

Let me know if relevant to share.

Are you using external storage, if yes which one: No

Are you using encryption: No

Are you using an external user-backend, if yes which one: No

Client configuration

Browser: Firefox 60.0.1

Operating system: MacOS

CardDAV-clients: N/A

Logs

Web server error log

N/A

Nextcloud log (data/nextcloud.log)

N/A

Browser log

Let me know if relevant to share.
@skjnldsv
Copy link
Member

Hello! :)
Yes, we do no supports X-properties yet.

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of compatibility Compatibility with other services labels May 29, 2018
@raoulbhatia
Copy link
Contributor Author

Thanks for the update. Let me know if there is anything I can help with – besides writing a patch ;-)

@skjnldsv
Copy link
Member

skjnldsv commented Oct 1, 2018

Let's put his into another issue
Duplicate of #27 and #42

@skjnldsv skjnldsv closed this as completed Oct 1, 2018
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 compatibility Compatibility with other services
Projects
None yet
Development

No branches or pull requests

2 participants