Bug 346515 - During import from vcard 3.0 first name and last name are reversed.
Summary: During import from vcard 3.0 first name and last name are reversed.
Status: RESOLVED UNMAINTAINED
Alias: None
Product: kaddressbook
Classification: Applications
Component: general (show other bugs)
Version: 4.11.5
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-23 17:13 UTC by Günter Ohmer
Modified: 2018-02-01 09:48 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
after import (17.72 KB, image/jpeg)
2015-04-24 07:24 UTC, Günter Ohmer
Details
before export (17.92 KB, image/jpeg)
2015-04-24 07:28 UTC, Günter Ohmer
Details
vcard (371 bytes, text/vcard)
2015-04-24 07:30 UTC, Günter Ohmer
Details
Lastname_5_imported.jpg (63.56 KB, image/jpeg)
2015-04-27 07:16 UTC, Günter Ohmer
Details
Lastname_1_new_contact.jpg (75.25 KB, image/jpeg)
2015-04-27 07:16 UTC, Günter Ohmer
Details
Lastname_2_Anzeigen:_umgekehrter_Name_mit_ Komma.jpg (10.82 KB, image/jpeg)
2015-04-27 07:16 UTC, Günter Ohmer
Details
Lastname_4_Kontaktname bearbeiten.jpg (17.74 KB, image/jpeg)
2015-04-27 07:16 UTC, Günter Ohmer
Details
Lastname_3_changed_contact.jpg (71.39 KB, image/jpeg)
2015-04-27 07:16 UTC, Günter Ohmer
Details
Adam_Lastname.vcf (111 bytes, text/vcard)
2015-04-27 07:16 UTC, Günter Ohmer
Details
Lastname,_Adam.vcf (115 bytes, text/vcard)
2015-04-27 07:16 UTC, Günter Ohmer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Günter Ohmer 2015-04-23 17:13:23 UTC
Create a contact with first name and last name, export it to vcard v3.0 and import it again, then first name and last name are reversed.

