]> cloud.milkyroute.net Git - dolphin.git/commit
Always clear DolphinView's m_currentItemUrl member in updateViewState()
authorFrank Reininghaus <frank78ac@googlemail.com>
Tue, 12 Aug 2014 07:17:41 +0000 (09:17 +0200)
committerFrank Reininghaus <frank78ac@googlemail.com>
Tue, 12 Aug 2014 07:17:41 +0000 (09:17 +0200)
commitbf1ac8ee9b2edcf9e90f74bae7f392c1b3c318b2
tree5d6b00138557b353711a3864f6d1903480f79e4b
parent4ed068fffff2343cc0dcd236c38832b7662f8d78
Always clear DolphinView's m_currentItemUrl member in updateViewState()

Before this patch, we still kept the URL in m_currentItemUrl if the URL
was not found in the model. This could cause problems the next time
updateViewState() was called, because the current index would then be
set to 0, which could cause other issues.

For example, all items between the first item in the view and a folder
which was expanded in Details View could be selected.

In principle, it is possible that updateViewState() is called multiple
times if many large files are being pasted in the view, but since the
item which should be made the current item (and which the view should
be scrolled to) should always be the first pasted item, this change
will most likely not cause any other problems.

BUG: 329377
REVIEW: 119703
FIXED-IN: 4.14.0
src/views/dolphinview.cpp