kio-admin: Risk dialog saves only on accept

The "Risks of Acting as an Administrator" prompt now only runs
saveDontShowAgainContinue when the prompt is confirmed. This fixes it
saving even when the prompt was cancelled.

diff --git a/src/admin/workerintegration.cpp b/src/admin/workerintegration.cpp
index a6635749807306ae666647ef332fbb7b044a4a9b..b769e8a7c313a3d140773ef3a38445ea62f1d373 100644 (file)
--- a/src/admin/workerintegration.cpp
+++ b/src/admin/workerintegration.cpp
@@ -165,14 +165,15 @@ void WorkerIntegration::toggleActAsAdmin()
         warningDialog.setDontAskAgainText(i18nc("@option:check", "Do not warn me about these risks again"));
 
         risksAccepted = warningDialog.exec() == KMessageDialog::PrimaryAction;
-        if (warningDialog.isDontAskAgainChecked()) {
-            KMessageBox::saveDontShowAgainContinue(warningDontShowAgainName);
-        }
 
         if (!risksAccepted) {
             updateActAsAdminAction(); // Uncheck the action
             return;
         }
+
+        if (warningDialog.isDontAskAgainChecked()) {
+            KMessageBox::saveDontShowAgainContinue(warningDontShowAgainName);
+        }
     }
 
     url.setScheme(QStringLiteral("admin"));