jenkins-bot has submitted this change and it was merged.

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(-)

Approvals:
  Bmansurov: Looks good to me, approved
  jenkins-bot: Verified



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: merged
Gerrit-Change-Id: I63e42e779c85c0beb75ee1069267b90ce936dc80
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
Gerrit-Reviewer: Bmansurov <[email protected]>
Gerrit-Reviewer: Jdlrobson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to