Page MenuHomePhabricator
Create Task
Maniphest T261067

Use DI in AbuseFilterHooks
Open, Needs TriagePublic
Actions

Description

The AbuseFilterHooks class should use dependency injection to allow unit testing, as per DI docs.

Details

ProjectBranchLines +/-Subject
mediawiki/extensions/AbuseFiltermaster+289 -224Move all "secondary" hooks away from AbuseFilterHooks
mediawiki/extensions/AbuseFiltermaster+184 -32Implement GetAutoPromoteGroupsHook
mediawiki/extensions/AbuseFiltermaster+270 -244Add dedicated classes for more hook handlers
mediawiki/extensions/AbuseFiltermaster+49 -26Migrate change tags hooks to DI
mediawiki/extensions/AbuseFiltermaster+95 -74Migrate a few hook handlers to DI
Customize query in gerrit

Related Objects

Event Timeline

Daimona created this task.Aug 23 2020, 10:05 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 23 2020, 10:05 AM
Daimona moved this task from Backlog to Ideas for overhaul on the User-Daimona board.Aug 23 2020, 10:06 AM
Daimona edited projects, added AbuseFilter (Overhaul-2020); removed User-Daimona, AbuseFilter.Sep 16 2020, 3:13 PM
gerritbot added a comment.Oct 4 2020, 11:46 AM

Change 631966 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[mediawiki/extensions/AbuseFilter@master] Migrate a few hook handlers to DI

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

gerritbot added a project: Patch-For-Review.Oct 4 2020, 11:46 AM
gerritbot added a comment.Oct 13 2020, 8:36 AM

Change 631966 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Migrate a few hook handlers to DI

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

gerritbot added a comment.Oct 13 2020, 9:04 AM

Change 632970 had a related patch set uploaded (by Daimona Eaytoy; owner: Daimona Eaytoy):
[mediawiki/extensions/AbuseFilter@master] Add dedicated classes for more hook handlers

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

ReleaseTaggerBot added a project: MW-1.36-notes (1.36.0-wmf.14; 2020-10-20).Oct 13 2020, 10:00 AM
gerritbot added a comment.Oct 14 2020, 2:34 PM

Change 634033 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[mediawiki/extensions/AbuseFilter@master] Migrate change tags hooks to DI

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

gerritbot added a comment.Oct 21 2020, 6:04 PM

Change 634033 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Migrate change tags hooks to DI

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

ReleaseTaggerBot edited projects, added MW-1.36-notes (1.36.0-wmf.16; 2020-11-03); removed MW-1.36-notes (1.36.0-wmf.14; 2020-10-20).Oct 21 2020, 7:00 PM
gerritbot added a comment.Oct 23 2020, 11:38 AM

Change 632970 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Add dedicated classes for more hook handlers

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

Maintenance_bot removed a project: Patch-For-Review.Oct 23 2020, 12:10 PM
Reedy mentioned this in T266681: MediaWiki installer doesn't create tables for extensions registering updates with new Hook handler.Oct 28 2020, 4:33 PM
gerritbot added a comment.Dec 15 2020, 4:33 PM

Change 649672 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[mediawiki/extensions/AbuseFilter@master] Implement GetAutoPromoteGroupsHook

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

gerritbot added a project: Patch-For-Review.Dec 15 2020, 4:34 PM
gerritbot added a comment.Dec 18 2020, 1:41 PM

Change 649672 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Implement GetAutoPromoteGroupsHook

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

ReleaseTaggerBot edited projects, added MW-1.36-notes (1.36.0-wmf.25; 2021-01-05); removed MW-1.36-notes (1.36.0-wmf.16; 2020-11-03).Dec 18 2020, 2:00 PM
Maintenance_bot removed a project: Patch-For-Review.Dec 18 2020, 2:10 PM
gerritbot added a comment.Mon, Feb 1, 4:19 PM

Change 660861 had a related patch set uploaded (by Daimona Eaytoy; owner: Daimona Eaytoy):
[mediawiki/extensions/AbuseFilter@master] Move all "secondary" hooks away from AbuseFilterHooks

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

gerritbot added a project: Patch-For-Review.Mon, Feb 1, 4:19 PM
matej_suchanek moved this task from Backlog to In progress on the AbuseFilter (Overhaul-2020) board.Fri, Feb 5, 12:59 PM
gerritbot added a comment.Sat, Feb 6, 6:46 PM

Change 660861 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Move all "secondary" hooks away from AbuseFilterHooks

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

ReleaseTaggerBot edited projects, added MW-1.36-notes (1.36.0-wmf.30; 2021-02-09); removed MW-1.36-notes (1.36.0-wmf.25; 2021-01-05).Sat, Feb 6, 7:00 PM
Maintenance_bot removed a project: Patch-For-Review.Sat, Feb 6, 7:10 PM
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL