[Kde-pim] Review Request: vCard export: export birthday field as date only

Jonathan Marten jjm at keelhaul.me.uk
Mon Jan 23 20:27:21 GMT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103776/
-----------------------------------------------------------

(Updated Jan. 23, 2012, 8:27 p.m.)


Review request for KDEPIM.


Changes
-------

Fix typo.


Description (updated)
-------

The referenced bugs refer to the BDAY field in an exported vCard including a time field of midnight, even though the user will not have entered this in KAddressBook and there is no GUI to do so:

BDAY:1978-07-20T00:00:00Z

This is valid according to RFC2426, but unfortunately some receiving applications - notably GMail and Android - do not recognise the birthday field in this format and ignore it.  If the time part is removed:

BDAY:1978-07-20

then the birthday date is recognised.

This patch implements that, only for vCard export from KAddressBook.  It does not modify or change the behaviour of the KABC library, so not affecting any other applications that may use this format with a valid time (and kdelibs is frozen now anyway).


This addresses bugs 118289 and 282169.
    http://bugs.kde.org/show_bug.cgi?id=118289
    http://bugs.kde.org/show_bug.cgi?id=282169


Diffs
-----

  kaddressbook/xxport/vcard/vcard_xxport.cpp bb4211b 

Diff: http://git.reviewboard.kde.org/r/103776/diff/diff


Testing
-------

Build kdepim with these changes, checked vCard export by visual inspection of the output and by import into GMail Contacts.


Thanks,

Jonathan Marten

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list