diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2014-03-15 14:56:32 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2014-03-15 14:56:32 -0700 |
| commit | 36ba353450b9df1b4d75aa6a4deef8fb10400f1a (patch) | |
| tree | a17ebadbcc1c651c3aff4f6bfb36da31ebc21f1d /qt-ui | |
| parent | 4a1b9312f38222af6ef1f19b8551cfdeb47ad8df (diff) | |
| download | subsurface-36ba353450b9df1b4d75aa6a4deef8fb10400f1a.tar.gz | |
Correctly check if tank is in use
We always checked if the tank was in use by the planner / dive editor -
even if we were not in dive edit mode.
With this patch, when not in dive edit mode, we check our cylinder "used"
flag instead.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-ui')
| -rw-r--r-- | qt-ui/models.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/qt-ui/models.cpp b/qt-ui/models.cpp index 01a3c33b3..1689139e1 100644 --- a/qt-ui/models.cpp +++ b/qt-ui/models.cpp @@ -336,7 +336,10 @@ void CylindersModel::remove(const QModelIndex &index) return; } cylinder_t *cyl = ¤t->cylinder[index.row()]; - if (DivePlannerPointsModel::instance()->tankInUse(cyl->gasmix.o2.permille, cyl->gasmix.he.permille)) { + if ((DivePlannerPointsModel::instance()->currentMode() != DivePlannerPointsModel::NOTHING && + DivePlannerPointsModel::instance()->tankInUse(cyl->gasmix.o2.permille, cyl->gasmix.he.permille)) || + (DivePlannerPointsModel::instance()->currentMode() == DivePlannerPointsModel::NOTHING && cyl->used)) + { QMessageBox::warning(MainWindow::instance(), TITLE_OR_TEXT( tr("Cylinder cannot be removed"), tr("This gas in use. Only cylinders that are not used in the dive can be removed.")), |