MarkTraceur has uploaded a new change for review.
https://gerrit.wikimedia.org/r/118648
Change subject: Add user preference for non-beta disabling
......................................................................
Add user preference for non-beta disabling
Change-Id: I3f581975cfdf33bc15b8a4b23549c6401b4bfb87
---
M MultimediaViewer.i18n.php
M MultimediaViewerHooks.php
2 files changed, 22 insertions(+), 4 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MultimediaViewer
refs/changes/48/118648/1
diff --git a/MultimediaViewer.i18n.php b/MultimediaViewer.i18n.php
index f303f88..9f65fce 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-optout-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',
@@ -94,6 +95,7 @@
'multimediaviewer-desc-nil' => 'Text to be used when no description is
available.',
'multimediaviewer-pref' => 'Preference title',
'multimediaviewer-pref-desc' => 'Description of preference',
+ 'multimediaviewer-optout-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 669a3f3..e3a88aa 100644
--- a/MultimediaViewerHooks.php
+++ b/MultimediaViewerHooks.php
@@ -38,11 +38,15 @@
if ( $wgMediaViewerIsInBeta && class_exists( 'BetaFeatures' ) )
{
return BetaFeatures::isFeatureEnabled( $user,
'multimedia-viewer' );
- } else if ( $wgEnableMediaViewerForLoggedInUsersOnly ) {
- return $user->isLoggedIn();
+ } else if ( $user->getOption( 'media-vewer-enable' ) ) {
+ if ( $wgEnableMediaViewerForLoggedInUsersOnly ) {
+ return $user->isLoggedIn();
+ } else {
+ // Default to enabling for everyone.
+ return true;
+ }
} else {
- // Default to enabling for everyone.
- return true;
+ return false;
}
}
@@ -112,6 +116,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-optout-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/118648
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: wmf/1.23wmf17
Gerrit-Owner: MarkTraceur <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
