[pim/pim-sieve-editor] src: Don't override checked state. We change enable to false => double click it will reactivate => bug

Laurent Montel <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit b95cac84c78d0cb73ba9602c418a363db33f6845 by Laurent Montel.
Committed on 14/08/2026 at 17:44.
Pushed by mlaurent into branch 'master'.

Don't override checked state. We change enable to false => double click it will reactivate => bug

M  +4    -1    src/serversievelistwidget.cpp

https://invent.kde.org/pim/pim-sieve-editor/-/commit/b95cac84c78d0cb73ba9602c418a363db33f6845

diff --git a/src/serversievelistwidget.cpp b/src/serversievelistwidget.cpp
index 0cdcb88c..40c396d0 100644
--- a/src/serversievelistwidget.cpp
+++ b/src/serversievelistwidget.cpp
@@ -76,7 +76,9 @@ void ServerSieveListWidget::modifyServerConfig()
     dlg->setWindowTitle(i18nc("@title:window", "Modify Settings"));
     dlg->setServerSieveConfig(serverSieveListItem->serverConfig());
     if (dlg->exec()) {
-        serverSieveListItem->setServerConfig(dlg->serverSieveConfig());
+        SieveEditorUtil::SieveServerConfig newConfig = dlg->serverSieveConfig();
+        newConfig.enabled = (serverSieveListItem->checkState() == Qt::Checked);
+        serverSieveListItem->setServerConfig(newConfig);
     }
     delete dlg;
 }
@@ -99,6 +101,7 @@ void ServerSieveListWidget::addServerConfig()
     QPointer<ServerSieveSettingsDialog> dlg = new ServerSieveSettingsDialog(this);
     if (dlg->exec()) {
         auto item = new ServerSieveListWidgetItem(this);
+
         item->setServerConfig(dlg->serverSieveConfig());
     }
     delete dlg;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.