<table><tr><td style="">davidedmundson added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26977">View Revision</a></tr></table><br /><div><div><p>Aha, found what changed it:</p>

<p><a href="https://phabricator.kde.org/R128:2471fa7d252de904b305dac77171fa300d66c05d" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">2471fa7d252de904b305dac77171fa300d66c05d</a></p>

<ul class="remarkup-list">
<li class="remarkup-list-item">if (!roles.contains(Username) || !roles.contains(RealName) || !roles.contains(Administrator)) {</li>
</ul>

<p>+    if (!roles.contains(Username) || !roles.contains(RealName)) {</p>

<p>because administrator is set last, it was previously effectively doing what this patch is doing</p>

<p>*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</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R128 User Manager</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26977">https://phabricator.kde.org/D26977</a></div></div><br /><div><strong>To: </strong>davidedmundson, Plasma, meven<br /><strong>Cc: </strong>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<br /></div>