Filter: allow filtering multiple dive sites

In the edit-dive-site tab the filter is switched to a particular
mode where only dives at that site are shown.

If we want to reuse this for the dive-site tab the mode has to
be extended to allow for multiple dive sites. This is trivially
done by replacing a pointer by a vector of pointers.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2019-04-12 15:47:41 +02:00 committed by Dirk Hohndel
parent 5aacd73a00
commit afde4dce0d
3 changed files with 15 additions and 15 deletions

View file

@ -184,7 +184,7 @@ void LocationInformationWidget::acceptChanges()
MainWindow::instance()->setApplicationState("Default");
MapWidget::instance()->endGetDiveCoordinates();
MapWidget::instance()->repopulateLabels();
MultiFilterSortModel::instance()->stopFilterDiveSite();
MultiFilterSortModel::instance()->stopFilterDiveSites();
}
void LocationInformationWidget::initFields(dive_site *ds)
@ -195,7 +195,7 @@ void LocationInformationWidget::initFields(dive_site *ds)
updateLabels();
enableLocationButtons(dive_site_has_gps_location(ds));
QSortFilterProxyModel *m = qobject_cast<QSortFilterProxyModel *>(ui.diveSiteListView->model());
MultiFilterSortModel::instance()->startFilterDiveSite(ds);
MultiFilterSortModel::instance()->startFilterDiveSites(QVector<dive_site *>{ ds });
if (m)
m->invalidate();
} else {

View file

@ -105,8 +105,7 @@ MultiFilterSortModel *MultiFilterSortModel::instance()
}
MultiFilterSortModel::MultiFilterSortModel(QObject *parent) : QSortFilterProxyModel(parent),
divesDisplayed(0),
curr_dive_site(NULL)
divesDisplayed(0)
{
setFilterKeyColumn(-1); // filter all columns
setFilterCaseSensitivity(Qt::CaseInsensitive);
@ -125,9 +124,9 @@ void MultiFilterSortModel::resetModel(DiveTripModelBase::Layout layout)
bool MultiFilterSortModel::showDive(const struct dive *d) const
{
// If curr_dive_site is set, we are in a special dive-site editing mode.
if (curr_dive_site)
return d->dive_site == curr_dive_site;
// If dive_sites is not empty, we are in a special dive-site filtering mode.
if (!dive_sites.isEmpty())
return dive_sites.contains(d->dive_site);
if (!filterData.validFilter)
return true;
@ -245,7 +244,7 @@ void MultiFilterSortModel::myInvalidate()
emit filterFinished();
#if !defined(SUBSURFACE_MOBILE)
if (curr_dive_site)
if (!dive_sites.isEmpty())
MainWindow::instance()->diveList->expandAll();
#endif
}
@ -267,15 +266,15 @@ void MultiFilterSortModel::clearFilter()
myInvalidate();
}
void MultiFilterSortModel::startFilterDiveSite(struct dive_site *ds)
void MultiFilterSortModel::startFilterDiveSites(QVector<dive_site *> ds)
{
curr_dive_site = ds;
dive_sites = ds;
myInvalidate();
}
void MultiFilterSortModel::stopFilterDiveSite()
void MultiFilterSortModel::stopFilterDiveSites()
{
curr_dive_site = NULL;
dive_sites.clear();
myInvalidate();
}

View file

@ -67,8 +67,8 @@ public
slots:
void myInvalidate();
void clearFilter();
void startFilterDiveSite(struct dive_site *ds);
void stopFilterDiveSite();
void startFilterDiveSites(QVector<dive_site *> ds);
void stopFilterDiveSites();
void filterChanged(const QModelIndex &from, const QModelIndex &to, const QVector<int> &roles);
void resetModel(DiveTripModelBase::Layout layout);
void filterDataChanged(const FilterData &data);
@ -80,7 +80,8 @@ signals:
private:
MultiFilterSortModel(QObject *parent = 0);
struct dive_site *curr_dive_site;
// Dive site filtering has priority over other filters
QVector<dive_site *> dive_sites;
void countsChanged();
FilterData filterData;
};