davidedmundson created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  When previewing files in a KIOslave KIO will copy the file out to a
  temporary directory and then run the preview on that.
  
  If I have a symlink in a slave (for example trashing a symlink) the
  preview job will copy the symlink to the temporary directory.
  
  QFileInfo follows symlinks for methods like isDir(), in this case we
  shouldn't assert as QFile::remove will correctly just cleanup the
  symlink.

TEST PLAN
  Created a symlink to a directory
  Trashed it
  Opened trash:// in dolphin with previews enabled
  Used to hit an assert
  Now don't

REPOSITORY
  R241 KIO

BRANCH
  master

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

AFFECTED FILES
  src/widgets/previewjob.cpp

To: davidedmundson
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to