Reproducible: Always
Comment 1 Laurent Montel 2015-04-24 05:18:24 UTC
In 4.14.x it works fine.
What is "first name"/"last name" for you .?
Could you provide a screenshot for it ?
Comment 2 Günter Ohmer 2015-04-24 07:24:30 UTC
Created attachment 92191 [details]
after import
Comment 3 Günter Ohmer 2015-04-24 07:28:09 UTC
Created attachment 92192 [details]
before export
Comment 4 Günter Ohmer 2015-04-24 07:30:41 UTC
Created attachment 92193 [details]
vcard
Comment 5 Laurent Montel 2015-04-24 07:57:36 UTC
Thanks for screenshot.
But in 4.14.x it works fine.
Regards
Comment 6 Günter Ohmer 2015-04-24 11:05:05 UTC
(In reply to Laurent Montel from comment #5)
> Thanks for screenshot.
> But in 4.14.x it works fine.
> Regards

Ok, I installed 4.14.6 from the openSUSE repository. Nothing changed for me.
But I found the error, the button "Show: reverse name with comma" makes a wrong vcard:
=========== without the button ==============
BEGIN:VCARD
VERSION:4.0
FN:Adam Lastname
N:Lastname;Adam;;;
UID:74d63af5-16b8-4144-abcd-8db0d36f5174
END:VCARD
=========================================
=========== with the button ================
BEGIN:VCARD
VERSION:4.0
FN:Lastname\, Adam
N:Adam;Lastname\,;;;
UID:74d63af5-16b8-4144-abcd-8db0d36f5174
END:VCARD
=========================================
The function should not reverse the name in the "N:"  line.
=========== with the button, edited==========
BEGIN:VCARD
VERSION:4.0
FN:Lastname\, Adam
N:Lastname;Adam;;;
UID:74d63af5-16b8-4144-abcd-8db0d36f5174
END:VCARD
=========================================
It  w o r k s!
Best regards
Günter
Comment 7 Laurent Montel 2015-04-24 13:53:39 UTC
"Ok, I installed 4.14.6 from the openSUSE repository. Nothing changed for me." ?
you still have a bug ?


"Show: reverse name with comma" makes a wrong vcard:

could you open a new bug report ?
you exported to vcard4.0 not vcard3.0
Comment 8 Günter Ohmer 2015-04-24 16:38:18 UTC
Am Freitag, 24. April 2015, 13:53:39 schrieben Sie:
> https://bugs.kde.org/show_bug.cgi?id=346515
> 
> --- Comment #7 from Laurent Montel <montel@kde.org> ---
> "Ok, I installed 4.14.6 from the openSUSE repository. Nothing changed for
> me." ?
> you still have a bug ?
> 
> 
> "Show: reverse name with comma" makes a wrong vcard:
> 
> could you open a new bug report ?
> you exported to vcard4.0 not vcard3.0

Yes, I thought the bug might be eliminated with vcard4.0.
I wrote my comments on bug 34615, can you reopen it?
regards
Günter
Comment 9 Laurent Montel 2015-04-24 21:50:06 UTC
you reopened it but it's not the same bug no ?
I don't understand bug is still here or not ?
(initial bug)
Comment 10 Günter Ohmer 2015-04-25 07:42:56 UTC
Hi Laurent,
I'm not familiar with the Bugtracking System. For me it is the same bug, 
because the symptoms are the same. I export a contact to vcard import it again 
and the contact changes.

But I found the reason why:
The order of the names in the "N" line in the vcard must always be lastName, 
firstName have, regardless of the order in the "FN:" line.

=========== without "Show: reverse name with comma" ========================
BEGIN:VCARD
VERSION:4.0
FN:Adam Lastname
N:Lastname;Adam;;;
UID:74d63af5-16b8-4144-abcd-8db0d36f5174
END:VCARD
=============================================================================
=========== with "Show: reverse name with comma"=============================
BEGIN:VCARD
VERSION:4.0
FN:Lastname\, Adam
N:Adam;Lastname\,;;;
UID:74d63af5-16b8-4144-abcd-8db0d36f5174
END:VCARD
=============================================================================
=========== the right form ==================================================
BEGIN:VCARD
VERSION:4.0
FN:Lastname\, Adam
N:Lastname;Adam;;;
UID:74d63af5-16b8-4144-abcd-8db0d36f5174
END:VCARD
==============================================================================

see also: http://en.wikipedia.org/wiki/VCard

FN	The formatted name string associated with the vCard object. 	FN:Dr. John 
Doe

N	A structured representation of the name of the person, place or thing 
associated with the vCard object. 	N:Doe;John;;Dr;

Best regards
Günter
Comment 11 Günter Ohmer 2015-04-25 07:49:55 UTC
Oh, excuse in my example interchanged N: and FN:
Comment 12 Laurent Montel 2015-04-25 12:06:10 UTC
"Show: reverse name with comma" where do you have it ?
Comment 13 Laurent Montel 2015-04-27 04:52:54 UTC
Ok I see a bug
Comment 14 Günter Ohmer 2015-04-27 07:16:54 UTC
Created attachment 92245 [details]
Lastname_5_imported.jpg

Hi Laurent,

I'll send you some screenshots and the vcards:

I created a new contact:
	Lastname_1_new_contact.jpg
	
I choose "Show: reverse name with comma"
	Lastname_2_Anzeigen:_umgekehrter_Name_mit_ Komma.jpg

I save the contact and open it again 
	Lastname_3_changed_contact.jpg
	Lastname_4_Kontaktname bearbeiten.jpg

I export the contact as vcard,delete it and import it again
	Lastname_5_imported.jpg


regards
Günter
Comment 15 Günter Ohmer 2015-04-27 07:16:56 UTC
Created attachment 92246 [details]
Lastname_1_new_contact.jpg
Comment 16 Günter Ohmer 2015-04-27 07:16:56 UTC
Created attachment 92247 [details]
Lastname_2_Anzeigen:_umgekehrter_Name_mit_ Komma.jpg
Comment 17 Günter Ohmer 2015-04-27 07:16:56 UTC
Created attachment 92248 [details]
Lastname_4_Kontaktname bearbeiten.jpg
Comment 18 Günter Ohmer 2015-04-27 07:16:56 UTC
Created attachment 92249 [details]
Lastname_3_changed_contact.jpg
Comment 19 Günter Ohmer 2015-04-27 07:16:56 UTC
Created attachment 92250 [details]
Adam_Lastname.vcf
Comment 20 Günter Ohmer 2015-04-27 07:16:56 UTC
Created attachment 92251 [details]
Lastname,_Adam.vcf
Comment 21 Denis Kurz 2017-06-23 19:59:33 UTC
This bug has never been confirmed for a KDE PIM version that is based on KDE Frameworks (5.x). Those versions differ significantly from the old 4.x series. Therefore, I plan to close it in around two or three months. In the meantime, it is set to WAITINGFORINFO to give reporters the oportunity to check if it is still valid. As soon as someone confirms it for a recent version (at least 5.1, ideally even more recent), I'll gladly reopen it.

Please understand that we lack the manpower to triage bugs reported for versions almost two years beyond their end of life.
Comment 22 Denis Kurz 2018-02-01 09:48:29 UTC
Just as announced in my last comment, I close this bug. If you encounter it again in a recent version (at least 5.1 aka 15.12; preferably much more recent), please open a new one unless it already exists. Thank you for all your input.