From 9376721873f9897dcc2a451b5b8e0c56ca82f939 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Thu, 2 Sep 2021 19:11:35 +0200 Subject: [PATCH] desktop: warn when deleting a cylinder with sensor readings This makes the sensors pointless and in the future, they will be removed. Signed-off-by: Berthold Stoeger --- .../tab-widgets/TabDiveEquipment.cpp | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/desktop-widgets/tab-widgets/TabDiveEquipment.cpp b/desktop-widgets/tab-widgets/TabDiveEquipment.cpp index 5105029a3..ad13e6b7e 100644 --- a/desktop-widgets/tab-widgets/TabDiveEquipment.cpp +++ b/desktop-widgets/tab-widgets/TabDiveEquipment.cpp @@ -3,11 +3,14 @@ #include "maintab.h" #include "desktop-widgets/simplewidgets.h" // For isGnome3Session() #include "desktop-widgets/modeldelegates.h" +#include "core/dive.h" +#include "core/selection.h" #include "commands/command.h" #include "qt-models/cylindermodel.h" #include "qt-models/weightmodel.h" +#include #include #include @@ -162,10 +165,21 @@ void TabDiveEquipment::editCylinderWidget(const QModelIndex &index) if (!index.isValid()) return; - if (index.column() == CylindersModel::REMOVE) + if (index.column() == CylindersModel::REMOVE) { + int cylinder_id = cylindersModel->mapToSource(index).row(); + for (dive *d: getDiveSelection()) { + if (cylinder_with_sensor_sample(d, cylinder_id)) { + if (QMessageBox::warning(this, tr("Remove cylinder?"), + tr("The deleted cylinder has sensor readings, which will be lost.\n" + "Do you want to continue?"), + QMessageBox::Yes|QMessageBox::No) != QMessageBox::Yes) + return; + } + } divesEdited(Command::removeCylinder(cylindersModel->mapToSource(index).row(), false)); - else + } else { ui.cylinders->edit(index); + } } void TabDiveEquipment::editWeightWidget(const QModelIndex &index)