Review Request 129770: Return stringlist types in KPluginMetaData::value

David Edmundson david at davidedmundson.co.uk
Wed Jan 4 15:01:46 UTC 2017


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

Review request for KDE Frameworks.


Repository: kcoreaddons


Description
-------

Plasma code is littered with
KPluginMetaData::value("someStringList").contains("blah")

This used to accidentally work fine because we would be converting
.desktop files and have no concept of what KPluginMetaData entry should
have been a stringlist or a string.

Since 49bc26a135bf92bcddecabfdd4f3f71f65540104 we now correctly work out
the service type of a .desktop file, and so when we parse it, entries
that should have been lists are now considered a QJsonValue of type
Array rather than String, meaning KPluginMetaData::value returns nothing
instead of the original comma separated list in the .desktop file.

Even though we should fix all Plasma instances to use the correct
KPluginMetaData method for the data they're fetching, we can't afford
to have a behavioural change that will cause huge breakages.


Diffs
-----

  src/lib/plugin/kpluginmetadata.cpp 453d243dd3c9287b9637eecd1f4b9840eea09838 

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


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170104/4cd72e0f/attachment.html>


More information about the Kde-frameworks-devel mailing list