[Nepomuk] Re: Review Request: Add more methods to Nepomuk Service

[Sebastian Trueg]

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

Ship it!


Very good.

- Sebastian


On June 26, 2011, 9:27 p.m., Artem Serebriyskiy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101759/
> -----------------------------------------------------------
> 
> (Updated June 26, 2011, 9:27 p.m.)
> 
> 
> Review request for Nepomuk.
> 
> 
> Summary
> -------
> 
> Add methods userName() and description() to Nepomuk Services - in object /servicecontrol
> These methods are necessary for Nepomuk Plasma dataengine, usefull for Nepomuk System Tray, and, generally speaking, usefull for every application that don't want to mess up with .desktop files. Because before this patch there is no way to query name and description of running Nepomuk Service except to find corresponding .desktop file and parse it.
> 
> 
> Diffs
> -----
> 
>   nepomuk/interfaces/org.kde.nepomuk.ServiceControl.xml dd91c8f 
>   nepomuk/servicestub/servicecontrol.h bd77324 
>   nepomuk/servicestub/servicecontrol.cpp 2ad4f9f 
> 
> Diff: http://git.reviewboard.kde.org/r/101759/diff
> 
> 
> Testing
> -------
> 
> Compling. Launching. 
> The only error discovered - qdbusviewer refused to call these methods. But calling using python-dbus works as expected, so I think it is qdbusviewer problem.
> 
> 
> Thanks,
> 
> Artem
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/nepomuk/attachments/20110627/a5d4609e/attachment-0001.htm