Fix message box logic

qgis · Nov 20, 2017 · 06017b9242111e98ea461d57c12333c69f10a8ca · 06017b9
1 parent ecde835
commit 06017b9242111e98ea461d57c12333c69f10a8ca
Unified Split

Showing with 2 additions and 4 deletions.

+2 −4 src/app/qgsmapthemes.cpp
diff --git a/src/app/qgsmapthemes.cpp b/src/app/qgsmapthemes.cpp
@@ -163,10 +163,8 @@ void QgsMapThemes::removeCurrentPreset()
       int res = QMessageBox::question( mMenu, tr( "Remove Theme" ),
                                        trUtf8( "Are you sure you want to remove the existing theme “%1”?" ).arg( actionPreset->text() ),
                                        QMessageBox::Yes | QMessageBox::No, QMessageBox::No );
-      if ( res != QMessageBox::Yes )
-        return;
-      //remove the selected preset
-      QgsProject::instance()->mapThemeCollection()->removeMapTheme( actionPreset->text() );
+      if ( res == QMessageBox::Yes )
+        QgsProject::instance()->mapThemeCollection()->removeMapTheme( actionPreset->text() );
       break;
     }
   }