From 7bd7d2a6c19885605ae1f2c9210597f3f72f57e3 Mon Sep 17 00:00:00 2001 From: DannyS712 Date: Sun, 11 Apr 2021 20:49:08 +0000 Subject: Remove hooks that use Revision objects All hooks were previously hard deprecated in 1.35. Affected hooks: * ArticleRevisionUndeleted - use RevisionUndeleted * ArticleRollbackComplete - use RollbackComplete * DiffRevisionTools - use DiffTools * DiffViewHeader - use DifferenceEngineViewHeader * HistoryRevisionTools - use HistoryTools * NewRevisionFromEditComplete - use RevisionFromEditComplete * PageContentInsertComplete - use PageSaveComplete * PageContentSaveComplete - use PageSaveComplete * ParserFetchTemplate - use BeforeParserFetchTemplateRevisionRecord * RevisionInsertComplete - use RevisionRecordInserted * TitleMoveComplete - use PageMoveComplete * TitleMoveCompleting - use PageMoveCompleting * UndeleteShowRevision - no replacement Includes a fix for setting the associated rev id of page protections, which previously was only done using $nullRevision which was a Revision object created if any hooks needed it; those hooks were hard deprecated and so for WMF prod the rev id was not being set. Bug: T247143 Depends-On: Idfa345193ae99fb2f1c9a8f8d28d8d540a6e3d62 Change-Id: I519167f76a5a3c1f5410415b2721462a3dcc3ec8 --- docs/contenthandler.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'docs/contenthandler.md') diff --git a/docs/contenthandler.md b/docs/contenthandler.md index 48b2e9eb5670..b09c944b257d 100644 --- a/docs/contenthandler.md +++ b/docs/contenthandler.md @@ -78,9 +78,9 @@ For rendering page content, `Content::getParserOutput()` should be used instead Besides some functions, some hooks have also been replaced by new versions (see hooks.txt for details). These hooks will now trigger a warning when used: * `ArticleAfterFetchContent` was replaced by `ArticleAfterFetchContentObject`, later replaced by `ArticleRevisionViewCustom` -* `ArticleInsertComplete` was replaced by `PageContentInsertComplete` +* `ArticleInsertComplete` was replaced by `PageContentInsertComplete`, later replaced by `PageSaveComplete` * `ArticleSave` was replaced by `PageContentSave` -* `ArticleSaveComplete` was replaced by `PageContentSaveComplete` +* `ArticleSaveComplete` was replaced by `PageContentSaveComplete`, later replaced by `PageSaveComplete` * `ArticleViewCustom` was replaced by `ArticleContentViewCustom`, which was later removed entirely * `EditFilterMerged` was replaced by `EditFilterMergedContent` * `EditPageGetDiffText` was replaced by `EditPageGetDiffContent` -- cgit v1.2.3