Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372595 )
Change subject: Mark various skin/OutputPage hooks as unabortable ...................................................................... Mark various skin/OutputPage hooks as unabortable Follows-up I94c7ab656bd1a04. An initial pass of marking various hooks as unabortable. * BeforePageDisplay (T173411) * OutputPageParserOutput * AfterFinalPageOutput * SkinTemplateNavigation (T107980) * SkinTemplateNavigation::SpecialPage (T107980) * SkinTemplateNavigation::Universal * PersonalUrls Bug: T173615 Change-Id: I0cb333dd5ff74f7dca725ff210200a0779a9a360 --- M includes/OutputPage.php M includes/skins/SkinTemplate.php 2 files changed, 9 insertions(+), 8 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/95/372595/1 diff --git a/includes/OutputPage.php b/includes/OutputPage.php index 2a3a7cd..0f95607 100644 --- a/includes/OutputPage.php +++ b/includes/OutputPage.php @@ -1839,7 +1839,7 @@ // Avoid PHP 7.1 warning of passing $this by reference $outputPage = $this; Hooks::run( 'LanguageLinks', [ $this->getTitle(), &$this->mLanguageLinks, &$linkFlags ] ); - Hooks::run( 'OutputPageParserOutput', [ &$outputPage, $parserOutput ] ); + Hooks::run( 'OutputPageParserOutput', [ &$outputPage, $parserOutput ], Hooks::NO_ABORT ); // This check must be after 'OutputPageParserOutput' runs in addParserOutputMetadata // so that extensions may modify ParserOutput to toggle TOC. @@ -1877,7 +1877,7 @@ $text = $parserOutput->getText(); // Avoid PHP 7.1 warning of passing $this by reference $outputPage = $this; - Hooks::run( 'OutputPageBeforeHTML', [ &$outputPage, &$text ] ); + Hooks::run( 'OutputPageBeforeHTML', [ &$outputPage, &$text ], Hooks::NO_ABORT ); $this->addHTML( $text ); } @@ -2423,7 +2423,7 @@ $outputPage = $this; // Hook that allows last minute changes to the output page, e.g. // adding of CSS or Javascript by extensions. - Hooks::run( 'BeforePageDisplay', [ &$outputPage, &$sk ] ); + Hooks::run( 'BeforePageDisplay', [ &$outputPage, &$sk ], Hooks::NO_ABORT ); try { $sk->outputPage(); @@ -2435,7 +2435,7 @@ try { // This hook allows last minute changes to final overall output by modifying output buffer - Hooks::run( 'AfterFinalPageOutput', [ $this ] ); + Hooks::run( 'AfterFinalPageOutput', [ $this ], Hooks::NO_ABORT ); } catch ( Exception $e ) { ob_end_clean(); // bug T129657 throw $e; diff --git a/includes/skins/SkinTemplate.php b/includes/skins/SkinTemplate.php index cbffe1e..8b15788 100644 --- a/includes/skins/SkinTemplate.php +++ b/includes/skins/SkinTemplate.php @@ -729,7 +729,7 @@ } } - Hooks::run( 'PersonalUrls', [ &$personal_urls, &$title, $this ] ); + Hooks::run( 'PersonalUrls', [ &$personal_urls, &$title, $this ], Hooks::NO_ABORT ); return $personal_urls; } @@ -1092,7 +1092,7 @@ // Avoid PHP 7.1 warning of passing $this by reference $skinTemplate = $this; - Hooks::run( 'SkinTemplateNavigation', [ &$skinTemplate, &$content_navigation ] ); + Hooks::run( 'SkinTemplateNavigation', [ &$skinTemplate, &$content_navigation ], Hooks::NO_ABORT ); if ( $userCanRead && !$wgDisableLangConversion ) { $pageLang = $title->getPageLanguage(); @@ -1137,13 +1137,14 @@ // Avoid PHP 7.1 warning of passing $this by reference $skinTemplate = $this; Hooks::run( 'SkinTemplateNavigation::SpecialPage', - [ &$skinTemplate, &$content_navigation ] ); + [ &$skinTemplate, &$content_navigation ], Hooks::NO_ABORT ); } // Avoid PHP 7.1 warning of passing $this by reference $skinTemplate = $this; // Equiv to SkinTemplateContentActions - Hooks::run( 'SkinTemplateNavigation::Universal', [ &$skinTemplate, &$content_navigation ] ); + Hooks::run( 'SkinTemplateNavigation::Universal', + [ &$skinTemplate, &$content_navigation ], Hooks::NO_ABORT ); // Setup xml ids and tooltip info foreach ( $content_navigation as $section => &$links ) { -- To view, visit https://gerrit.wikimedia.org/r/372595 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0cb333dd5ff74f7dca725ff210200a0779a9a360 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Krinkle <krinklem...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits