Geoclient fixes for https and WMF
* Don't call freegeoip on https, it wont work * Use Geo instead of GEO like WMF Change-Id: I9ebdd9f3518e15e70122a9c8c8d64240d5e44c5b
This commit is contained in:
committed by
Gerrit Code Review
parent
8d5307852a
commit
3c9e7249a8
@@ -22,19 +22,31 @@
|
||||
|
||||
mw.uls = mw.uls || {};
|
||||
mw.uls.setGeo = function ( data ) {
|
||||
window.GEO = data;
|
||||
window.Geo = data;
|
||||
};
|
||||
|
||||
mw.uls.getCountryCode = function () {
|
||||
/*jshint camelcase: false*/
|
||||
return window.GEO && ( window.GEO.country || window.GEO.country_code );
|
||||
return window.Geo && ( window.Geo.country || window.Geo.country_code );
|
||||
};
|
||||
|
||||
var settings = {
|
||||
cache: true,
|
||||
dataType: 'jsonp',
|
||||
jsonpCallback: 'mw.uls.setGeo'
|
||||
};
|
||||
$.ajax( mw.config.get( 'wgULSGeoService' ), settings );
|
||||
var currentProto, httpOnly,
|
||||
service = mw.config.get( 'wgULSGeoService' );
|
||||
|
||||
// Call the service only if defined, and if the current
|
||||
// protocol is https, only if the service is not configured
|
||||
// with http:// as the protocol
|
||||
if ( service ) {
|
||||
httpOnly = service.substring(0, 7) === 'http://';
|
||||
currentProto = document.location.protocol;
|
||||
if ( !httpOnly || currentProto === 'http:' ) {
|
||||
var settings = {
|
||||
cache: true,
|
||||
dataType: 'jsonp',
|
||||
jsonpCallback: 'mw.uls.setGeo'
|
||||
};
|
||||
|
||||
$.ajax( service, settings );
|
||||
}
|
||||
}
|
||||
|
||||
}( mediaWiki, jQuery ) );
|
||||
|
||||
@@ -63,7 +63,8 @@
|
||||
};
|
||||
|
||||
mw.uls.getFrequentLanguageList = function () {
|
||||
var unique = [],
|
||||
var countryCode,
|
||||
unique = [],
|
||||
list = [
|
||||
mw.config.get( 'wgUserLanguage' ),
|
||||
mw.config.get( 'wgContentLanguage' ),
|
||||
@@ -72,8 +73,9 @@
|
||||
.concat( mw.uls.getPreviousLanguages() )
|
||||
.concat( mw.uls.getAcceptLanguageList() );
|
||||
|
||||
if ( window.GEO ) {
|
||||
list = list.concat( $.uls.data.getLanguagesInTerritory( mw.uls.getCountryCode() ) );
|
||||
countryCode = mw.uls.getCountryCode()
|
||||
if ( countryCode ) {
|
||||
list = list.concat( $.uls.data.getLanguagesInTerritory( countryCode ) );
|
||||
}
|
||||
|
||||
$.each( list, function ( i, v ) {
|
||||
|
||||
Reference in New Issue
Block a user