broulik created this revision.
broulik added reviewers: Frameworks, dfaure.
Restricted Application added a project: Frameworks.
broulik 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`
  
  BUG: 391606
  
  CHANGELOG: Undoing trashing files on the desktop has been fixed

TEST PLAN
  I always looked in the wrong place, the undo manager code was all sane it was 
just that we tried to restore (using the direct rename shortcut) a file that 
didn't exist :(
  
  Trashed a file on `desktop:/`, hit Ctrl+Z, file showed up again immediately.

REPOSITORY
  R241 KIO

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

AFFECTED FILES
  src/core/copyjob.cpp

To: broulik, #frameworks, dfaure
Cc: michaelh, ngraham

Reply via email to