Review Request 128616: Fix wrong or missing "X-KDE-ParentApp" in desktop file definitions (and fix copy&paste errors in API dox)

Friedrich W. H. Kossebau kossebau at kde.org
Sat Aug 6 00:08:32 UTC 2016


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

Review request for KDE Frameworks and Plasma.


Repository: plasma-framework


Description
-------

plasma-dataengine.desktop misses at least the definition of "X-KDE-ParentApp" property.

plasma-applet.desktop defines a "X-Plasma-ParentApp" property, but in code and API dox the respective key used is named "X-KDE-ParentApp".

E.g. `kcoreaddons_desktop_to_json` used with `SERVICE_TYPES` `plasma-dataengine.desktop` or `plasma-applet.desktop` complains about that for desktop files with such entries.

While looking at that property, the related API dox of `Plasma::PluginLoader` has lots of copy&paste errors (talks about "applets" when dataengines, containments, etc. are meant). Also inconsistent use of "applets" vs. "Applets", "dataengines" vs. "DataEngines", etc. This patch also cleans that up (separate commit).


Diffs
-----

  src/plasma/data/servicetypes/plasma-applet.desktop e142552 
  src/plasma/data/servicetypes/plasma-dataengine.desktop 9280645 
  src/plasma/pluginloader.h 80cd1e9 

Diff: https://git.reviewboard.kde.org/r/128616/diff/


Testing
-------


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160806/d38b96ce/attachment.html>


More information about the Plasma-devel mailing list