mirror of
https://github.com/subsurface/subsurface.git
synced 2024-12-03 15:43:09 +00:00
f8a3a85210
It's entirely unclear why std::sort sometimes accesses the element with index -1. In my limited testing switching to qSort avoids that. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
91 lines
2.3 KiB
C++
91 lines
2.3 KiB
C++
#include "divelocationmodel.h"
|
|
#include "dive.h"
|
|
#include <QDebug>
|
|
|
|
bool dive_site_less_than(dive_site *a, dive_site *b)
|
|
{
|
|
return QString(a->name) <= QString(b->name);
|
|
}
|
|
|
|
LocationInformationModel *LocationInformationModel::instance()
|
|
{
|
|
static LocationInformationModel *self = new LocationInformationModel();
|
|
return self;
|
|
}
|
|
|
|
LocationInformationModel::LocationInformationModel(QObject *obj) : QAbstractListModel(obj), internalRowCount(0)
|
|
{
|
|
}
|
|
|
|
int LocationInformationModel::rowCount(const QModelIndex &parent) const
|
|
{
|
|
Q_UNUSED(parent);
|
|
return internalRowCount;
|
|
}
|
|
|
|
QVariant LocationInformationModel::data(const QModelIndex &index, int role) const
|
|
{
|
|
if (!index.isValid())
|
|
return QVariant();
|
|
struct dive_site *ds = get_dive_site(index.row());
|
|
|
|
if (!ds)
|
|
return QVariant();
|
|
|
|
switch(role) {
|
|
case Qt::DisplayRole : return qPrintable(ds->name);
|
|
case DIVE_SITE_UUID : return ds->uuid;
|
|
}
|
|
|
|
return QVariant();
|
|
}
|
|
|
|
void LocationInformationModel::update()
|
|
{
|
|
beginResetModel();
|
|
internalRowCount = dive_site_table.nr;
|
|
qSort(dive_site_table.dive_sites, dive_site_table.dive_sites + dive_site_table.nr, dive_site_less_than);
|
|
endResetModel();
|
|
}
|
|
|
|
int32_t LocationInformationModel::addDiveSite(const QString& name, int lon, int lat)
|
|
{
|
|
degrees_t latitude, longitude;
|
|
latitude.udeg = lat;
|
|
longitude.udeg = lon;
|
|
|
|
beginInsertRows(QModelIndex(), dive_site_table.nr, dive_site_table.nr);
|
|
uint32_t uuid = create_dive_site_with_gps(name.toUtf8().data(), latitude, longitude);
|
|
qSort(dive_site_table.dive_sites, dive_site_table.dive_sites + dive_site_table.nr, dive_site_less_than);
|
|
internalRowCount = dive_site_table.nr;
|
|
endInsertRows();
|
|
return uuid;
|
|
}
|
|
|
|
bool LocationInformationModel::setData(const QModelIndex &index, const QVariant &value, int role)
|
|
{
|
|
if (!index.isValid())
|
|
return false;
|
|
|
|
if (role != Qt::EditRole)
|
|
return false;
|
|
|
|
struct dive_site *ds = get_dive_site(index.row());
|
|
free(ds->name);
|
|
ds->name = copy_string(qPrintable(value.toString()));
|
|
emit dataChanged(index, index);
|
|
return true;
|
|
}
|
|
|
|
bool LocationInformationModel::removeRows(int row, int count, const QModelIndex & parent) {
|
|
if(row >= rowCount())
|
|
return false;
|
|
|
|
beginRemoveRows(QModelIndex(), row, row);
|
|
struct dive_site *ds = get_dive_site(row);
|
|
if (ds)
|
|
delete_dive_site(ds->uuid);
|
|
internalRowCount = dive_site_table.nr;
|
|
endRemoveRows();
|
|
return true;
|
|
}
|