Map: pass down dive site in prepareForGetDiveCoordinates()

Instead of reading out the global object "displayed_dive_site",
pass the dive site to be edited in arguments to
prepareForGetDiveCoordinates() and enter edit mode.

Simplify the code in LocationInformationWidget by not using
signals to call the prepareForGetDiveCoordinates() function.
While doing this, collect common code in accept() and reject()
in the already existing resetState() function.

This is another entry in a series of commits that makes
data-flow more clear by removing access to the global
"displayed_dive_site" object.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 9 insertions, 5 deletions

diff --git a/map-widget/qmlmapwidgethelper.cpp b/map-widget/qmlmapwidgethelper.cpp
index b892cdcea..8164abd3a 100644
--- a/map-widget/qmlmapwidgethelper.cpp
+++ b/map-widget/qmlmapwidgethelper.cpp
@@ -291,16 +291,20 @@ void MapWidgetHelper::exitEditMode()
 	emit editModeChanged();
 }
 
-void MapWidgetHelper::enterEditMode()
+void MapWidgetHelper::enterEditMode(quint32 uuid)
 {
+	// We don't support editing of a dive site that doesn't exist
+	struct dive_site *ds = get_dive_site_by_uuid(uuid);
+	if (!ds)
+		return;
+
 	m_editMode = true;
-	MapLocation *exists = m_mapLocationModel->getMapLocationForUuid(displayed_dive_site.uuid);
+	MapLocation *exists = m_mapLocationModel->getMapLocationForUuid(uuid);
 	QGeoCoordinate coord;
-	// if divesite uuid doesn't exist in the model, add a new MapLocation.
+	// if divesite doesn't exist in the model, add a new MapLocation.
 	if (!exists) {
 		coord = m_map->property("center").value<QGeoCoordinate>();
-		m_mapLocationModel->add(new MapLocation(displayed_dive_site.uuid, coord,
-							QString(displayed_dive_site.name)));
+		m_mapLocationModel->add(new MapLocation(uuid, coord, QString(ds->name)));
 	} else {
 		coord = exists->coordinate();
 	}