Mark Action::getHookContainer() as @internal

Once all actions are having dependencies injected,
those that need a hook container can just have it injected
instead of relying on the global state, which is what
Action::getHookContainer() and ::getHookRunner()
use.

Bug: T253078
Change-Id: I09c70703d4634f855620a56273a9ad924cf061a7

RELEASE-NOTES-1.37

@@ -389,6 +389,8 @@ because of Phabricator reports.
   WatchlistManager::addWatch() or WatchlistManager::removeWatch(), instead.
 * WatchAction::getWatchToken is hard deprecated. Instead use
   CsrfTokenSet::getToken
+* Action::getHookContainer() has been marked as internal. Actions that require
+  access to a hook container should have one injected instead.
 * The getTitle() and setTitle() methods in Parser have been deprecated.
   Use getPage() and setPage() instead.
 * Title::isWatchable() has been deprecated. Use WatchlistManager::isWatchable()

includes/actions/Action.php

@@ -231,6 +231,7 @@ abstract class Action implements MessageLocalizer {
 
 	/**
 	 * @since 1.35
+	 * @internal since 1.37
 	 * @return HookContainer
 	 */
 	protected function getHookContainer() {