Jdlrobson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/249872

Change subject: Don't set target=mobile on blacklisted pages
......................................................................

Don't set target=mobile on blacklisted pages

To test:
$wgMFNoMobilePages = array( 'Blacklisted test' )

Bug: T116934
Change-Id: I63e42e779c85c0beb75ee1069267b90ce936dc80
---
M includes/MobileFrontend.hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/72/249872/1

diff --git a/includes/MobileFrontend.hooks.php 
b/includes/MobileFrontend.hooks.php
index 61c0c59..345bb86 100644
--- a/includes/MobileFrontend.hooks.php
+++ b/includes/MobileFrontend.hooks.php
@@ -315,7 +315,7 @@
         */
        public static function onSkinPreloadExistence( array &$titles, Skin 
$skin ) {
                $context = MobileContext::singleton();
-               if ( $context->shouldDisplayMobileView() ) {
+               if ( $context->shouldDisplayMobileView() && 
!$context->isBlacklistedPage() ) {
                        $skin->getOutput()->setTarget( 'mobile' );
                }
                return true;

-- 
To view, visit https://gerrit.wikimedia.org/r/249872
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I63e42e779c85c0beb75ee1069267b90ce936dc80
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to