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

Harald Sitter <sit...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-workspace/commit/
                   |                            |9f162e232dbbbaeaef41ad6f69a
                   |                            |048cbc0b4f815

--- Comment #5 from Harald Sitter <sit...@kde.org> ---
Git commit 9f162e232dbbbaeaef41ad6f69a048cbc0b4f815 by Harald Sitter.
Committed on 21/09/2022 at 10:31.
Pushed by sitter into branch 'master'.

users: delay result submission

the user change application is blocking (because saving as a whole is
blocking) BUT result submission doesn't need to be and indeed need not
be because the applyjob is running as part of the save() call stack, and
that will set needssave at the very end, effectively undermining any
state management done as part of error handling.

effectively this now means that save() sets needsSave=false and then on
the next event loop run our error handling kicks in and reevaluates
resolvePending on the qml side, potentially setting it true if necessary

M  +1    -1    kcms/users/src/user.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/9f162e232dbbbaeaef41ad6f69a048cbc0b4f815

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

Reply via email to