Merge lp://staging/~laney/unity-control-center/lp1630156 into lp://staging/unity-control-center

Proposed by Iain Lane
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 12878
Merged at revision: 12878
Proposed branch: lp://staging/~laney/unity-control-center/lp1630156
Merge into: lp://staging/unity-control-center
Diff against target: 17 lines (+2/-2)
1 file modified
panels/user-accounts/um-password-dialog.c (+2/-2)
To merge this branch: bzr merge lp://staging/~laney/unity-control-center/lp1630156
Reviewer Review Type Date Requested Status
Robert Ancell Approve
Sebastien Bacher Approve
Review via email: mp+316835@code.staging.launchpad.net

Commit message

user-accounts: Reset the AccountsService password mode when setting the password. There's a codepath for directly setting the password when the user is setting their own, but this doesn't set the AS mode back to ACT_USER_PASSWORD_MODE_REGULAR. If you don't change this and you're in ACT_USER_PASSWORD_MODE_NONE, then you end up staying in the nopasswdlogin group.

Description of the change

user-accounts: Reset the AccountsService password mode when setting the password

There's a codepath for directly setting the password when the user is setting their own, but this doesn't set the AS mode back to ACT_USER_PASSWORD_MODE_REGULAR. If you don't change this and you're in ACT_USER_PASSWORD_MODE_NONE, then you end up staying in the nopasswdlogin group.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

that looks good, thanks

review: Approve
Revision history for this message
Robert Ancell (robert-ancell) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches