D5234: [Folder View] Follow targetUrl() instead of url()

[Kai Uwe Broulik]

broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  This makes it consistent with Dolphin which, when cd'ing into a folder on desktop:/ actually opens /home/foo/Desktop/path (the target URL set by the KIO) and also fixes opening subfolders in my places:/ KIO which just links to the actual locations.

TEST PLAN
  Both desktop popups and panel works, places:/ works fine now.
  However, the title for files on the desktop is no longer "desktop > foo" but "Home > Desktop > foo" (which might even elide depending on language :/)
  
  From KFileItem docs
  
  > Returns the target url of the file, which is the same as url() in cases where the slave doesn't specify UDS_TARGET_URL

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  containments/desktop/plugins/folder/foldermodel.cpp

To: broulik, #plasma, hein, emmanuelp
Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170329/452aad3f/attachment.html>