D23758: [CopyJob] Use resolved URL in copyingDone so undo can rename back using local file

David Faure noreply at phabricator.kde.org
Fri Sep 6 17:16:26 BST 2019


dfaure created this revision.
dfaure added reviewers: broulik, ngraham.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
dfaure requested review of this revision.

REVISION SUMMARY
  In case of desktop:/ KIO which rewrites URLs, looking up the URL the trashed file got didn't work.
  This resulted in trash:/filename being recorded by the undo manager which then
  failed to restore the file as it was actually trashed to e.g. trash:/0-filename.
  
  In addition, the source URL must be resolved too, since kio_trash
  doesn't support renaming from trash to desktop URLs.
  
  BUG: 391606
  FIXED-IN: 5.62
  
  CHANGELOG: Undoing trashing files on the desktop has been fixed
  
  Thanks: to Kai-Uwe for the debugging and the initial patch; to Nate for pinging me a few times for this to happen ;)

TEST PLAN
  Extended kio_desktop's unittest: http://www.davidfaure.fr/2019/test_trash_and_undo.diff
  Did not test as user.

REPOSITORY
  R241 KIO

BRANCH
  master

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

AFFECTED FILES
  src/core/copyjob.cpp

To: dfaure, broulik, ngraham
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190906/0efcca42/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list