Review Request 110909: DO not rename the wrong item when items are added to the view while renaming

Frank Reininghaus frank78ac at googlemail.com
Sun Jun 9 11:23:37 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110909/
-----------------------------------------------------------

Review request for Dolphin.


Description
-------

The problem is that KItemListRoleEditor remembers the index of the item that is currently being renamed, and finally tells DolphinView to rename the item with that index. However, the index might change if new files are added to the directory.

To fix this, I propose to not tell KItemListRoleEditor about the index at all. Instead, just let the KStandardItemListWidget (which always knows about the correct index) determine the index when renaming is finished, and forward that one to DolphinView.


This addresses bug 320823.
    http://bugs.kde.org/show_bug.cgi?id=320823


Diffs
-----

  dolphin/src/kitemviews/kstandarditemlistwidget.h 386f60e 
  dolphin/src/kitemviews/kstandarditemlistwidget.cpp f0394df 
  dolphin/src/kitemviews/private/kitemlistroleeditor.h aa2c977 
  dolphin/src/kitemviews/private/kitemlistroleeditor.cpp 9f3b53c 

Diff: http://git.reviewboard.kde.org/r/110909/diff/


Testing
-------

Works for me.


Thanks,

Frank Reininghaus

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130609/29cd25ec/attachment.htm>


More information about the kfm-devel mailing list