[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] Review Request: Add new "default" fields to KPIMIdentities::Identity
From:       "Kevin Krammer" <kevin.krammer () gmx ! at>
Date:       2012-03-08 14:04:43
Message-ID: 20120308140443.28122.95534 () vidsolbach ! de
[Download RAW message or body]



> On March 8, 2012, 9:30 a.m., Kevin Krammer wrote:
> > I am wondering if it would make more sense to return this additional data as a \
> > KABC::Addressee instead of adding getters for each new field. Or allow to \
> > associate an addressbook entry which would then contain these details.
> 
> Robert Mathias Marmorstein wrote:
> Is it just KABC::StdAddressBook which is deprecated, then?  I was afraid the whole \
> KABC namespace was going away...

Just StdAddressBook, AddressBook and KABC::Resource stuff.
KABC::Addressee is still the main data object for contacts, they are just loaded and \
saved differently


- Kevin


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


On March 7, 2012, 11:34 p.m., Robert Mathias Marmorstein wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104181/
> -----------------------------------------------------------
> 
> (Updated March 7, 2012, 11:34 p.m.)
> 
> 
> Review request for KDEPIM and KDEPIM-Libraries.
> 
> 
> Description
> -------
> 
> The KABC::StdAddressBook class has been deprecated and KPIMIdentities is the \
> suggested replacement.  Unfortunately, many of the fields that were available in \
> StdAddressBook are missing in the Identity class.  It's possible to use the \
> property() and setProperty() methods to store custom fields, but that means that \
> different apps might use different field names for things like home phone number \
> and locality (city).   
> We use this functionality in KOffice to store information about the document \
> author.  Internally, we can just use the property methods, but it would be nice to \
> have interoperability with other applications (such as KAddressBook).   
> This patch adds methods for accessing some of the missing fields (the ones we use \
> in KOffice).  Some of the functionality of StdAddressBook is still missing -- \
> instead of a list of phone numbers, I hardwired in a "Home" and "Work" phone number \
> -- but it is a step forward. 
> This is code I am not that familiar with, but it seems pretty straightforward.  I \
> welcome any comments/constructive criticism. 
> 
> Diffs
> -----
> 
> kpimidentities/identity.h b0d0e7c 
> kpimidentities/identity.cpp e1023b3 
> 
> Diff: http://git.reviewboard.kde.org/r/104181/diff/
> 
> 
> Testing
> -------
> 
> kdepimlibs compiles and passes the same unit tests it did before the change.  \
> Unfortunately, I just switched my main development computer to Kubuntu from \
> Archlinux (which made testing library changes very easy) and am still trying to \
> figure out the best way to test library changes without clobbering the system \
> packages.  Any suggestions/help would be much appreciated. 
> 
> Thanks,
> 
> Robert Mathias Marmorstein
> 
> 

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


[prev in list] [next in list] [prev in thread] [next in thread]