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
