davidedmundson added a comment.
Aha, found what changed it: 2471fa7d252de904b305dac77171fa300d66c05d <https://phabricator.kde.org/R128:2471fa7d252de904b305dac77171fa300d66c05d> - if (!roles.contains(Username) || !roles.contains(RealName) || !roles.contains(Administrator)) { + if (!roles.contains(Username) || !roles.contains(RealName)) { because administrator is set last, it was previously effectively doing what this patch is doing *but* on closer inspection what this is doing should work regardless, any setData that wasn't needed for creating an account was buffered, any new stuff in theory is on the same row, that row now has an entry in m_users so it should make the call REPOSITORY R128 User Manager REVISION DETAIL https://phabricator.kde.org/D26977 To: davidedmundson, #plasma, meven Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
