https://bugs.kde.org/show_bug.cgi?id=481113
Marco Martin <notm...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
|ma/libplasma/-/commit/f7b3e |ma/libplasma/-/commit/3bc19
|98bd5c777a8c7510ec0edaaba64 |9d92402e316541d6dec00af9fee
|12cac474 |66afc341
Status|ASSIGNED |RESOLVED
--- Comment #63 from Marco Martin <notm...@gmail.com> ---
Git commit 3bc199d92402e316541d6dec00af9fee66afc341 by Marco Martin, on behalf
of David Edmundson.
Committed on 12/06/2024 at 09:16.
Pushed by mart into branch 'master'.
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/3bc199d92402e316541d6dec00af9fee66afc341
--
You are receiving this mail because:
You are watching all bug changes.
