mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

QML UI: refresh the dive list after edit

Browse source 
This fixes two issues. In general, after edits the dive list wasn't updated so
it showed data inconsistent with what the dive details showed (clearly bogus).
Even more annoyingly, when we change the date or time of a dive it could
obviously move around in the dive list. So we need to resort the dive table and
recreate the dive list. For really long dive lists this is possibly overkill,
but in my testing this seemed very quick and much easier than trying to
manually get this right, even in the case where the list wasn't resorted.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2016-01-05 22:57:40 -08:00
parent e774c8077b
commit 4db5e840bf
1 changed files with 10 additions and 3 deletions

13
qt-mobile/qmlmanager.cpp
View file

@ -314,9 +314,10 @@ void QMLManager::commitChanges(QString diveId, QString date, QString location, Q
return;
}
bool diveChanged = false;
bool needResort = false;
if (date != get_dive_date_string(d->when)) {
diveChanged = true;
needResort = true;
QDateTime newDate;
// what a pain - Qt will not parse dates if the day of the week is incorrect
// so if the user changed the date but didn't update the day of the week (most likely behavior, actually),
@ -438,8 +439,14 @@ void QMLManager::commitChanges(QString diveId, QString date, QString location, Q
free(d->notes);
d->notes = strdup(qPrintable(notes));
}
if (diveChanged) {
DiveListModel::instance()->updateDive(d);
if (needResort)
sort_table(&dive_table);
if (diveChanged || needResort) {
int i;
DiveListModel::instance()->clear();
for_each_dive(i, d) {
DiveListModel::instance()->addDive(d);
}
mark_divelist_changed(true);
}
}