D7567: Support edit and appstream actions also for application search results

[Matthias Klumpp]

mak added inline comments.

INLINE COMMENTS

> actionlist.cpp:380
> +
> +    const auto components = appstreamPool->componentsById(service->desktopEntryName()+QLatin1String(".desktop"));
> +    for(const auto &component: components) {

This will not work, because the component-ID can be an arbitrary reverse-DNS string identifying the application, and doesn't have to resemble the .desktop-entry-id at all. If that happens, it's purely accidental.

I was planning to add a `componentsByLaunchable(TYPE, STR)` method to AppStream anyway, I could probably do that for the next release of AS.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D7567

To: hein, #plasma, broulik, davidedmundson
Cc: mak, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170828/3b889c00/attachment-0001.html>