Use User::getDefaultOptions instead of wgDefaultUserOptions

multimediaviewer-enable default option does not exist in
$wgDefaultUserOptions, but is added in a UserGetDefaultOptions
hook callback.

User::getDefaultOptions will trigger that hook (and merge it
with $wgDefaultUserOptions), so that's the result that should
be used to figure out the value for multimediaviewer-enable.

Bug: T166058
Change-Id: Ib16a779430886305582dfce7db6600edd87ace11
This commit is contained in:
Matthias Mullie 2017-10-17 14:17:14 +02:00
parent b360693984
commit 51baec14a3

View file

@ -346,13 +346,13 @@ class MultimediaViewerHooks {
* @param OutputPage $out
*/
public static function makeGlobalVariablesScript( &$vars, OutputPage $out ) {
global $wgDefaultUserOptions;
$defaultUserOptions = User::getDefaultOptions();
$user = $out->getUser();
$vars['wgMediaViewerOnClick'] = self::shouldHandleClicks( $user );
// needed because of bug 69942; could be different for anon and logged-in
$vars['wgMediaViewerEnabledByDefault'] =
!empty( $wgDefaultUserOptions['multimediaviewer-enable'] );
!empty( $defaultUserOptions['multimediaviewer-enable'] );
}
/**