ngraham added a comment.

  Thanks Méven! This fixes the case of creating a non-admin user. However while 
testing it, I found a regression from D25439 
<https://phabricator.kde.org/D25439>: when I create an admin user that has a 
space in the Real Name field, the "Enable administror privileges for this user" 
checkbox becomes unchecked and it isn't actually created as an admin user. Do 
you think you could also fix that in this patch?

REPOSITORY
  R128 User Manager

REVISION DETAIL
  https://phabricator.kde.org/D25976

To: meven, ngraham, #plasma, apol
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to