On Tuesday, June 16, 2015 18:06:06 Bhushan Shah wrote: > On Tue, Jun 16, 2015 at 8:33 AM, Rex Dieter <[email protected]> wrote: > > Yes, per the aforementioned bug reports, trying to place the trashcan > > applet yields: > > > > Error loading Applet: package inexistent > > I debugged this little bit and found that trash applet have > X-KDE-Library set in its metadata.desktop which is unused. I removed > it now however I see that if condition added in > 462ad8a6cae22bc2855c8e4461ca503d6fd15884, > > + if (!offer->property("X-Plasma-API").toString().isEmpty() && > + offer->property("Library").toString().isEmpty()) { > > is removed in ea924b14691da3819ca17d876f63ffc686fcf840 as I think > there is no replacement for this in KPluginMetadata or something.. > > If we want to make it backwards compatible this case should be handled > somehow or otherwise any other desktop files which are ported to QML > and have stray X-KDE-Library needs fixing.
There's not reason why this can't be done (in a similar way) with KPluginMetaData, you can go through KPluginMetaData::rawData() to look up these keys. -- sebas Sebastian Kügler | http://vizZzion.org | http://kde.org _______________________________________________ release-team mailing list [email protected] https://mail.kde.org/mailman/listinfo/release-team
