Kaldari has uploaded a new change for review.
https://gerrit.wikimedia.org/r/158548
Change subject: Add a testing mode to WikiGrok
......................................................................
Add a testing mode to WikiGrok
Also add an extra undefined check to avoid JS errors.
Change-Id: I15c78c8363889d93d068ad35ce92b8a0913f5ec9
---
M javascripts/modules/wikigrok/wikigrok.js
M javascripts/modules/wikigrok/wikigrokeval.js
2 files changed, 12 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/48/158548/1
diff --git a/javascripts/modules/wikigrok/wikigrok.js
b/javascripts/modules/wikigrok/wikigrok.js
index cb5168d..993dd1e 100644
--- a/javascripts/modules/wikigrok/wikigrok.js
+++ b/javascripts/modules/wikigrok/wikigrok.js
@@ -1,7 +1,16 @@
// Determine whether or not it is appropriate to load WikiGrok, and if so,
load it.
( function( M ) {
- var wikidataID = mw.config.get( 'wgWikibaseItemId' );
- //var wikidataID = 'Q508703';
+ var wikidataID = mw.config.get( 'wgWikibaseItemId' ),
+ idOverride;
+
+ // Allow query string override for testing, for example,
'?wikidataid=Q508703'
+ if ( !wikidataID ) {
+ idOverride = window.location.search.match( /wikidataid=([^&]*)/
);
+ if ( idOverride ) {
+ mw.config.set( 'wgWikibaseItemId', idOverride[1] );
+ wikidataID = idOverride[1];
+ }
+ }
// Only run in alpha mode
M.assertMode( [ 'alpha' ] );
diff --git a/javascripts/modules/wikigrok/wikigrokeval.js
b/javascripts/modules/wikigrok/wikigrokeval.js
index abfdb17..79161ee 100644
--- a/javascripts/modules/wikigrok/wikigrokeval.js
+++ b/javascripts/modules/wikigrok/wikigrokeval.js
@@ -24,7 +24,7 @@
WikiGrokDrawer;
// See if the page has any 'instance of' claims.
- if ( data.entities[wikidataID].claims.P31 !== undefined
) {
+ if ( data.entities !== undefined &&
data.entities[wikidataID].claims.P31 !== undefined ) {
instanceClaims =
data.entities[wikidataID].claims.P31;
$.each( instanceClaims, function( id, claim ) {
// See if any of the claims state that
the topic is a human.
--
To view, visit https://gerrit.wikimedia.org/r/158548
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I15c78c8363889d93d068ad35ce92b8a0913f5ec9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Kaldari <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
