[user-manager] [Bug 414860] User Manager can not create a user account unless "Enable administrator privileges for this user" is checked

2019-12-16 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=414860

Méven Car  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
   Version Fixed In||5.18
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/use
   ||r-manager/2471fa7d252de904b
   ||305dac77171fa300d66c05d

--- Comment #3 from Méven Car  ---
Git commit 2471fa7d252de904b305dac77171fa300d66c05d by Méven Car.
Committed on 16/12/2019 at 21:00.
Pushed by meven into branch 'master'.

Use a default non-administrator user Type for new users in model

Summary:
If the administrator role was not set for the index assume the default normal
user type.

Regression was introduced in D25398.
FIXED-IN: 5.18

Test Plan: Created a user as described in
https://bugs.kde.org/show_bug.cgi?id=414860

Reviewers: ngraham, #plasma, apol

Reviewed By: ngraham

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D25976

M  +6-3src/lib/accountmodel.cpp

https://commits.kde.org/user-manager/2471fa7d252de904b305dac77171fa300d66c05d

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

[user-manager] [Bug 414860] User Manager can not create a user account unless "Enable administrator privileges for this user" is checked

2019-12-14 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=414860

--- Comment #2 from Méven Car  ---
Patch proposed at https://phabricator.kde.org/D25976

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

[user-manager] [Bug 414860] User Manager can not create a user account unless "Enable administrator privileges for this user" is checked

2019-12-05 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=414860

Nate Graham  changed:

   What|Removed |Added

   Keywords||regression
 CC||meve...@gmail.com,
   ||n...@kde.org,
   ||nicolas.fe...@gmx.de
   Priority|NOR |VHI
 Status|REPORTED|CONFIRMED
   Severity|normal  |grave
 Ever confirmed|0   |1

--- Comment #1 from Nate Graham  ---
Oh yikes, can confirm. This is a really bad regression.

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