owen/lms
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

[UI] Corrected crash when deleting multiple tracks from play queue

Browse Source
This commit is contained in:
emeric
2015-08-05 12:31:57 +02:00
parent 0742e1d681
commit 2a7f5ae886
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ui/audio/desktop/PlayQueue.cpp
+10 -5
View File
@@ -501,12 +501,17 @@ PlayQueue::delSelected(void)
int minId = _model->rowCount(); int minId = _model->rowCount();
Wt::WModelIndexSet indexSet = this->selectedIndexes(); Wt::WModelIndexSet indexSet = this->selectedIndexes();
// Make sure to delete entries in the reverse order
std::vector<int> rowIds;
for (Wt::WModelIndex index : indexSet) for (Wt::WModelIndex index : indexSet)
{ rowIds.push_back(index.row());
_model->removeRow(index.row());
if (index.row() < minId) std::sort(rowIds.begin(), rowIds.end(), std::greater<int>());
minId = index.row(); if (!rowIds.empty())
} minId = rowIds.back();
for (int rowId : rowIds)
_model->removeRow(rowId);
// If the current played track is removed, make sure to unselect it // If the current played track is removed, make sure to unselect it
_trackSelector->setSize(_model->rowCount()); _trackSelector->setSize(_model->rowCount());