From 9fc9dbafa3772027455334002dbf93eb24e16ce3 Mon Sep 17 00:00:00 2001 From: "James D. Forrester" Date: Thu, 17 Dec 2020 18:22:55 -0800 Subject: [PATCH] Use User->isRegistered(), not deprecated isLoggedIn() Bug: T270450 Change-Id: Ibeb882b8917f43aed9fd2411a6d4a2400532da33 --- extension.json | 2 +- includes/UniversalLanguageSelectorHooks.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/extension.json b/extension.json index dea015e2..7c775d08 100644 --- a/extension.json +++ b/extension.json @@ -17,7 +17,7 @@ "descriptionmsg": "uls-desc", "license-name": "GPL-2.0-or-later OR MIT", "requires": { - "MediaWiki": ">= 1.33.0" + "MediaWiki": ">= 1.34.0" }, "Hooks": { "BeforePageDisplay": "UniversalLanguageSelectorHooks::addModules", diff --git a/includes/UniversalLanguageSelectorHooks.php b/includes/UniversalLanguageSelectorHooks.php index 73d6b4a3..9ea07022 100644 --- a/includes/UniversalLanguageSelectorHooks.php +++ b/includes/UniversalLanguageSelectorHooks.php @@ -395,7 +395,7 @@ class UniversalLanguageSelectorHooks { public static function addVariables( array &$vars, OutputPage $out ) { // Place request context dependent stuff here $user = $out->getUser(); - $loggedIn = $user->isLoggedIn(); + $loggedIn = $user->isRegistered(); // Do not output accept languages if there is risk it will get cached across requests if ( $out->getConfig()->get( 'ULSAnonCanChangeLanguage' ) || $loggedIn ) {