feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

Examples/Marketplace: Move ownership to the "all" products model

Browse Source 
To allow the same item to appear in multiple categories.

Change-Id: Ib957b1fb0d98a28ceeb1d80e9fb497cc2a6b1f48
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: David Schulz <david.schulz@qt.io>
This commit is contained in:
Eike Ziller
2023-03-09 15:22:24 +01:00
parent 0c4dff7c10
commit a98c45f9ba
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/plugins/coreplugin/welcomepagehelper.cpp
View File

@@ -637,8 +637,6 @@ SectionedGridView::SectionedGridView(QWidget *parent)
{
auto allItemsModel = new ListModel(this);
allItemsModel->setPixmapFunction(m_pixmapFunction);
// it just "borrows" the items from the section models:
allItemsModel->setOwnsItems(false);
m_filteredAllItemsModel = new Core::ListModelFilter(allItemsModel, this);
auto area = new QScrollArea(this);
@@ -690,6 +688,10 @@ ListModel *SectionedGridView::addSection(const Section &section, const QList<Lis
{
auto model = new ListModel(this);
model->setPixmapFunction(m_pixmapFunction);
// the sections only keep a weak reference to the items,
// they are owned by the allProducts model, since multiple sections can contain duplicates
// of the same item
model->setOwnsItems(false);
model->appendItems(items);
auto gridView = new SectionGridView(this);
@@ -716,7 +718,11 @@ ListModel *SectionedGridView::addSection(const Section &section, const QList<Lis
// add the items also to the all products model to be able to search correctly
auto allProducts = static_cast<ListModel *>(m_filteredAllItemsModel->sourceModel());
allProducts->appendItems(items);
const QSet<ListItem *> allItems = toSet(allProducts->items());
const QList<ListItem *> newItems = filtered(items, [&allItems](ListItem *item) {
return !allItems.contains(item);
});
allProducts->appendItems(newItems);
// only show section label(s) if there is more than one section
m_sectionLabels.at(0)->setVisible(m_sectionLabels.size() > 1);