Deprecation of 2 more ContentHandler hooks

ArticleInsertComplete, ArticleSaveComplete

Bug: T145728
Bug: T147388
Bug: T147390
Change-Id: Ib861d3ee72bdaecfd91b7e719bc8054789ccd489

diff --git a/RELEASE-NOTES-1.28 b/RELEASE-NOTES-1.28
index 0078399..78986d7 100644 (file)
--- a/RELEASE-NOTES-1.28
+++ b/RELEASE-NOTES-1.28
@@ -214,9 +214,9 @@ changes to languages because of Phabricator reports.
 * IP::isConfiguredProxy() and IP::isTrustedProxy() were removed. Callers should
   migrate to using the same functions on a ProxyLookup instance, obtainable from
   MediaWikiServices.
-* The ArticleAfterFetchContent, ArticleSave, ArticleViewCustom, EditPageGetDiffText,
-  EditPageGetPreviewText and ShowRawCssJs hooks will now emit deprecation warnings if
-  used.
+* The ArticleAfterFetchContent, ArticleInsertComplete, ArticleSave, ArticleSaveComplete,
+  ArticleViewCustom, EditPageGetDiffText, EditPageGetPreviewText and ShowRawCssJs hooks
+  will now emit deprecation warnings if used.
 
 == Compatibility ==
 

diff --git a/includes/page/WikiPage.php b/includes/page/WikiPage.php
index 6c85070..2833965 100644 (file)
--- a/includes/page/WikiPage.php
+++ b/includes/page/WikiPage.php
@@ -2023,11 +2023,11 @@ class WikiPage implements Page, IDBAccessObject {
                                        // Trigger post-create hook
                                        $params = [ &$this, &$user, $content, $summary,
                                                $flags & EDIT_MINOR, null, null, &$flags, $revision ];
-                                       ContentHandler::runLegacyHooks( 'ArticleInsertComplete', $params );
+                                       ContentHandler::runLegacyHooks( 'ArticleInsertComplete', $params, '1.21' );
                                        Hooks::run( 'PageContentInsertComplete', $params );
                                        // Trigger post-save hook
                                        $params = array_merge( $params, [ &$status, $meta['baseRevId'] ] );
-                                       ContentHandler::runLegacyHooks( 'ArticleSaveComplete', $params );
+                                       ContentHandler::runLegacyHooks( 'ArticleSaveComplete', $params, '1.21' );
                                        Hooks::run( 'PageContentSaveComplete', $params );
 
                                }