https://bugs.kde.org/show_bug.cgi?id=373276
Unknown <n...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://commits.kde.org/use
| |r-manager/f2c69db182fb20453
| |e671359e90a3bc6de40c7b0
Resolution|--- |FIXED
Status|UNCONFIRMED |RESOLVED
--- Comment #5 from Unknown <n...@kde.org> ---
Git commit f2c69db182fb20453e671359e90a3bc6de40c7b0 by Harald Sitter.
Committed on 06/12/2016 at 10:54.
Pushed by sitter into branch 'Plasma/5.8'.
Revert "Do not ask for root permissions when it's unnecessary"
Summary:
This reverts commit a666712102be7ef4dd48202cc2411921fc4d392b.
This broke adding new users when not setting realname or adminflag (i.e.
at present there is no way to create a !admin user at all).
Distributions, as we are still 3 weeks away from 5.8.5 I'd advise patching
this to restore working behavior for the time being.
The problem in particular is that the model gobbles up setData requests to
new rows and forwards them to newUserSetData which in turn caches them
until username&realname&admin are present and only then forwards the call
to accountsservice. By calling setData on-demand the three fields are not
set unless they in fact all where "toggled" from their default.
I suggest that the noop decision be moved into the setData itself. In there
it should be possible to accurately decide whether or not the data
actually changed and accountsservice needs to be called.
(Ideally though IMO the collection in newUserSetData should be gotten
rid of. I haven't had a close look, but creating the user with random
data for everything but username and then manipulating it on the
subsequent setData calls should be a more future-proof and reliable
approach)
CCMAIL: kde-distro-packag...@kde.org
CCMAIL: larr...@kde.org
PHAB: https://phabricator.kde.org/D3102
Reviewers: davidedmundson, mart
Subscribers: antlarr, plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D3605
M +10 -20 src/accountinfo.cpp
M +2 -0 src/lib/accountmodel.cpp
https://commits.kde.org/user-manager/f2c69db182fb20453e671359e90a3bc6de40c7b0
--
You are receiving this mail because:
You are watching all bug changes.
