Selecting a role at /admin/config/people/password_policy/password_change and submitting the form does not force users to change their password.

This bug has to do with $db_uids = db_query('SELECT uid FROM {users_roles} WHERE rid = :rid', array('rid' => $role))->fetchCol(); and the following foreach loop returning a int value instead of an object.

Patch to follow.

CommentFileSizeAuthor
#2 1211080-force-users-change-password-2.patch2.98 KBerikwebb
#1 force-users-to-change-password-1211080-1.patch1.2 KBmukhsim

Comments

mukhsim’s picture

mukhsim commented
StatusFileSize
new force-users-to-change-password-1211080-1.patch1.2 KB

Patch attached.

erikwebb’s picture

erikwebb commented
Status: Active » Needs review
StatusFileSize
new 1211080-force-users-change-password-2.patch2.98 KB

The original code looks like it may have problems. I needed to add (or remove from both) ->fetchCol() from both nearby db_query() calls. Looks much simplified using your example and works fine for me. Please confirm.

deekayen’s picture

deekayen commented
Status: Needs review » Fixed

committed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.