Catrope has uploaded a new change for review.

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

Change subject: DesktopArticleTarget.init: Tolerate missing wgRelevantPageName
......................................................................

DesktopArticleTarget.init: Tolerate missing wgRelevantPageName

For some reason this happens on some zhwiki pages.

Bug: T116693
Change-Id: I2880b8e7f2ed933928b155dd8727012f79854e1d
---
M modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/22/249022/1

diff --git a/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js 
b/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
index 4b39277..469b3b7 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
@@ -567,6 +567,9 @@
                // Only in supported skins
                $.inArray( mw.config.get( 'skin' ), conf.skins ) !== -1 &&
 
+               // Only where wgRelevantPageName is present (should be 
everywhere, but see T116693)
+               mw.config.get( 'wgRelevantPageName' ) !== null &&
+
                // Only in enabled namespaces
                $.inArray(
                        new mw.Title( mw.config.get( 'wgRelevantPageName' ) 
).getNamespaceId(),

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2880b8e7f2ed933928b155dd8727012f79854e1d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>

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

Reply via email to