Review Request 120198: KPluginInfo: use KPluginMetaData instead of a QVariantMap for storage

Alexander Richardson arichardson.kde at gmail.com
Sun Sep 14 14:05:34 UTC 2014


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

Review request for KDE Frameworks.


Repository: kservice


Description
-------

A series of 4 commits:

----

KPluginInfo: use KPluginMetaData instead of a QVariantMap for storage

This means that KPluginInfo can now simply reuse the QJsonObject
returned by QPluginLoader.metaData() (by storing it in a 
KPluginMetaData object instead of having to convert the JSON to a
QVariantMap first.

Additionally this allows very efficient conversion between KPluginInfo
and KPluginMetaData.

---
Add compatibility key names to KPluginInfo::property()

---

KPluginInfo: Fix loading JSON metadata that only has compatibility keys

This can be removed in KF6, but for now allows loading all both old
style as well as new style metadata

----

kplugininfotest: also test objects constructed from JSON

This tests both new style JSON as well as JSON using the old key names


Diffs
-----

  autotests/CMakeLists.txt 913e848ba5d1754ef7726f92604d1aaa398fa107 
  autotests/kplugininfotest.cpp 34f87028ce08f2db1e5f57edbc6f99a237bf90ac 
  src/services/kplugininfo.h dea07e6e63baf2483afc4a6d43d0892efc485903 
  src/services/kplugininfo.cpp 50a6564edbbb1890c0b91badad69db967035231f 

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


Testing
-------

All unit tests still work


Thanks,

Alexander Richardson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140914/1a053808/attachment.html>


More information about the Kde-frameworks-devel mailing list