Review Request 122794: Make actions plugin return actions also for all subcontacts

Martin Klapetek martin.klapetek at gmail.com
Tue Mar 3 17:36:43 UTC 2015


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

Review request for Telepathy.


Repository: ktp-common-internals


Description
-------

Till now the plugin returns only actions for most online contact. With this it iterates over the subcontacts and creates list of actions for them too, restoring the 0.9 version behavior.

Single actions for subcontacts remain broken as the actions plugin now always operates on person.


Diffs
-----

  kpeople/actionsplugin/kpeople-actions-plugin.cpp 24712a8 

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


Testing
-------

See screenshot; one of the actions is there twice because there are two different contacts from the same account merged into one. Making this more clever would require changing strings which is not allowed now.


File Attachments
----------------

context-menu.png
  https://git.reviewboard.kde.org/media/uploaded/files/2015/03/03/0cbbbedd-a0a1-4c5c-a592-3aba41829112__context-menu.png


Thanks,

Martin Klapetek

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


More information about the KDE-Telepathy mailing list