Review Request: Update the 'isCut' state of items in the view if, e.g., text is copied to the clipboard

[Frank Reininghaus]

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

Review request for Dolphin.


Description
-------

Right now, we only update the "isCut" state of file items when other files are copied or cut to the clipboard. The expected behaviour is to make the items look "uncut" also when some non-file thing, like, e.g., text, is cut or copied.

The complexity of the previous code which I'm proposing to remove makes me wonder if there could be any problem with this approach - does anyone see anything that might go wrong?

(Yes, I do know that KUrl::List::fromMimeData() is deprecated and that I should be using KUrlMimeData::urlsFromMimeData() instead. It seems that kdelibs doesn't install the required header here, I'll look into that, unless the reason is that I'm doing somthing wrong, in which case I'd appreciate feedback about that).


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


Diffs
-----

  dolphin/src/kitemviews/private/kfileitemclipboard.cpp 6d60856 

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


Testing
-------

Works for me, no regressions seen so far.


Thanks,

Frank Reininghaus

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