Commit 4f6d95e5 authored by Christian Ehrlicher's avatar Christian Ehrlicher Committed by Luigi Toscano
Browse files

Sort List by piece count does not work

Summary:
The sorting in the overview dialog did not work because the
QSortFilterProxyModel was not set up correctly

BUG: 376607

Test Plan:
Test if the sorting works by selecting the appropriate sorting

Reviewers: #kde_games, ltoscano

Reviewed By: ltoscano

Subscribers: ltoscano, #kde_games

Differential Revision: https://phabricator.kde.org/D7578

(cherry picked from commit 2a78d0bd)
parent 5f75a539
......@@ -57,7 +57,10 @@ Palapeli::CollectionView::CollectionView(QWidget* parent)
m_view->setModel(m_proxyModel);
connect(m_view->selectionModel(), SIGNAL(selectionChanged(QItemSelection,QItemSelection)), SLOT(handleSelectionChanged()));
m_proxyModel->setDynamicSortFilter(true);
m_proxyModel->sort(Qt::DisplayRole, Qt::AscendingOrder);
m_proxyModel->setSortCaseSensitivity(Qt::CaseInsensitive);
m_proxyModel->setSortLocaleAware(true);
m_proxyModel->setSortRole(Qt::DisplayRole);
m_proxyModel->sort(0, Qt::AscendingOrder);
//TODO: save sorting role between sessions
//setup filter search line
KFilterProxySearchLine* searchLine = new KFilterProxySearchLine(this);
......@@ -126,7 +129,7 @@ void Palapeli::CollectionView::sortMenuTriggered(QAction* action)
if (action == m_sortByPieceCount)
sortRole = Palapeli::Collection::PieceCountRole;
//update sorting and menu
m_proxyModel->sort(sortRole, Qt::AscendingOrder);
m_proxyModel->setSortRole(sortRole);
m_sortByTitle->setChecked(sortRole == Qt::DisplayRole);
m_sortByPieceCount->setChecked(sortRole == Palapeli::Collection::PieceCountRole);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment