Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
mirror of https://github.com/clementine-player/Clementine synced 2025-01-11 09:24:50 +01:00
Code Issues Releases Wiki Activity

Fix a crash when changing 'group by' while album covers are still loading

Browse Source
This commit is contained in:
Arnaud Bienner 2015-07-17 22:45:30 +02:00
parent 85858d7bbe
commit 59589494ae
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/globalsearch/globalsearchview.cpp
View File

@ -560,6 +560,11 @@ void GlobalSearchView::GroupByClicked(QAction* action) {
} }
void GlobalSearchView::SetGroupBy(const LibraryModel::Grouping& g) { void GlobalSearchView::SetGroupBy(const LibraryModel::Grouping& g) {
// Clear requests: changing "group by" on the models will cause all the items to be removed/added
// again, so all the QModelIndex here will become invalid. New requests will be created for those
// songs when they will be displayed again anyway (when GlobalSearchItemDelegate::paint will call
// LazyLoadArt)
art_requests_.clear();
// Update the models // Update the models
front_model_->SetGroupBy(g, true); front_model_->SetGroupBy(g, true);
back_model_->SetGroupBy(g, false); back_model_->SetGroupBy(g, false);