jenkins-bot has submitted this change and it was merged. Change subject: Show the newarticle CX campaign to VE users only in the main namespace ......................................................................
Show the newarticle CX campaign to VE users only in the main namespace Bug: T135685 Change-Id: If642960593c5abad7fd663db2dac2c49f0d32c04 --- M ContentTranslation.hooks.php 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Nikerabbit: Looks good to me, approved jenkins-bot: Verified diff --git a/ContentTranslation.hooks.php b/ContentTranslation.hooks.php index 14beb58..0ad8b9f 100644 --- a/ContentTranslation.hooks.php +++ b/ContentTranslation.hooks.php @@ -47,12 +47,16 @@ $title = $out->getTitle(); $user = $out->getUser(); - // Check if CX is available for current user. + // Load the new article campaign for VisualEditor if it's relevant. + // Done separately from loading the newarticle campaign for the + // wiki syntax editor because of the different actions with which + // the editing page is loaded. if ( !self::isEnabledForUser( $user ) ) { if ( !$title->exists() && $wgContentTranslationCampaigns['newarticle'] && !$out->getRequest()->getCookie( 'cx_campaign_newarticle_hide', '' ) && + $title->inNamespace( NS_MAIN ) && !$user->isAnon() ) { $out->addModules( 'ext.cx.campaigns.newarticle.veloader' ); -- To view, visit https://gerrit.wikimedia.org/r/289602 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: If642960593c5abad7fd663db2dac2c49f0d32c04 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/ContentTranslation Gerrit-Branch: master Gerrit-Owner: Amire80 <amir.ahar...@mail.huji.ac.il> Gerrit-Reviewer: Nikerabbit <niklas.laxst...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits