Jeroen De Dauw has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/383045 )
Change subject: Use variable already in scope ...................................................................... Use variable already in scope Change-Id: Ieab09680b5ae509b2d6e77086809c1f9eb82ad48 --- M modules/dm/ext.advancedSearch.SearchModel.js 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/AdvancedSearch refs/changes/45/383045/1 diff --git a/modules/dm/ext.advancedSearch.SearchModel.js b/modules/dm/ext.advancedSearch.SearchModel.js index 7db542b..a7af335 100644 --- a/modules/dm/ext.advancedSearch.SearchModel.js +++ b/modules/dm/ext.advancedSearch.SearchModel.js @@ -194,7 +194,7 @@ if ( !fileType ) { return false; } - var generalFileType = this.getOption( 'filetype' ).replace( /\/.*/, '' ); + var generalFileType = fileType.replace( /\/.*/, '' ); return FILETYPES_WITH_DIMENSIONS.indexOf( generalFileType ) !== -1; }; -- To view, visit https://gerrit.wikimedia.org/r/383045 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ieab09680b5ae509b2d6e77086809c1f9eb82ad48 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/AdvancedSearch Gerrit-Branch: master Gerrit-Owner: Jeroen De Dauw <jeroended...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits