kde/dolphin
kde/dolphin
1
0
Fork 0
mirror of https://invent.kde.org/system/dolphin synced 2024-10-28 03:21:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

Correctly save behaviourOnLaunch

Browse source 
Summary:
We only wrote out a checkbox value if the checkbox was checked.
Unchecking it would not do anything.

BUG: 398022

Test Plan:
Read KRun's exepcted values
Unchecked config option
Reloaded dolphin
Checkbox remained off

Reviewers: #dolphin, ngraham, elvisangelaccio

Reviewed By: #dolphin, ngraham, elvisangelaccio

Subscribers: ngraham, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D15147
This commit is contained in:
David Edmundson 2018-09-04 14:20:33 +02:00
parent b734386cbc
commit 5f252da8b5
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/settings/general/confirmationssettingspage.cpp
View file

@ -92,13 +92,14 @@ void ConfirmationsSettingsPage::applySettings()
confirmationGroup.writeEntry("ConfirmTrash", m_confirmMoveToTrash->isChecked());
confirmationGroup.writeEntry("ConfirmEmptyTrash", m_confirmEmptyTrash->isChecked());
confirmationGroup.writeEntry("ConfirmDelete", m_confirmDelete->isChecked());
confirmationGroup.sync();
KConfigGroup scriptExecutionGroup(kioConfig, "Executable scripts");
if (m_confirmScriptExecution->isChecked()) {
KConfigGroup scriptExecutionGroup(kioConfig, "Executable scripts");
scriptExecutionGroup.writeEntry("behaviourOnLaunch", "alwaysAsk");
scriptExecutionGroup.sync();
} else {
scriptExecutionGroup.writeEntry("behaviourOnLaunch", "dontAsk");
}
kioConfig->sync();
GeneralSettings* settings = GeneralSettings::self();
settings->setConfirmClosingMultipleTabs(m_confirmClosingMultipleTabs->isChecked());