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