In T106067#1535731, @Krenair wrote:I think we also need a blocker about writing (and then running) a script to migrate current users who have been disabled via DisableAccount to blocks.
Description
Description
Details
Details
Customize query in gerrit
Event Timeline
Comment Actions
Noting that password and email gets set to null with DisableAccount so these users would still be unable to login and it would still need a sysadmin to restore accounts for those disabled accounts.
Comment Actions
And the indication that an account was DisableAccount'd is that 'inactive' group. Maybe we can just leave it then, and undeploy the extension?
Comment Actions
In my opinion what we should do is to enable UserMerge in all private and fishbowl wikis. This make re-activate account easy - just create a new user and merge the old one to it.
An alternative idea (but much harder) is to create a new extension to copy an account (with password and email) from one wiki to another (and then they can override the orginal setting). This can even replace the current account request workflow of private and fishbowl wikis. No e-mails will be leaked.
Comment Actions
Change 241512 had a related patch set uploaded (by Glaisher):
Add script to migrate disabled accounts to blocked accounts
Comment Actions
Change 241512 merged by jenkins-bot:
Add script to migrate disabled accounts to blocked accounts
Comment Actions
It needs signoff from SuSa and/or Legal I think?
CommonSettings.php
// DO NOT DISABLE WITHOUT CONTACTING PHILIPPE / LEGAL! // Installed by Andrew, 2011-04-26 if ( $wmgUseDisableAccount ) { require_once( "$IP/extensions/DisableAccount/DisableAccount.php" ); $wgGroupPermissions['bureaucrat']['disableaccount'] = true; }
Comment Actions
Definitely T106067: Undeploy DisableAccount extension needs sign-off, but this task as titled is complete, I assume. There's "write script", "run script", "disable extension".
Comment Actions
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
Comment Actions
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
Comment Actions
Change 461702 merged by jenkins-bot:
[mediawiki/extensions/DisableAccount@master] Check for users with "" for password and email too
Comment Actions
Change 461704 merged by jenkins-bot:
[mediawiki/extensions/DisableAccount@wmf/1.32.0-wmf.22] Check for users with "" for password and email too