Review Request 121737: KPluginInfo: Fix properties not being copied from KService::Ptr

[David Faure]

> On Dec. 29, 2014, 5:56 p.m., David Faure wrote:
> > propertyNames() includes the common stuff like Name, Comment, etc.
> > 
> > So maybe this removes the need for the code that follows? Or at least a good part of it?
> 
> Alex Richardson wrote:
>     Yeah we probably don't want those properties, do you think letting KPluginInfo access the QVariantMap in KServicePrivate is a better idea?
>     
>     I.e. `json = QJsonObject::fromVariantMap(service->d->m_mapProps)`
>     
>     That would be more efficient and skip all those standard .desktop properties.

Ah, hmm, not exactly what I meant :-)

Are you sure you don't want stuff like "Keywords"? What if someone wants to find KCMs that have certain keywords, shouldn't that be in the json file?

Or "MimeTypes", for the same reason?
Or "Icon", to display per-plugin icons without loading them? (e.g. in the plugin selector)

I'm surprised by the idea of having only Name, Comment, ServiceType, and custom properties, but not some of the other standard stuff above.

But maybe I'm missing some context.


- David


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


On Dec. 29, 2014, 5:51 p.m., Alex Richardson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121737/
> -----------------------------------------------------------
> 
> (Updated Dec. 29, 2014, 5:51 p.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kservice
> 
> 
> Description
> -------
> 
> This should fix property() returning empty QVariants even if it
> existed in the .desktop file
> 
> 
> Diffs
> -----
> 
>   src/services/kplugininfo.cpp aa5d26386eb17a7e552b29c8fa66667961d68b16 
> 
> Diff: https://git.reviewboard.kde.org/r/121737/diff/
> 
> 
> Testing
> -------
> 
> KDED now correctly reads X-Kded-* in https://git.reviewboard.kde.org/r/121315/ instead of reading the default value.
> 
> 
> Thanks,
> 
> Alex Richardson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141229/1618d5b9/attachment.html>