jenkins-bot has submitted this change and it was merged. Change subject: Fix fatal on empty query submodule list ......................................................................
Fix fatal on empty query submodule list Bug: 67577 Change-Id: I2a872f56a8c703da13f82570620d40928656a7ab --- M resources/main.js 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alex Monk: Looks good to me, approved Legoktm: Looks good to me, approved TTO: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/resources/main.js b/resources/main.js index 45da2d0..d16ff63 100644 --- a/resources/main.js +++ b/resources/main.js @@ -617,7 +617,7 @@ } ); if ( $action.val() === 'query' ) { var lists = [], metas = [], props = []; - $.each( $query.val(), function ( index, queryParam ) { + $.each( $query.val() || [], function ( index, queryParam ) { if ( queryParam.substring( 0, 5 ) == 'list=' ) { lists.push( queryParam.substring( 5 ) ); } else if ( queryParam.substring( 0, 5 ) == 'meta=' ) { -- To view, visit https://gerrit.wikimedia.org/r/168786 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a872f56a8c703da13f82570620d40928656a7ab Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/ApiSandbox Gerrit-Branch: master Gerrit-Owner: Umherirrender <umherirrender_de...@web.de> Gerrit-Reviewer: Alex Monk <kren...@wikimedia.org> Gerrit-Reviewer: Anomie <bjor...@wikimedia.org> Gerrit-Reviewer: Jackmcbarn <jackmcb...@gmail.com> Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com> Gerrit-Reviewer: TTO <at.li...@live.com.au> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits