Combine SpecialPageBeforeFormDisplay set of hooks into one
authorKevin Israel <pleasestand@live.com>
Sun, 22 Jun 2014 11:50:10 +0000 (07:50 -0400)
committerKunal Mehta <legoktm@gmail.com>
Sun, 29 Jun 2014 00:33:14 +0000 (17:33 -0700)
commit5b40c74620cae9da879068d9912eade8f943e057
treea741cc95e3021d2eafe540c5707a7ad4517ef0be
parentbabe76221c258093cc22029a6188e6cd6783ed00
Combine SpecialPageBeforeFormDisplay set of hooks into one

Hooks with dynamically generated names cannot be documented in the normal
manner, and indeed this set, added in r86482 / 10d93c34cb17, was not.

In contrast, every other hook in core has a name that is a fixed string
and thus is easy to grep for, create a manual page title for, and so on.
Conceptually, each hook's name should correspond to one or more locations
at which code can be inserted.

So I changed the code to use hook name "SpecialPageBeforeFormDisplay"
for all FormSpecialPages and added the special page name as a parameter,
which is consistent with existing hook "ActionBeforeFormDisplay".

From extensions in Gerrit, the only use I found was in StopForumSpam,
which is updated to use the new hook in Id474915a. So I chose to break
backward compatibility instead of using a hack to hide the old hooks
from maintenance/findHooks.php.

After three years, the script again reports, "Looks good!"

Change-Id: I7ea6521b47fb034bc367a1d06d477a7654035a5f
RELEASE-NOTES-1.24
docs/hooks.txt
includes/specialpage/FormSpecialPage.php