DlgPrefEffects: fix preset removed from list when canceling delete

fixes https://bugs.launchpad.net/mixxx/+bug/1947922
mixxxdj · Oct 22, 2021 · d59011f · d59011f
1 parent cfaeec9
commit d59011f
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/src/effects/presets/effectchainpresetmanager.cpp b/src/effects/presets/effectchainpresetmanager.cpp
@@ -294,17 +294,17 @@ void EffectChainPresetManager::renamePreset(const QString& oldName) {
     emit effectChainPresetListUpdated();
 }
 
-void EffectChainPresetManager::deletePreset(const QString& chainPresetName) {
+bool EffectChainPresetManager::deletePreset(const QString& chainPresetName) {
     VERIFY_OR_DEBUG_ASSERT(m_effectChainPresets.contains(chainPresetName)) {
-        return;
+        return false;
     }
     auto pressedButton = QMessageBox::question(nullptr,
             tr("Remove effect chain preset"),
             tr("Are you sure you want to delete the effect chain preset "
                "\"%1\"?")
                     .arg(chainPresetName));
     if (pressedButton != QMessageBox::Yes) {
-        return;
+        return false;
     }
 
     QFile file(m_pConfig->getSettingsPath() + kEffectChainPresetDirectory +
@@ -318,7 +318,7 @@ void EffectChainPresetManager::deletePreset(const QString& chainPresetName) {
         msgBox.setDetailedText(file.errorString());
         msgBox.setIcon(QMessageBox::Critical);
         msgBox.exec();
-        return;
+        return false;
     }
 
     EffectChainPresetPointer pPreset =
@@ -327,6 +327,7 @@ void EffectChainPresetManager::deletePreset(const QString& chainPresetName) {
     m_quickEffectChainPresetsSorted.removeAll(pPreset);
     emit effectChainPresetListUpdated();
     emit quickEffectChainPresetListUpdated();
+    return true;
 }
 
 void EffectChainPresetManager::setPresetOrder(

diff --git a/src/effects/presets/effectchainpresetmanager.h b/src/effects/presets/effectchainpresetmanager.h
@@ -54,7 +54,7 @@ class EffectChainPresetManager : public QObject {
     void importPreset();
     void exportPreset(const QString& chainPresetName);
     void renamePreset(const QString& oldName);
-    void deletePreset(const QString& chainPresetName);
+    bool deletePreset(const QString& chainPresetName);
 
     void resetToDefaults();
 

diff --git a/src/preferences/dialog/dlgprefeffects.cpp b/src/preferences/dialog/dlgprefeffects.cpp
@@ -283,9 +283,12 @@ void DlgPrefEffects::slotDeletePreset() {
     for (const auto& index : selectedIndices) {
         QString selectedPresetName =
                 m_pFocusedChainList->model()->data(index).toString();
-        focusedChainStringList.removeAll(selectedPresetName);
         if (!unfocusedChainStringList.contains(selectedPresetName)) {
-            m_pChainPresetManager->deletePreset(selectedPresetName);
+            if (m_pChainPresetManager->deletePreset(selectedPresetName)) {
+                focusedChainStringList.removeAll(selectedPresetName);
+            }
+        } else {
+            focusedChainStringList.removeAll(selectedPresetName);
         }
     }