Review Request 121639: Split out KF5Contacts dependency

Aleix Pol Gonzalez aleixpol at kde.org
Wed Dec 31 12:16:37 UTC 2014


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

(Updated Dec. 31, 2014, 12:16 p.m.)


Review request for KDEPIM, Telepathy, Christian Mollekopf, David Edmundson, and Martin Klapetek.


Changes
-------

Fixes small logic issue with the patch that didn't let avatars work on the PersonsModel. Now I can use it normally like I used to (ktp-contactlist, plasmoid, etc).


Repository: libkpeople


Description
-------

Last week we had a couple of discussions regarding KPeople relationship with KDE PIM and KContacts and we agreed that we want to keep KContacts as a framework that is good at dealing with vCards, which leaves KPeople without a good way to send around its data.

This patch proposes an internal AbstractContact class that the backends will have to re-implement. At the moment it only requires a "customProperty" that lets us fetch whatever is needed.
Furthermore, it also includes a set of convenience methods in PersonData to have a type-safe API.


Diffs (updated)
-----

  CMakeLists.txt 3df5f0b 
  autotests/CMakeLists.txt 676f170 
  autotests/fakecontactsource.h 5852297 
  autotests/fakecontactsource.cpp 269f94f 
  autotests/persondatatests.cpp 47a6213 
  examples/CMakeLists.txt 74bd188 
  examples/duplicates.cpp 72ff53b 
  src/CMakeLists.txt 534066b 
  src/backends/abstractcontact.h PRE-CREATION 
  src/backends/abstractcontact.cpp PRE-CREATION 
  src/backends/abstractpersonaction.h 703063e 
  src/backends/abstractpersonaction.cpp be93d90 
  src/backends/allcontactsmonitor.h 47e5dfd 
  src/backends/allcontactsmonitor.cpp e517f17 
  src/backends/basepersonsdatasource.h 3c25e28 
  src/backends/basepersonsdatasource.cpp b794917 
  src/backends/contactmonitor.h cf25953 
  src/backends/contactmonitor.cpp ceb5d74 
  src/declarative/personactionsmodel.cpp 4b34e15 
  src/defaultcontactmonitor.cpp 36c31a7 
  src/defaultcontactmonitor_p.h 8fc2812 
  src/duplicatesfinder.cpp c6a1014 
  src/global.h 9540623 
  src/global.cpp 0a3b00d 
  src/match.cpp 3a93aba 
  src/match_p.h 5727270 
  src/metacontact.cpp bba6b65 
  src/metacontact_p.h 0d738cc 
  src/persondata.h 66b19a3 
  src/persondata.cpp 40156a9 
  src/personpluginmanager.cpp 59f5a48 
  src/personsmodel.h db907d4 
  src/personsmodel.cpp c2d97f6 
  src/plugins/akonadi/akonadidatasource.cpp ae4ace1 
  src/widgets/abstractfieldwidgetfactory.h 1d2d8aa 
  src/widgets/persondetailsdialog.cpp 8ca0a99 
  src/widgets/persondetailsview.cpp b7535bd 
  src/widgets/plugins/emaildetailswidget.h 72a96ee 
  src/widgets/plugins/emaildetailswidget.cpp 839c5f6 
  src/widgets/plugins/emails.h ede1686 
  src/widgets/plugins/emails.cpp d69101b 

Diff: https://git.reviewboard.kde.org/r/121639/diff/


Testing
-------

Builds, the test passes. ktp tests work, somewhat.
Also there's few tests altogether.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20141231/1a71fa41/attachment.html>


More information about the KDE-Telepathy mailing list