Skip to content

[kio][383502] Fixing Apply To All when copying only one folder

Andreas Bontozoglou requested to merge (removed):fix_kio_383502 into master

BUG: 383502

tl;dr: Not showing "Apply to All" checkbox, nor "Skip" button in the RenameDialog when only one folder is being moved.

It seems that RenameDialog_MultipleItems and RenameDialog_Skip can be used to define this behavior. The caller (CopyJobPrivate) was always setting these options without checking the number of dirs.

NOTES/WARNINGS/QUESTIONS:

  • Still very n00b so atm just hacking :) far from understanding the full code flow there
  • Not sure if I need to update any unittests
  • There was a comment about "always multi" which I think is wrong and I removed

Examples:

Single folder copy:
copy_single_folder

Single folder move:
move_single_folder

Multiple folders copy, one of which conflicts:
copy_multiple_folders

Multiple folders with a file conflict:
copy_multiple_folders_file_conflict

Merge request reports