Rename Hook interfaces that don't follow naming convention
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	Ammarpad
	Apr 16 2023, 2:26 PM

Project Tags

Referenced Files

None

Subscribers

Description

From T334689: Ensure all Hook interfaces contain 'Hook' suffix in their names, three extant hooks from CheckUser are exempted from adhering to the naming convention. These should be fixed, so the exception can be removed.

Related Changes in Gerrit:

	Subject	Author	Repo	Branch	Lines +/-
	Add Hook suffix to CheckUserInsert*Hook classes	Umherirrender	mediawiki/extensions/CheckUser	master	+32 -8

Mentioned Here: T334689: Ensure all Hook interfaces contain 'Hook' suffix in their names

Change 908988 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CheckUser@master] Add Hook suffix to CheckUserInsert*Hook classes

I think another patch will be needed to remove the exception to the core test now that these are renamed.

Change 908988 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add Hook suffix to CheckUserInsert*Hook classes

Umherirrender claimed this task.