D4923: [desktop:/ KIO] Strip superfluous slashes and fixup local root url

[David Faure]

dfaure added a comment.


  KFileItem::localPath() should not ever ever be a URL like desktop:/. That would be very wrong.
  It just returns what you set in UDS_LOCAL_PATH, so it's kio_desktop you should debug, not kio ;)
  Well except for KIO::ForwardingSlaveBase, called by kio_desktop -- note that ForwardingSlaveBase::prepareUDSEntry already sets UDS_LOCAL_PATH, but ok you call insert again which replaces that value.
  
  The code looks ok to me, are you 100% sure about your diagnostic that localPath() returns desktop:/... ?

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, hein, dfaure
Cc: anthonyfieroni, 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/20170307/1a5b6201/attachment.html>