A recently used file can also be a remote location
BUG: 401579
Details
Details
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.
Diff Detail
Diff Detail
- Repository
- R161 KActivity Manager Service
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
Not sure what the implications are for this change, it does fix Dolphin showing folders in its
This I don't like.
KAMD stores file paths, not URIs for local files in the database.
This needs more investigation of the consequences.
Comment Actions
Alright, so we have two bugs then:
- Task Manager does QUrl("/foo/bar") which results in an invalid URL. It must use fromLocalFile and/or check for whether it's a /simple/path or sftp://proper/url
- EventSpy plugin just does toLocalFile() so recent documents on remote locations aren't stored correctly, despite Url::file() term explicitly mentioning sftp et al