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