Dive site: add edit field to dive site table

Add an edit column that calls the new editDiveSite() function
of MainWindow. The calling code is in DiveSiteSortedModel.
Quite illogical, but that's how TableView works, for now.

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

2 files changed, 18 insertions, 11 deletions

diff --git a/qt-models/divelocationmodel.cpp b/qt-models/divelocationmodel.cpp
index c18f85932..b01ae03b6 100644
--- a/qt-models/divelocationmodel.cpp
+++ b/qt-models/divelocationmodel.cpp
@@ -6,7 +6,7 @@
 #include "core/divesite.h"
 #include "core/metrics.h"
 #ifndef SUBSURFACE_MOBILE
-#include "cleanertablemodel.h" // for trashIcon();
+#include "cleanertablemodel.h" // for trashIcon() and editIcon()
 #include "desktop-widgets/mainwindow.h" // to place message box
 #include "desktop-widgets/command.h"
 #include <QMessageBox>
@@ -100,12 +100,14 @@ QVariant LocationInformationModel::getDiveSiteData(const struct dive_site *ds, i
 	break;
 	case Qt::ToolTipRole:
 		switch(column) {
+		case EDIT: return tr("Click here to edit the divesite.");
 		case REMOVE: return tr("Clicking here will remove this divesite.");
 		}
 	break;
 	case Qt::DecorationRole:
 		switch(column) {
 #ifndef SUBSURFACE_MOBILE
+		case EDIT: return editIcon();
 		case REMOVE: return trashIcon();
 #endif
 		case NAME: return dive_site_has_gps_location(ds) ? QIcon(":geotag-icon") : QVariant();
@@ -238,21 +240,26 @@ bool DiveSiteSortedModel::setData(const QModelIndex &index, const QVariant &valu
 	}
 }
 
-// TODO: Remove from model. It doesn't make sense to call the model here, which calls the undo command,
+// TODO: Remove or edit. It doesn't make sense to call the model here, which calls the undo command,
 // which in turn calls the model.
 void DiveSiteSortedModel::remove(const QModelIndex &index)
 {
-	if (index.column() != LocationInformationModel::REMOVE)
-		return;
 	struct dive_site *ds = getDiveSite(index);
 	if (!ds)
 		return;
-	if (ds->dives.nr > 0 &&
-	    QMessageBox::warning(MainWindow::instance(), tr("Delete dive site?"),
-				 tr("This dive site has %n dive(s). Do you really want to delete it?\n", "", ds->dives.nr),
-				 QMessageBox::Yes|QMessageBox::No) == QMessageBox::No)
-			return;
-	Command::deleteDiveSites(QVector<dive_site *>{ds});
+	switch (index.column()) {
+	case LocationInformationModel::EDIT:
+		MainWindow::instance()->editDiveSite(ds);
+		break;
+	case LocationInformationModel::REMOVE:
+		if (ds->dives.nr > 0 &&
+		    QMessageBox::warning(MainWindow::instance(), tr("Delete dive site?"),
+					 tr("This dive site has %n dive(s). Do you really want to delete it?\n", "", ds->dives.nr),
+					 QMessageBox::Yes|QMessageBox::No) == QMessageBox::No)
+				return;
+		Command::deleteDiveSites(QVector<dive_site *>{ds});
+		break;
+	}
 }
 #endif // SUBSURFACE_MOBILE
 
diff --git a/qt-models/divelocationmodel.h b/qt-models/divelocationmodel.h
index 60c5c993a..bca833a06 100644
--- a/qt-models/divelocationmodel.h
+++ b/qt-models/divelocationmodel.h
@@ -17,7 +17,7 @@ class LocationInformationModel : public QAbstractTableModel {
 public:
 	// Common columns, roles and accessor function for all dive-site models.
 	// Thus, different views can connect to different models.
-	enum Columns { REMOVE, NAME, DESCRIPTION, NUM_DIVES, LOCATION, NOTES, DIVESITE, TAXONOMY, COLUMNS};
+	enum Columns { EDIT, REMOVE, NAME, DESCRIPTION, NUM_DIVES, LOCATION, NOTES, DIVESITE, TAXONOMY, COLUMNS };
 	enum Roles { DIVESITE_ROLE = Qt::UserRole + 1 };
 	static QVariant getDiveSiteData(const struct dive_site *ds, int column, int role);