Details
- Reviewers
elvisangelaccio - Group Reviewers
Dolphin - Commits
- R318:c5eb4e31161c: Use BatchRenameJob to rename multiple files in Dolphin
Diff Detail
- Repository
- R318 Dolphin
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
src/views/renamedialog.cpp | ||
---|---|---|
173 | KFileItemList has a urlList() method that does that | |
177–178 | The job is async, so by the time you end up here the job hasn't even started yet, your handling should be in slotFileRenamed or rather connect to result signal? | |
178 | Do we need QUrl::StripTrailingSlash here? At least most other places in KIO that use RemoveFileName set it |
src/views/renamedialog.cpp | ||
---|---|---|
178 | When undoing BatchRename parentUrl is not used. So IMO setting QUrl::StripTrailingSlash won't make any difference. |
Nice. Tested, and it works as expected, including undo, as advertised. So that's nice. +1 from me, but I'll wait for someone more experienced to do a code review.
Looks good, but please bump the minimum KF5 version in CMakeLists.txt to 5.42
src/views/renamedialog.cpp | ||
---|---|---|
201–202 | Nitpick, semicolon not necessary. |
@chinmoyr Friendly ping! If we can get this in before tomorrow, I'll be able to highlight it in my weekly Usability and Productivity status report. :)