Page MenuHomePhabricator

Use DI in AbuseFilterHooks
Open, Needs TriagePublic

Description

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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