D16087: RFC: Store full URL as URI

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, hein, ivan.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  A recently used file can also be a remote location

TEST PLAN
  It took me forever to figure out where this entry actually came from :/
  Task Manager uses the URI as `QUrl` for recent documents and tries to exclude Folders which didn't work as `QUrl("/foo/bar") is not `localFile` and as such the codepath was never hit.
  
  Not sure what the implications are for this change, it does fix Dolphin showing folders in its task manager context menu, often duplicating the ones from places. Right-click Dolphin, chose e.g. "Downloads", and you end up with a "Downloads" entry without icon in the menu.

REPOSITORY
  R161 KActivity Manager Service

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

AFFECTED FILES
  src/service/plugins/eventspy/EventSpy.cpp

To: broulik, #plasma, hein, ivan
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181009/31fe26ec/attachment.html>