Page MenuHomePhabricator

maintenance/removeUnusedAccounts.php now not delete sysops and bureaucrats.
Closed, ResolvedPublic

Description

Author: plmoell

Description:
2 files has been modified.

I fixed an element of the Doxygen TODO list, added by Rob Church in the revision 14082. Now the script don't ignores recent touched accounts, sysop and bureaucrats. See patch.


Version: unspecified
Severity: enhancement

Attached:

Details

Reference
bz14929

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:15 PM
bzimport set Reference to bz14929.
bzimport added a subscriber: Unknown Object (MLST).

plmoell wrote:

oops : "ignores" like the title, not "don't ignores". Sorry.

Patch applied in r38072. Fwiw: Those might be better suited as params (--ignore-groups=sysop,bureaucrat, --ignore-touched=How long ago). Might toy with this in the future.

plmoell wrote:

You want to have it, you have it (implements params).

Done. Anyone may commit it ?

Attached:

Added in r39127. Thank you for your contribution.