D9107: Add undo support to BatchRenameJob

David Faure noreply at phabricator.kde.org
Sat Dec 2 14:46:44 UTC 2017


dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.


  Good stuff.

INLINE COMMENTS

> fileundomanagertest.cpp:697
> +
> +    createTestFile(srcList[0].path(), "foo");
> +    createTestFile(srcList[1].path(), "foo");

use .at(0) to avoid detaching

> fileundomanager.h:136
>       *                Used when pasting data from clipboard or drag-n-drop.
>       * Mkpath: @since 5.4, represents a KIO::mkpath() job.
>       */

add docu for BatchRename

REPOSITORY
  R241 KIO

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

To: chinmoyr, #frameworks, dfaure
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171202/9ae9aacb/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list