----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110922/#review34017 -----------------------------------------------------------
I don't like the duplication very much (can't the old method call the V2 method, if the only difference is the return value, which the old method would then ignore?). Otherwise OK, as it's kind of temporary anyway. It just increases the risks of someone fixing a bug in one copy and not the other, so if we can avoid the duplication, it's better. lib/konq/konq_operations.h <http://git.reviewboard.kde.org/r/110922/#comment25032> This change is BIC. Add the new flag at the end of the list. - David Faure On June 9, 2013, 10 p.m., Emmanuel Pescosta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110922/ > ----------------------------------------------------------- > > (Updated June 9, 2013, 10 p.m.) > > > Review request for Dolphin, KDE Base Apps, David Faure, and Frank Reininghaus. > > > Description > ------- > > Change the data in the model before the real renaming is done by > KonqOperations::rename(), > but when the rename operation fails, revert the data changes in the model. > > The problem is that DolphinView::slotRoleEditingFinished() changes the data > in the model *before* the actual renaming is performed by KonqOperations/KIO. > But we need this approach for the following cases: > * Immediate feedback from the users point of view (No delay between finish > renaming and DolphinView updates) > * Missing file system/dir lister signals, when there is no file system > notification system (ftp, ssh, ...) > > A lot of code in konq_operations.h and konq_operations.cpp is just copy and > paste, to guarantee binary compatibility. (added some TODOs for KF 5.0) > > > This addresses bug 319119. > http://bugs.kde.org/show_bug.cgi?id=319119 > > > Diffs > ----- > > dolphin/src/views/dolphinview.h 5a70c55 > dolphin/src/views/dolphinview.cpp 9a4b863 > lib/konq/konq_operations.h a9aec89 > lib/konq/konq_operations.cpp cbb058c > > Diff: http://git.reviewboard.kde.org/r/110922/diff/ > > > Testing > ------- > > Works for me. > > Tested with a remote ssh file system on a virtual machine. > > > Thanks, > > Emmanuel Pescosta > >
