D6413: Use KIO::rename instead of KIO::moveAs in setData

David Faure noreply at phabricator.kde.org
Thu Jun 29 08:45:12 UTC 2017


dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  Yep, using `KIO::rename` to perform a renaming sounds fully correct ;-)
  
  One possible downside would be a kioslave that doesn't implement SlaveBase::rename, but that's unlikely (especially, unlikely to be triggered from here, which is typically only using "file:" and "desktop:").

REPOSITORY
  R241 KIO

BRANCH
  master

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

To: hein, #frameworks, dfaure, davidedmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170629/a11ca767/attachment.html>


More information about the Kde-frameworks-devel mailing list