Page MenuHomePhabricator

Expiring userrights: update Special:UserRights after rights expiration
Closed, ResolvedPublic


I've been testing on this feature.

  • Granted MarcoAurelio (test) temporary adminship for 10 minutes.
    • account appears flagged at Special:ListUsers with a temporary warning and log format is correct
  • After the expiration, the account is removed from Special:Listusers/sysops as expected, however Special:UserRights still displays the checkbox checked.

When the right expire, the checkboxes at special:userrights should be updated accordingly. Thanks.

Event Timeline

Special:UserRights has been updated now. Can you confirm that it takes a while to update? Thanks.

I can't reproduce this. Special:UserRights updated immediately for me.

Did you just press F5/click Refresh in your browser on Special:UserRights? That could explain why it is remaining checked - the browser is remembering your form settings after refreshing the page.

I'm doing further checks to confirm.

This time I've hit the "edit user rights" button and while the right has expired, it remains checked. Exit the page and opened Special:UserRights for that account again and it remains checked as well. Maybe I'm doing something wrong?

I was testing with local user rights, which would explain why I didn't notice it.

I can indeed see at that the box is still checked. Interesting...

This is indeed a bug in the code. Not a major one, but a bug all the same.

Note that I've granted myself now local rights at deployment wiki after your comment. Will remove them now and see how they appear in the userrights-interwiki form.

Change 335205 had a related patch set uploaded (by TTO):
Don't return expired user groups in UserRightsProxy

When updated local rights, remote form is now updated as well. Will return to leave things as they were so it can be investigated.

Re-granted temporary admin rights for 10 seconds at deploymentwiki via metawiki beta cluster. Checkbox still not updated. Sorry for the noise and testing and thanks for taking care of this! Regards.

Change 335205 merged by jenkins-bot:
Don't return expired user groups in UserRightsProxy

Thanks MA once again :)