Review Request 123485: ktp-kded-module status handler multiple account presence

[James Smith]

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

(Updated Jan. 28, 2017, 4:54 p.m.)


Review request for Telepathy, Dominik Cermak, David Edmundson, Lucas Betschart, and Martin Klapetek.


Changes
-------

Copyright cleanup.

PluginQueue:
1) Inline and separate the status message and presence queue code from queueChanged().
2) Remove checks for running timers of the other queue.

AccountStatusHelper:
1) Check for account offline before setting it to an online presence while terminating.
2) Clean up after disabled accounts. Fix lambda signal disconnects. Remove destructor account signal disconnects.
3) Move getAutoConnectMode() into reloadConfig(), other AutoConnect changes and cleanups.
4) DBus for-export strictness.
5) Presence isValid() check on load from disk.

StatusHandler:
1) Remove status message and presence caching.
2) No longer check for invalid presences.
3) Sanitize status message in accountCompositePresence().
4) Simplify requestedAccountPresenceChanged().
5) accountCompositePresence() -> compositePresence().
6) Include KTp/types.h.

TelepathyKDEDModulePlugin:
1) Overload setPlugin() replaces setPluginState(), setRequestedPresence(), and setRequestedStatusMessage().
2) New State Q_ENUM replaces KTp::PluginQueue enum.
3) Include ktp_kded_debug.h.

Include reordering.
StatusInserter:
1) Presence isValid() check.
2) Intermediate local presence variable.

StatusMessageInserter:
1) Intermediate local status message variable.


Bugs: 293443, 297924, 334542, 337881, 340201, and 340781
    http://bugs.kde.org/show_bug.cgi?id=293443
    http://bugs.kde.org/show_bug.cgi?id=297924
    http://bugs.kde.org/show_bug.cgi?id=334542
    http://bugs.kde.org/show_bug.cgi?id=337881
    http://bugs.kde.org/show_bug.cgi?id=340201
    http://bugs.kde.org/show_bug.cgi?id=340781


Repository: ktp-kded-module


Description (updated)
-------

New classes:
-AccountStatusHelper, a multiple-account-presence QDBusAbstractInterface replacement for AutoConnect.
-PluginQueue, a QDBusAbstractInterface for presence and status message plugins.
-Presence / Status Message Inserter classes, for queuing presences and status messages.

New properties:
-TelepathyKDEDModulePlugin: KTp::PluginQueue pluginState().

New methods:
-TelepathyKDEDModulePlugin: virtual pluginInit(bool), method setPlugin(State), method setPlugin(KTp::Presence), method setPlugin(QString), method configureEnabled(bool).

New features:
-Newly enabled and online account presence changes to account requested presence, or plugin presence depending on autoconnect property, and plugin queue state.
-Offline global presence will override per-account presences temporarily.
-PluginQueue globs presence change events.
-StatusHandler dynamically disables and enables plugin queues if there are online accounts.
-StatusHandler and TelepathyKDEDModulePlugin no longer require GlobalPresence.

Various other cleanups, documentation, and improvements.


Diffs (updated)
-----

  CMakeLists.txt 4dbaab64189d819a27e5b36fd10098f0fac3dbc6 
  account-status-helper.h PRE-CREATION 
  account-status-helper.cpp PRE-CREATION 
  autoaway.h 5e2a56100b56e6d21bc649f9032c8bece9031452 
  autoaway.cpp 4881c71afab2be2468c3c883523d2844865e6247 
  autoconnect.h 0f3ab137cf67cce9c7467144a45d1706ec9cc007 
  autoconnect.cpp 99512b265e0e6ae34f83263ffa6337741db652ea 
  config/telepathy-kded-config.cpp 88220645c2e119dc7879cdae065cbbf06aa13329 
  config/telepathy-kded-config.ui 2b80dfa34381af2e9206384a31c025f9b914854a 
  org.kde.Telepathy.AccountStatusHelper.xml PRE-CREATION 
  org.kde.Telepathy.PluginQueue.xml PRE-CREATION 
  screensaveraway.h 92344e891ccde16d53771c3e279d845f4d800b2d 
  screensaveraway.cpp f226564a5bf7b396965b5ae21f81d93b7edc3ef8 
  status-handler-plugin-queue.h PRE-CREATION 
  status-handler-plugin-queue.cpp PRE-CREATION 
  status-handler.h 06240ff17e22148f2b128bc0eb8cec6d6abe68ff 
  status-handler.cpp 4b9c25a2ccba451f6e608bb704626e33149108cc 
  telepathy-kded-module-plugin.h 4c161696a706e82059a7eb314773c3644fe26bd7 
  telepathy-kded-module-plugin.cpp daf73c66947bc946097de7a8e8a1518555131145 
  telepathy-module.h 17ef4cef27b90cbced6c66846ed82cb5c36fb532 
  telepathy-module.cpp b74053bc1817d773a4fe37769e42f4013ced5425 
  telepathy-mpris.h 05b77c90a50372fd9ed66bde0ab8a287caf34b51 
  telepathy-mpris.cpp ee0e622c68bdd156e45914f542d2fe13f0ddb610 
  telepathy-status-inserter.h PRE-CREATION 
  telepathy-status-inserter.cpp PRE-CREATION 
  tests/CMakeLists.txt 7ec77495417a6790060ea5ea7126c46399dff755 

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


Testing
-------

Compile, run.


Thanks,

James Smith

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20170128/3fcbec83/attachment.html>