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

An email address label is not set if the address type is given twice #1030

Closed
j-ed opened this issue Mar 29, 2019 · 4 comments · Fixed by #1042
Closed

An email address label is not set if the address type is given twice #1030

j-ed opened this issue Mar 29, 2019 · 4 comments · Fixed by #1042
Labels
3. to review Waiting for reviews bug Something isn't working

Comments

@j-ed
Copy link

j-ed commented Mar 29, 2019

Expected behavior

The Contacts app should show the correct email address label if the address type is given twice.

Current behavior

The contacts app shows an empty email address label.

Example:

EMAIL;TYPE=WORK,WORK,pref:[email protected]

** Result:**
20190329_nc-kontakte_empty_email_label

Environment

Server Configuration

OS: Linux 3.16.63
Web server: Apache2 2.4.37
Database: MariaDB 10.2.22
PHP version: 7.2.14
Nextcloud version: 15.0.5
Contacts app version: 3.0.5

Client Configuration

Browser: Mozilla Firefox 66.0.2
Operating system: Windows 10

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #771 (add email address not possible if contact already has one or more email addresses), #1027 (ITEM2.xxx properties - the url label is not set at all), #1026 (ITEM2.xxx properties - the url label is not correctly set), #1028 (ITEM2.xxx properties - the phone number label is not correctly set), and #518 (CardDav types).

@nextcloud-bot nextcloud-bot added the bug Something isn't working label Mar 29, 2019
@skjnldsv
Copy link
Member

EMAIL;TYPE=WORK,WORK,pref:[email protected]

I'm not sure I get it. You mean because there is two WORK types?
We should indeed have a repair step for this, as this is not valid 🤔

@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

That's my assumption, because several other entries are correctly displayed. Unfortunately I found several different ways how email address entry types have been formatted in my address book. All others seem to be correctly displayed. Here is the list:

EMAIL
EMAIL;TYPE="home,pref"
EMAIL;TYPE="work,pref"
EMAIL;TYPE=HOME
EMAIL;TYPE=HOME,PREF
EMAIL;TYPE=WORK
EMAIL;TYPE=WORK,PREF
EMAIL;TYPE=WORK,WORK,pref
EMAIL;TYPE=home
EMAIL;TYPE=home,pref
EMAIL;TYPE=work
EMAIL;TYPE=work,pref
EMAIL;TYPE=work;PREF=1

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of and removed needs info Not enough information provided labels Mar 29, 2019
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Apr 6, 2019
@skjnldsv
Copy link
Member

skjnldsv commented Apr 6, 2019

Fix in #1042

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants