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

Reply via email to