D20344: Remove Kolab <-> LDAP person identity merging code
Daniel Vrátil
noreply at phabricator.kde.org
Sun Apr 7 13:00:39 BST 2019
dvratil created this revision.
Herald added a project: KDE PIM.
Herald added a subscriber: kde-pim.
dvratil requested review of this revision.
REVISION SUMMARY
For shared people's calendars on Kolab this code would lookup a matching
entry in LDAP directory to display the real name, and allow for showing
FB calendar. Since this only works with calendaring through Kolab
resource and the current recommended approach is to use DAV for
calendaring instead, we can remove this rarely-used code and simplify
the logic in CalendarView.
This also allow us to remove the KPIM::Person and related code from
libkdepim and finally drop collection referencing from Akonadi.
TEST PLAN
KOrganizer compiles, quick view still works.
REPOSITORY
R210 KOrganizer
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D20344
AFFECTED FILES
src/CMakeLists.txt
src/akonadicollectionview.cpp
src/akonadicollectionview.h
src/views/collectionview/calendardelegate.cpp
src/views/collectionview/calendardelegate.h
src/views/collectionview/controller.cpp
src/views/collectionview/controller.h
src/views/collectionview/quickview.cpp
src/views/collectionview/quickview.h
src/views/multiagendaview/multiagendaview.cpp
To: dvratil
Cc: kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190407/6d1fd3de/attachment.html>
More information about the kde-pim
mailing list