Details
Event Timeline
@Jdforrester-WMF what are your plans for this one? Are you going to run it or do you need help?
Would be nice to just get this done and out of the way
Do we want to temporarily enable it on somewhere like testwiki, create an account, disable them, and then see what the migration looks like? Ie the blocking and the log entries. Then look to run it on the other wikis in one go?
@Reedy If that helps to know later what is exactly needed to do, then okay. The most important is that the wikis using this extension should also be set to $wgBlockDisablesLogin (which should be true for all private wikis) and that they indeed remain unable to access after they're migrated.
Since wgBlockDisablesLogin is set to true for private wikis, is there anything blocking this task?
I also think there was some issues with the script @Reedy created not picking the right accounts sometimes.
Thanks @Reedy - please bear in mind what we said at T106067#3041391. As far as I can see the script just looks for accounts in the 'inactive' user group which is not an accurate search parameter as per our comments above.
T106067#3041409 would be the most correct searching parameters as the 'inactive' user group has been manually added to accounts.
MariaDB [stewardwiki]> select count(*) from user where user_password = "" AND user_email = ""; +----------+ | count(*) | +----------+ | 5 | +----------+ 1 row in set (0.00 sec) MariaDB [stewardwiki]> select count(*) from user where user_password IS null AND user_email IS null; +----------+ | count(*) | +----------+ | 0 | +----------+ 1 row in set (0.00 sec)
Doesn't seem to be exactly the case... https://github.com/wikimedia/mediawiki-extensions-DisableAccount/blob/master/SpecialDisableAccount.php#L71 - The script adds it, and has for a long time
Yes, indeed DisableAccount does add an account to the inactive user group. But not all accounts flagged as "inactive" are disabled by that extension. Hope that it helps.
Change 461702 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/DisableAccount@master] Check for users with "" for password and email too
Change 461704 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/DisableAccount@wmf/1.32.0-wmf.22] Check for users with "" for password and email too
Change 461702 merged by jenkins-bot:
[mediawiki/extensions/DisableAccount@master] Check for users with "" for password and email too
Change 461704 merged by jenkins-bot:
[mediawiki/extensions/DisableAccount@wmf/1.32.0-wmf.22] Check for users with "" for password and email too