Bug 301702

Summary: KDE Telepathy support in IMStatus plugin
Product: [Applications] choqok Reporter: Mehrdad Momeny <mehrdad.momeny>
Component: generalAssignee: Mehrdad Momeny <mehrdad.momeny>
Status: RESOLVED FIXED    
Severity: wishlist CC: ahmedibrahimkhali, scarpino
Priority: NOR    
Version: unspecified   
Target Milestone: 1.5   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In: 1.5

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