https://bugs.kde.org/show_bug.cgi?id=481113

Marco Martin <notm...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/libplasma/-/commit/3bc19 |ma/libplasma/-/commit/1f3f9
                   |9d92402e316541d6dec00af9fee |84c02bcf6719fb96182e47546ce
                   |66afc341                    |62dbef73

--- Comment #64 from Marco Martin <notm...@gmail.com> ---
Git commit 1f3f984c02bcf6719fb96182e47546ce62dbef73 by Marco Martin, on behalf
of David Edmundson.
Committed on 12/06/2024 at 09:51.
Pushed by mart into branch 'Plasma/6.1'.

Applet: run full Applet::save when a config value changes

Plasma had two timers for configuration saving:
 - One applet specific, which eventually calls Applet::save
 - One at a corona level, which eventually calls KConfig::save

The former was incorrectly removed because it wasn't hooked up to
anything useful, but the real bug is that it should have been.

When a value changes in the KConfigPropertyMap we need to run through
Applet::save to sync that config loader with our main config. This
implicitly emits the configNeedsSaving which will trigger corona to sync
to disk.

M  +3    -1    src/plasma/applet.cpp

https://invent.kde.org/plasma/libplasma/-/commit/1f3f984c02bcf6719fb96182e47546ce62dbef73

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

Reply via email to