Review Request: Fix the runner applet to work with a newer version of Telepathy/KTp

Dan Vratil dan at progdan.cz
Sun Jul 1 08:05:29 UTC 2012


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

Review request for Telepathy.


Description
-------

Since some (git) version of KTp or TelepathyQt (hard to say) the Accounts returned from AccountsModel have "ktp_contacts_" prefix in their ID but accountItemForId() fails to lookup the account with such prefix. Removing the prefix before passing the ID to accountItemForId() fixes the problem.

This fix does not affect older versions of Telepathy (where accounts don't have the prefix in ID), it just works by default there.

I guess this should be actually fixed in the AccountsModel (or wherever the problem comes from), but I briefly checked the code and haven't seen any obvious reason for this behavior, so I leave it up to you guys :-)


This addresses bug 302176.
    http://bugs.kde.org/show_bug.cgi?id=302176


Diffs
-----

  src/contactrunner.h c316aa4 
  src/contactrunner.cpp db39190 

Diff: http://git.reviewboard.kde.org/r/105398/diff/


Testing
-------

Successfully reproduced the bug, with this patch it works again.


Thanks,

Dan Vratil

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


More information about the KDE-Telepathy mailing list