*
* @param OutputPage $out
*/
- function setupSkinUserCss( OutputPage $out ) {
+ public function setupSkinUserCss( OutputPage $out ) {
$moduleStyles = [
'mediawiki.legacy.shared',
'mediawiki.legacy.commonPrint',
$tpl->set( 'debughtml', $this->generateDebugHTML() );
$tpl->set( 'reporttime', wfReportTime() );
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
// original version by hansm
- if ( !Hooks::run( 'SkinTemplateOutputPageBeforeExec', [ &$this, &$tpl ] ) ) {
+ if ( !Hooks::run( 'SkinTemplateOutputPageBeforeExec', [ &$skinTemplate, &$tpl ] ) ) {
wfDebug( __METHOD__ . ": Hook SkinTemplateOutputPageBeforeExec broke outputPage execution!\n" );
}
MWNamespace::getSubject( $title->getNamespace() ) );
}
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
$result = [];
- if ( !Hooks::run( 'SkinTemplateTabAction', [ &$this,
+ if ( !Hooks::run( 'SkinTemplateTabAction', [ &$skinTemplate,
$title, $message, $selected, $checkEdit,
&$classes, &$query, &$text, &$result ] ) ) {
return $result;
$userCanRead = $title->quickUserCan( 'read', $user );
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
$preventActiveTabs = false;
- Hooks::run( 'SkinTemplatePreventOtherActiveTabs', [ &$this, &$preventActiveTabs ] );
+ Hooks::run( 'SkinTemplatePreventOtherActiveTabs', [ &$skinTemplate, &$preventActiveTabs ] );
// Checks if page is some kind of content
if ( $title->canExist() ) {
}
}
- Hooks::run( 'SkinTemplateNavigation', [ &$this, &$content_navigation ] );
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
+ Hooks::run( 'SkinTemplateNavigation', [ &$skinTemplate, &$content_navigation ] );
if ( $userCanRead && !$wgDisableLangConversion ) {
$pageLang = $title->getPageLanguage();
'context' => 'subject'
];
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
Hooks::run( 'SkinTemplateNavigation::SpecialPage',
- [ &$this, &$content_navigation ] );
+ [ &$skinTemplate, &$content_navigation ] );
}
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
// Equiv to SkinTemplateContentActions
- Hooks::run( 'SkinTemplateNavigation::Universal', [ &$this, &$content_navigation ] );
+ Hooks::run( 'SkinTemplateNavigation::Universal', [ &$skinTemplate, &$content_navigation ] );
// Setup xml ids and tooltip info
foreach ( $content_navigation as $section => &$links ) {
];
}
+ // Avoid PHP 7.1 warning of passing $this by reference
+ $skinTemplate = $this;
// Use the copy of revision ID in case this undocumented, shady hook tries to mess with internals
Hooks::run( 'SkinTemplateBuildNavUrlsNav_urlsAfterPermalink',
- [ &$this, &$nav_urls, &$revid, &$revid ] );
+ [ &$skinTemplate, &$nav_urls, &$revid, &$revid ] );
}
if ( $out->isArticleRelated() ) {
'href' => $this->getTitle()->getLocalURL( "action=info" )
];
- if ( $this->getTitle()->exists() ) {
+ if ( $this->getTitle()->exists() || $this->getTitle()->inNamespace( NS_CATEGORY ) ) {
$nav_urls['recentchangeslinked'] = [
'href' => SpecialPage::getTitleFor( 'Recentchangeslinked', $this->thispage )->getLocalURL()
];
if ( !$user->isAnon() ) {
$sur = new UserrightsPage;
$sur->setContext( $this->getContext() );
- $canChange = $sur->userCanChangeRights( $this->getUser(), false );
+ $canChange = $sur->userCanChangeRights( $user );
$nav_urls['userrights'] = [
'text' => $this->msg(
$canChange ? 'tool-link-userrights' : 'tool-link-userrights-readonly',
- $this->getUser()->getName()
+ $rootUser
)->text(),
'href' => self::makeSpecialUrlSubpage( 'Userrights', $rootUser )
];