Bug 301702 - KDE Telepathy support in IMStatus plugin
Summary: KDE Telepathy support in IMStatus plugin
Status: RESOLVED FIXED
Alias: None
Product: choqok
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: 1.5
Assignee: Mehrdad Momeny
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-12 03:38 UTC by Mehrdad Momeny
Modified: 2014-12-18 10:43 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 1.5
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mehrdad Momeny 2012-06-12 03:38:43 UTC
We need to support changing user status in KDE Telepathy in IMStatus plugin.
Comment 1 Ahmed Ibrahim 2014-12-15 16:22:29 UTC
I'll try to take a look on it
Comment 2 Ahmed Ibrahim 2014-12-15 20:29:25 UTC
I've tried doing it without Telepathy-Qt by using QtDBus to contact the Telepathy DBus interface directly, but my efforts went in vain in trying to parse the response.

Is it okay to link IMStatus plugin with Telepathy-Qt ?
Comment 3 Andrea Scarpino 2014-12-15 20:32:22 UTC
IMHO sure, if you keep it optional it's fine.
Comment 4 Mehrdad Momeny 2014-12-16 02:00:47 UTC
(In reply to Ahmed Ibrahim from comment #2)
> I've tried doing it without Telepathy-Qt by using QtDBus to contact the
> Telepathy DBus interface directly, but my efforts went in vain in trying to
> parse the response.
> 
> Is it okay to link IMStatus plugin with Telepathy-Qt ?
It's OK, just make sure if the Tepepathy-Qt is missing on system, that plugin won't stop the build. i.e. make it optional as Andrea mentioned.
Comment 5 Ahmed Ibrahim 2014-12-18 10:28:40 UTC
Git commit eb235c57b6d98270bc1d4fc3601a220597a1ff48 by Ahmed I. Khalil.
Committed on 17/12/2014 at 14:26.
Pushed by ibrahim into branch 'master'.

KDE Telepathy support in IMStatus Plugin

REVIEW: 121577

M  +6    -2    plugins/imstatus/CMakeLists.txt
A  +1    -0    plugins/imstatus/config-imstatus.h.cmake
M  +76   -19   plugins/imstatus/imqdbus.cpp
M  +28   -9    plugins/imstatus/imqdbus.h
M  +10   -2    plugins/imstatus/imstatus.cpp
M  +4    -0    plugins/imstatus/imstatus.h

http://commits.kde.org/choqok/eb235c57b6d98270bc1d4fc3601a220597a1ff48