bryce/jquery.uls
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Change code structure a bit per code review comments

Browse Source
This commit is contained in:
Niklas Laxström
2018-02-08 15:26:15 +02:00
parent 0352b3df0d
commit eccd0e1945
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/jquery.uls.languagefilter.js
View File

@@ -208,17 +208,16 @@
results = []; results = [];
$.each( result.languagesearch, function ( apiCode, name ) { $.each( result.languagesearch, function ( apiCode, name ) {
var code, var code, redirect;
redirect = $.uls.data.isRedirect( apiCode );
if ( this.options.languages[ apiCode ] ) { if ( this.options.languages[ apiCode ] ) {
code = apiCode; code = apiCode;
} else if ( redirect && this.options.languages[ redirect ] ) {
// Language tags are messy. Try to make sure we handle
// them gracefully with regards to redirects.
code = redirect;
} else { } else {
return; redirect = $.uls.data.isRedirect( apiCode );
if ( !redirect || !this.options.languages[ redirect ] ) {
return;
}
code = redirect;
} }
// Because of the redirect checking above, we might get duplicates. // Because of the redirect checking above, we might get duplicates.