https://bugs.kde.org/show_bug.cgi?id=398022
David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/dol
                   |                            |phin/5f252da8b58e03b623e56f
                   |                            |788ff6d1056269c005
             Status|CONFIRMED                   |RESOLVED

--- Comment #2 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 5f252da8b58e03b623e56f788ff6d1056269c005 by David Edmundson.
Committed on 04/09/2018 at 12:20.
Pushed by davidedmundson into branch 'Applications/18.08'.

Correctly save behaviourOnLaunch

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

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

M  +4    -3    src/settings/general/confirmationssettingspage.cpp

https://commits.kde.org/dolphin/5f252da8b58e03b623e56f788ff6d1056269c005

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to