D16087: RFC: Store full URL as URI

2018-10-12 Thread Kai Uwe Broulik
broulik added a comment. Alright, so we have two bugs then: 1. 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` 2. EventSpy plugin just does `toLocalFile()` so recent

D16087: RFC: Store full URL as URI

2018-10-10 Thread Ivan Čukić
ivan requested changes to this revision. ivan added a comment. This revision now requires changes to proceed. > 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 da

D16087: RFC: Store full URL as URI

2018-10-10 Thread Aleix Pol Gonzalez
apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Makes sense, even the argument is URI REPOSITORY R161 KActivity Manager Service REVISION DETAIL https://phabricator.kde.org/D16087 To: broulik, #plasma, hein, ivan, apol Cc: apol, plasma-d

D16087: RFC: Store full URL as URI

2018-10-09 Thread 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 f