From 0eae59bcd29ba1242e26e8e0c4f52c12f790e521 Mon Sep 17 00:00:00 2001
From: Dirk Hohndel <dirk@hohndel.org>
Date: Fri, 29 Jan 2016 06:47:27 -0800
Subject: QML UI: introduce "add" state and correctly clean up when canceled

Adding a dive is just like editing it, except that canceling the operation
has different consequences. Instead of trying to figure this out by some
inference on other state, let's just make it explicit and then clean up
after ourselves if the user canceled a manual dive add.

This also switches to use the properties that we defined in order for the
main menu to be able to setup these values. Makes the code easier to read
and is more consistent.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
---
 qt-mobile/qml/DiveDetails.qml | 44 ++++++++++++++++++++++++++++---------------
 qt-mobile/qml/main.qml        |  2 +-
 2 files changed, 30 insertions(+), 16 deletions(-)

(limited to 'qt-mobile')

diff --git a/qt-mobile/qml/DiveDetails.qml b/qt-mobile/qml/DiveDetails.qml
index 8460a7eec..77d52c77e 100644
--- a/qt-mobile/qml/DiveDetails.qml
+++ b/qt-mobile/qml/DiveDetails.qml
@@ -35,28 +35,42 @@ MobileComponents.Page {
 			name: "edit"
 			PropertyChanges { target: diveDetailList; visible: false }
 			PropertyChanges { target: detailsEditScroll; visible: true }
+		},
+		State {
+			name: "add"
+			PropertyChanges { target: diveDetailList; visible: false }
+			PropertyChanges { target: detailsEditScroll; visible: true }
 		}
+
 	]
 	mainAction: Action {
-		iconName: state === "edit" ? "dialog-cancel" : "document-edit"
+		iconName: state !== "view" ? "dialog-cancel" : "document-edit"
 		onTriggered: {
 			if (state === "edit") {
+				// just cancel the edit state
+				state = "view"
+			} else if (state === "add") {
+				// edit was canceled - so remove the dive from the dive list
+				manager.addDiveAborted(dive_id)
 				state = "view"
-				return
+			} else {
+				// set things up for editing - so make sure that the detailsEdit has
+				// all the right data (using the property aliases set up above)
+				dive_id = diveDetailsListView.currentItem.modelData.dive.id
+				number = diveDetailsListView.currentItem.modelData.dive.number
+				date = diveDetailsListView.currentItem.modelData.dive.date + " " + diveDetailsListView.currentItem.modelData.dive.time
+				location = diveDetailsListView.currentItem.modelData.dive.location
+				duration = diveDetailsListView.currentItem.modelData.dive.duration
+				depth = diveDetailsListView.currentItem.modelData.dive.depth
+				airtemp = diveDetailsListView.currentItem.modelData.dive.airTemp
+				watertemp = diveDetailsListView.currentItem.modelData.dive.waterTemp
+				suit = diveDetailsListView.currentItem.modelData.dive.suit
+				buddy = diveDetailsListView.currentItem.modelData.dive.buddy
+				divemaster = diveDetailsListView.currentItem.modelData.dive.divemaster
+				notes = diveDetailsListView.currentItem.modelData.dive.notes
+				weight = diveDetailsListView.currentItem.modelData.dive.sumWeight
+				diveDetailsPage.state = "edit"
 			}
-			detailsEdit.dive_id = diveDetailsListView.currentItem.modelData.dive.id
-			detailsEdit.number = diveDetailsListView.currentItem.modelData.dive.number
-			detailsEdit.dateText = diveDetailsListView.currentItem.modelData.dive.date + " " + diveDetailsListView.currentItem.modelData.dive.time
-			detailsEdit.locationText = diveDetailsListView.currentItem.modelData.dive.location
-			detailsEdit.durationText = diveDetailsListView.currentItem.modelData.dive.duration
-			detailsEdit.depthText = diveDetailsListView.currentItem.modelData.dive.depth
-			detailsEdit.airtempText = diveDetailsListView.currentItem.modelData.dive.airTemp
-			detailsEdit.watertempText = diveDetailsListView.currentItem.modelData.dive.waterTemp
-			detailsEdit.suitText = diveDetailsListView.currentItem.modelData.dive.suit
-			detailsEdit.buddyText = diveDetailsListView.currentItem.modelData.dive.buddy
-			detailsEdit.divemasterText = diveDetailsListView.currentItem.modelData.dive.divemaster
-			detailsEdit.notesText = diveDetailsListView.currentItem.modelData.dive.notes
-			diveDetailsPage.state = "edit"
 		}
 	}
 
diff --git a/qt-mobile/qml/main.qml b/qt-mobile/qml/main.qml
index 9c4a36c43..8a5347967 100644
--- a/qt-mobile/qml/main.qml
+++ b/qt-mobile/qml/main.qml
@@ -66,7 +66,7 @@ MobileComponents.ApplicationWindow {
 				Action {
 					text: "Add dive manually"
 					onTriggered: {
-						detailsWindow.state = "edit"
+						detailsWindow.state = "add"
 						detailsWindow.dive_id = manager.addDive();
 						detailsWindow.number = manager.getNumber(detailsWindow.dive_id)
 						detailsWindow.date = manager.getDate(detailsWindow.dive_id)
-- 
cgit v1.2.3-70-g09d2