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

Dive site: explicitly init displayed_dive_site on show widget

Browse source 
The global object "displayed_dive_site" is used to store the
old dive site data for the edit-dive-site widget. The fields
of the widget were initialized from this object in the show
event. Therefore the object was updated in numerous parts of
the code to make sure that it was up-to-date. Instead, move
the initialization of the object to the function that also
initiatlizes the fields. Call this function explicitly before
showing the widget.

This makes the data-fow distinctly easier to understand.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2018-10-13 08:57:46 +02:00 • committed by Dirk Hohndel
parent f0e349cdd6
commit 954820aa81
6 changed files with 11 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

8
desktop-widgets/locationinformation.cpp
View file

@ -200,7 +200,6 @@ void LocationInformationWidget::acceptChanges()
LocationInformationModel::instance()->removeRow(get_divesite_idx(currentDs));
displayed_dive.dive_site_uuid = 0;
}
copy_dive_site(currentDs, &displayed_dive_site);
mark_divelist_changed(true);
resetState();
}
@ -210,9 +209,10 @@ void LocationInformationWidget::rejectChanges()
resetState();
}
void LocationInformationWidget::showEvent(QShowEvent *ev)
void LocationInformationWidget::initFields(dive_site *ds)
{
if (displayed_dive_site.uuid) {
if (ds) {
copy_dive_site(ds, &displayed_dive_site);
filter_model.set(displayed_dive_site.uuid, displayed_dive_site.latitude, displayed_dive_site.longitude);
updateLabels();
enableLocationButtons(dive_site_has_gps_location(&displayed_dive_site));
@ -221,11 +221,11 @@ void LocationInformationWidget::showEvent(QShowEvent *ev)
if (m)
m->invalidate();
} else {
clear_dive_site(&displayed_dive_site);
filter_model.set(0, degrees_t{ 0 }, degrees_t{ 0 });
clearLabels();
}
MapWidget::instance()->prepareForGetDiveCoordinates(displayed_dive_site.uuid);
QGroupBox::showEvent(ev);
}
void LocationInformationWidget::markChangedWidget(QWidget *w)