Prevent predefined object methods to be interpreted as valid language
Change-Id: I97bb0f8eef6a9361118527cecb4680aeac51b7e1
This commit is contained in:
@@ -19,6 +19,8 @@
|
|||||||
( function ( $, mw ) {
|
( function ( $, mw ) {
|
||||||
'use strict';
|
'use strict';
|
||||||
|
|
||||||
|
var hasOwn = Object.prototype.hasOwnProperty;
|
||||||
|
|
||||||
mw.uls = mw.uls || {};
|
mw.uls = mw.uls || {};
|
||||||
mw.uls.previousLanguagesCookie = 'uls-previous-languages';
|
mw.uls.previousLanguagesCookie = 'uls-previous-languages';
|
||||||
mw.uls.previousLanguageAutonymCookie = 'uls-previous-language-autonym';
|
mw.uls.previousLanguageAutonymCookie = 'uls-previous-language-autonym';
|
||||||
@@ -163,7 +165,7 @@
|
|||||||
var target;
|
var target;
|
||||||
|
|
||||||
// If the language is already known and defined, just use it
|
// If the language is already known and defined, just use it
|
||||||
if ( $.fn.uls.defaults.languages[ langCode ] !== undefined ) {
|
if ( hasOwn.call( $.fn.uls.defaults.languages, langCode ) ) {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -174,7 +176,7 @@
|
|||||||
if ( target ) {
|
if ( target ) {
|
||||||
// Check that the redirect's target is known
|
// Check that the redirect's target is known
|
||||||
// to this instance of ULS
|
// to this instance of ULS
|
||||||
return $.fn.uls.defaults.languages[ target ] !== undefined;
|
return hasOwn.call( $.fn.uls.defaults.languages, target );
|
||||||
}
|
}
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
|
|||||||
Reference in New Issue
Block a user