The Authenticated user role doesn't reset any accounts. It should reset all users, but doesn't. Looks like @chim had covered this in his patches for #1903204: role specific resets. That may be a good place to look.

Anyway, currently, since the a role has to be selected, mass_pwreset will fail if there are only admin and authenticated user roles. Regular users who are not admin will not get passwords reset.

@bmeme noted this issue here: #2154539-4: Role specific reset and batching

Comments

shrop’s picture

Issue summary: View changes
mthomas’s picture

StatusFileSize
new1.1 KB

The attached patch fixes the issue for me. If the authenticated user rid is present, all users except uid 0 will be returned. Otherwise the old query is used.

charles belov’s picture

Status: Active » Needs review

Setting to needs review so it will go through the testbot and to call attention to it for testers.

For the issue itself, behind the scenes a repeated division-by-zero error was showing up many times in line 31 of mass_pwdreset.batch.inc:

$context['finished'] = $context['sandbox']['progress'] / $context['sandbox']['max'];

  • shrop committed 67845a5 on 7.x-1.x authored by mthomas
    Issue #2191935 by mthomas: Authenticate user role doesn't work
    
shrop’s picture

Status: Needs review » Fixed

Tested and confirmed patch works. Also, noticed the issue Charles Belov mentioend "Warning: Division by zero in mass_pwreset_batch_process() (line 31 of /var/www/html/sites/all/modules/contrib/mass_pwreset/mass_pwreset.batch.inc)."

Patched committed. Thank you all!

Status: Fixed » Closed (fixed)

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