Remove jQuery.inArray usages
Replace jQuery.inArray with JS native Array.prototype.indexOf Change-Id: I0031f0385e440a34846334d158425182542a9ac9
This commit is contained in:
committed by
jenkins-bot
parent
54a3313e1a
commit
5a1da4dbaa
@@ -420,7 +420,7 @@
|
||||
|
||||
$( '#mw-content-text [lang]' ).each( function ( i, el ) {
|
||||
var lang = convertMediaWikiLanguageCodeToULS( $( el ).attr( 'lang' ) );
|
||||
if ( $.inArray( lang, languagesInText ) === -1 && $.inArray( lang, languages ) >= 0 ) {
|
||||
if ( languagesInText.indexOf( lang ) === -1 && languages.indexOf( lang ) >= 0 ) {
|
||||
languagesInText.push( lang );
|
||||
}
|
||||
} );
|
||||
|
||||
@@ -232,7 +232,7 @@
|
||||
|
||||
for ( lang in suggestedLanguages ) {
|
||||
// Skip already found languages
|
||||
if ( $.inArray( suggestedLanguages[ lang ], languagesForButtons ) > -1 ) {
|
||||
if ( languagesForButtons.indexOf( suggestedLanguages[ lang ] ) > -1 ) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
||||
@@ -51,9 +51,9 @@
|
||||
previousIMELanguages = $.ime.preferences.getPreviousLanguages() || [];
|
||||
imeLanguageList = previousIMELanguages.concat( mw.uls.getFrequentLanguageList() );
|
||||
|
||||
$.each( imeLanguageList, function ( i, v ) {
|
||||
if ( $.inArray( v, unique ) === -1 ) {
|
||||
unique.push( v );
|
||||
imeLanguageList.forEach( function ( lang ) {
|
||||
if ( unique.indexOf( lang ) === -1 ) {
|
||||
unique.push( lang );
|
||||
}
|
||||
} );
|
||||
|
||||
|
||||
@@ -242,23 +242,23 @@
|
||||
// after selecting a different language
|
||||
$languages.empty();
|
||||
|
||||
// Selected IME language may be different, and it must
|
||||
// be present, too
|
||||
// Selected IME language may be different, and it must be present, too
|
||||
if ( $.uls.data.languages[ selectedImeLanguage ] &&
|
||||
$.inArray( selectedImeLanguage, languagesForButtons ) === -1 ) {
|
||||
languagesForButtons.indexOf( selectedImeLanguage ) === -1
|
||||
) {
|
||||
languagesForButtons.push( selectedImeLanguage );
|
||||
}
|
||||
|
||||
// UI language must always be present
|
||||
if ( this.uiLanguage !== this.contentLanguage &&
|
||||
$.uls.data.languages[ this.uiLanguage ] &&
|
||||
$.inArray( this.uiLanguage, languagesForButtons ) === -1 ) {
|
||||
languagesForButtons.indexOf( this.uiLanguage ) === -1 ) {
|
||||
languagesForButtons.push( this.uiLanguage );
|
||||
}
|
||||
|
||||
for ( lang in suggestedLanguages ) {
|
||||
// Skip already found languages
|
||||
if ( $.inArray( suggestedLanguages[ lang ], languagesForButtons ) > -1 ) {
|
||||
if ( languagesForButtons.indexOf( suggestedLanguages[ lang ] ) > -1 ) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
||||
@@ -91,7 +91,7 @@
|
||||
|
||||
defaultFont = repository.defaultFont( language );
|
||||
|
||||
if ( classes && $.inArray( 'autonym', classes ) >= 0 ) {
|
||||
if ( classes && classes.indexOf( 'autonym' ) >= 0 ) {
|
||||
// Do not load font for showing autonym.
|
||||
return null;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user