Page MenuHomePhabricator

special:checkuser no longer shows if an account has been locked
Closed, ResolvedPublic

Description

Per title!

Event Timeline

Vituzzu triaged this task as High priority.Jun 15 2016, 8:13 PM

It seems there's something wrong with APIs. popup returns "BLOCKED, LOCKED" for every user account.

It seems there's something wrong with APIs. popup returns "BLOCKED, LOCKED" for every user account.

However, if an account has no live edits, no status is shown at all.

It seems there's something wrong with APIs. popup returns "BLOCKED, LOCKED" for every user account.

However, if an account has no live edits, no status is shown at all.

Wops, errata corrige, LOCKED, HIDDEN, just global status then.

It seems there's something wrong with APIs. popup returns "BLOCKED, LOCKED" for every user account.

That was a navigation popups bug which was fixed on en.wp.

While $wgDisableAuthManager = false:

> $u = User::newfromName('Newuser');
> return $u->isLocked();
bool(false)
> $ca = CentralAuthUser::getInstance($u);
> return $ca->isLocked();
bool(true)

Works as expected when AuthManager is disabled. We might have permissions checking places with User::isLocked() so this will probably have to be fixed soon.

Change 294912 had a related patch set uploaded (by Glaisher):
Add UserIsLocked and UserIsHidden hook handlers

https://gerrit.wikimedia.org/r/294912

Change 294912 merged by jenkins-bot:
Add UserIsLocked and UserIsHidden hook handlers

https://gerrit.wikimedia.org/r/294912

Glaisher claimed this task.