Return stringlist types in KPluginMetaData::value

Review Request #129770 - Created Jan. 4, 2017 and submitted

Information
David Edmundson
kcoreaddons
master
153416c...
Reviewers
kdeframeworks

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.


  

Issues

  • 1
  • 0
  • 0
  • 1
Description From Last Updated
return defaultValue Kai Uwe Broulik Kai Uwe Broulik
Marco Martin
Kai Uwe Broulik
David Edmundson
Aleix Pol Gonzalez
David Edmundson
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted with commit 2ee9633a244a44d5b717fd9b00ec3e6e2fb236fa by David Edmundson to branch master.
Loading...