From 9ba7bdb7de392bee19012c181032e8fcf025c02b Mon Sep 17 00:00:00 2001
From: Tomaz Canabrava <tomaz.canabrava@gmail.com>
Date: Tue, 8 Mar 2016 02:25:40 -0300
Subject: [PATCH 10/21] Clean divecomputermodel

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@gmail.com>
---
 qt-models/divecomputermodel.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/qt-models/divecomputermodel.cpp b/qt-models/divecomputermodel.cpp
index 51b1a76..7b87d87 100644
--- a/qt-models/divecomputermodel.cpp
+++ b/qt-models/divecomputermodel.cpp
@@ -2,7 +2,7 @@
 #include "dive.h"
 #include "divelist.h"
 
-DiveComputerModel::DiveComputerModel(QMultiMap<QString, DiveComputerNode> &dcMap, QObject *parent) : CleanerTableModel()
+DiveComputerModel::DiveComputerModel(QMultiMap<QString, DiveComputerNode> &dcMap, QObject *parent) : CleanerTableModel(parent)
 {
 	setHeaderDataStrings(QStringList() << "" << tr("Model") << tr("Device ID") << tr("Nickname"));
 	dcWorkingMap = dcMap;
@@ -78,6 +78,10 @@ Qt::ItemFlags DiveComputerModel::flags(const QModelIndex &index) const
 
 bool DiveComputerModel::setData(const QModelIndex &index, const QVariant &value, int role)
 {
+	// We should test if the role == Qt::EditRole
+	Q_UNUSED(role);
+
+	// WARN: This seems wrong - The values don't are ordered - we need a map from the Key to Index, or something.
 	QList<DiveComputerNode> values = dcWorkingMap.values();
 	DiveComputerNode node = values.at(index.row());
 	dcWorkingMap.remove(node.model, node);
-- 
2.5.4 (Apple Git-61)

