MarkTraceur has uploaded a new change for review.
https://gerrit.wikimedia.org/r/119981
Change subject: Add user preference for non-beta disabling
......................................................................
Add user preference for non-beta disabling
Change-Id: I3f581975cfdf33bc15b8a4b23549c6401b4bfb87
(cherry picked from commit b0b1446f5b27021674a7ad6e97c03b9577093713)
---
M MultimediaViewer.i18n.php
M MultimediaViewerHooks.php
2 files changed, 25 insertions(+), 5 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MultimediaViewer
refs/changes/81/119981/1
diff --git a/MultimediaViewer.i18n.php b/MultimediaViewer.i18n.php
index 47744f4..eb88de6 100644
--- a/MultimediaViewer.i18n.php
+++ b/MultimediaViewer.i18n.php
@@ -30,6 +30,7 @@
'multimediaviewer-desc-nil' => 'No description available.',
'multimediaviewer-pref' => 'Media Viewer',
'multimediaviewer-pref-desc' => 'Improve your multimedia viewing
experience with this new tool. It displays images in larger size on pages that
have thumbnails. Images are shown in a nicer fullscreen interface overlay, and
can also be viewed in full-size.',
+ 'multimediaviewer-optin-pref' => 'Enable new media viewing experience',
'multimediaviewer-file-page' => 'Go to corresponding file page',
'multimediaviewer-repository' => 'Learn more on $1',
'multimediaviewer-repository-local' => 'Learn more',
@@ -121,6 +122,7 @@
'multimediaviewer-desc-nil' => 'Text to be used when no description is
available.',
'multimediaviewer-pref' => 'Preference title',
'multimediaviewer-pref-desc' => 'Description of preference',
+ 'multimediaviewer-optin-pref' => 'Label for non-beta preference.',
'multimediaviewer-file-page' => 'Text for a link to the file page for
an image.',
'multimediaviewer-repository' => 'Link to the repository where the
image is hosted. Parameters:
* $1 - the display name of that site
diff --git a/MultimediaViewerHooks.php b/MultimediaViewerHooks.php
index 18e9657..34a234b 100644
--- a/MultimediaViewerHooks.php
+++ b/MultimediaViewerHooks.php
@@ -38,12 +38,18 @@
if ( $wgMediaViewerIsInBeta && class_exists( 'BetaFeatures' ) )
{
return BetaFeatures::isFeatureEnabled( $user,
'multimedia-viewer' );
- } else if ( $wgEnableMediaViewerForLoggedInUsersOnly ) {
- return $user->isLoggedIn();
- } else {
- // Default to enabling for everyone.
- return true;
}
+
+ if ( $user->getOption( 'media-vewer-enable' ) ) {
+ if ( $wgEnableMediaViewerForLoggedInUsersOnly ) {
+ return $user->isLoggedIn();
+ } else {
+ // Default to enabling for everyone.
+ return true;
+ }
+ }
+
+ return false;
}
/**
@@ -112,6 +118,18 @@
return true;
}
+ // Adds a default-enabled preference to gate the feature on non-beta
sites
+ public static function getPreferences( $user, &$prefs ) {
+ $prefs['media-viewer-enable'] = array(
+ 'type' => 'toggle',
+ 'label-message' => 'multimediaviewer-optin-pref',
+ 'section' => 'rendering/files',
+ 'default' => true,
+ );
+
+ return true;
+ }
+
/**
* Export variables used in both PHP and JS to keep DRY
* @param array $vars
--
To view, visit https://gerrit.wikimedia.org/r/119981
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f581975cfdf33bc15b8a4b23549c6401b4bfb87
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MultimediaViewer
Gerrit-Branch: master
Gerrit-Owner: MarkTraceur <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
