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

Change subject: Protect against JS error when langlinks API query returns an 
error
......................................................................


Protect against JS error when langlinks API query returns an error

This happens e.g. when opening VE on a redirect page.

Change-Id: I75b77da42013c3d446cbc9c57aebe0907a2b40dc
---
M modules/ve-mw/ui/pages/ve.ui.MWLanguagesPage.js
1 file changed, 2 insertions(+), 1 deletion(-)

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



diff --git a/modules/ve-mw/ui/pages/ve.ui.MWLanguagesPage.js 
b/modules/ve-mw/ui/pages/ve.ui.MWLanguagesPage.js
index 1462b10..9474750 100644
--- a/modules/ve-mw/ui/pages/ve.ui.MWLanguagesPage.js
+++ b/modules/ve-mw/ui/pages/ve.ui.MWLanguagesPage.js
@@ -95,7 +95,8 @@
  * Handle language items being loaded.
  */
 ve.ui.MWLanguagesPage.prototype.onAllLanguageItemsSuccess = function ( 
deferred, response ) {
-       var i, iLen, languages = [], langlinks = 
response.visualeditor.langlinks;
+       var i, iLen, languages = [],
+               langlinks = response && response.visualeditor && 
response.visualeditor.langlinks;
        if ( langlinks ) {
                for ( i = 0, iLen = langlinks.length; i < iLen; i++ ) {
                        languages.push( {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I75b77da42013c3d446cbc9c57aebe0907a2b40dc
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <roan.katt...@gmail.com>
Gerrit-Reviewer: Esanders <esand...@wikimedia.org>
Gerrit-Reviewer: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to