Review Request: Clear the previous selection when pasting or dropping items (fixing a recent 4.9 branch regression)

[Frank Reininghaus]

> On Nov. 20, 2012, 5:46 p.m., Emmanuel Pescosta wrote:
> > dolphin/src/views/dolphinview.cpp, line 1545
> > <http://git.reviewboard.kde.org/r/107389/diff/1/?file=95413#file95413line1545>
> >
> >     What if we move this line to DolphinView::pasteToUrl(const KUrl& url)? - So your patch needs no additional adjustment when I submit my patch 107351 ;)

Thanks for taking the time to test this! Just moving to pasteUrl(KUrl) would not work because this also applies to the drag&drop use case. So it has to be moved to two places to make it work properly, which probably makes sense considering that DolphinView::markPastedUrlsAsSelected() won't even be needed any more if you include the drag&drop use case in your request 107351.


- Frank


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


On Nov. 19, 2012, 9:02 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107389/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2012, 9:02 p.m.)
> 
> 
> Review request for Dolphin and Emmanuel Pescosta.
> 
> 
> Description
> -------
> 
> To reduce the risk of regressions, I thought I'd better let other people have a look at non-trivial patches of mine before pushing. In particular, selecting items is an area where a lot can go wrong. Emmanuel, can you have a look at the patch? You're doing some work in the area at the moment, so maybe you see if that breaks something or if there's an easier way to do it (I doubt that we can avoid the new member though if we don't want to bring bug 217575 back though). This fix is mostly independent of your patch https://git.reviewboard.kde.org/r/107351/. That one should also go in, of course - I'm uploading the fix for the regression now because I want to fix it before KDE 4.9.4 is tagged.
> 
> Others are of course also welcome to comment on the patch!
> 
> 
> This addresses bug 310365.
>     http://bugs.kde.org/show_bug.cgi?id=310365
> 
> 
> Diffs
> -----
> 
>   dolphin/src/views/dolphinview.h 7d8e8b7 
>   dolphin/src/views/dolphinview.cpp 13db989 
> 
> Diff: http://git.reviewboard.kde.org/r/107389/diff/
> 
> 
> Testing
> -------
> 
> Fixes the bug for me, and I haven't found any new regressions so far.
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

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