Desktop: Use DiveImportedModel::deleteDeselected()

In DownloadFromDCWidget::on_ok_clicked() deselected dives were
directly deleted from the dive table, leaving DiveImportedModel
in an inconsistent state. Use the function in DiveImportedModel
instead. This also removes code duplication.

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

1 files changed, 1 insertions, 8 deletions

diff --git a/desktop-widgets/downloadfromdivecomputer.cpp b/desktop-widgets/downloadfromdivecomputer.cpp
index 129870727..1e957808f 100644
--- a/desktop-widgets/downloadfromdivecomputer.cpp
+++ b/desktop-widgets/downloadfromdivecomputer.cpp
@@ -529,14 +529,7 @@ void DownloadFromDCWidget::on_ok_clicked()
 	struct dive_site_table *sites = diveImportedModel->thread.sites();
 
 	// delete non-selected dives
-	int total = table->nr;
-	int j = 0;
-	for (int i = 0; i < total; i++) {
-		if (diveImportedModel->data(diveImportedModel->index(i, 0), Qt::CheckStateRole) == Qt::Checked)
-			j++;
-		else
-			delete_dive_from_table(diveImportedModel->thread.table(), j);
-	}
+	diveImportedModel->deleteDeselected();
 
 	if (table->nr > 0) {
 		auto data = diveImportedModel->thread.data();