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