D12520: [PasteJob] Emit KDirNotify on paste

[David Faure]

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


  "perhaps fix storedPut to do that?"  --> I'm not in favour. Generally speaking, we want to do this at the highest level possible, so that batching N jobs doesn't trigger N notifications.
  
  This is e.g. why CopyJob does it, but FileCopyJob doesn't.
  
  Granted we don't have jobs which do multiple storedPuts right now (IIRC), but I can imagine a day where this is the case.

REPOSITORY
  R241 KIO

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

To: broulik, #frameworks, dfaure
Cc: dfaure, ngraham, elvisangelaccio, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180916/85a4b58e/attachment.html>