D8260: [LauncherTasksModel] Try resolving absolute path to applications: URL before adding launcher

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, hein, ngraham, mvourlakos.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  This way when a third party like pinning or dragging from Kickoff sends an absolute desktop file path we would still only store the application id if possible.

TEST PLAN
  Pinned Dolphin from Kickoff, resulted in `applications:org.kde.dolphin.desktop` being stored in plasma rc. Kickoff would still know that the item is already pinned (ie. entry no longer shown in context menu)

REPOSITORY
  R120 Plasma Workspace

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

AFFECTED FILES
  libtaskmanager/launchertasksmodel.cpp

To: broulik, #plasma, hein, ngraham, mvourlakos
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171012/f30b2de6/attachment.html>