phillip/ingress-intel-total-conversion
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

add sanity checks to lat/lng to avoid issues like #153 (fixes #153)

Browse Source
This commit is contained in:
Stefan Breunig 2013-02-13 20:01:44 +01:00
parent a63370cae9
commit 3bbca39353
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
code/location.js
View File

@ -5,11 +5,17 @@
// retrieves current position from map and stores it cookies // retrieves current position from map and stores it cookies
window.storeMapPosition = function() { window.storeMapPosition = function() {
var m = window.map.getCenter(); var m = window.map.getCenter();
writeCookie('ingress.intelmap.lat', m['lat']);
writeCookie('ingress.intelmap.lng', m['lng']); if(m['lat'] >= -90 && m['lat'] <= 90)
writeCookie('ingress.intelmap.lat', m['lat']);
if(m['lng'] >= -180 && m['lng'] <= 180)
writeCookie('ingress.intelmap.lng', m['lng']);
writeCookie('ingress.intelmap.zoom', window.map.getZoom()); writeCookie('ingress.intelmap.zoom', window.map.getZoom());
} }
// either retrieves the last shown position from a cookie, from the // either retrieves the last shown position from a cookie, from the
// URL or if neither is present, via Geolocation. If that fails, it // URL or if neither is present, via Geolocation. If that fails, it
// returns a map that shows the whole world. // returns a map that shows the whole world.
@ -28,6 +34,10 @@ window.getPosition = function() {
var lat = parseFloat(readCookie('ingress.intelmap.lat')) || 0.0; var lat = parseFloat(readCookie('ingress.intelmap.lat')) || 0.0;
var lng = parseFloat(readCookie('ingress.intelmap.lng')) || 0.0; var lng = parseFloat(readCookie('ingress.intelmap.lng')) || 0.0;
var z = parseInt(readCookie('ingress.intelmap.zoom')) || 17; var z = parseInt(readCookie('ingress.intelmap.zoom')) || 17;
if(lat < -90 || lat > 90) lat = 0.0;
if(lng < -180 || lng > 180) lng = 0.0;
return {center: new L.LatLng(lat, lng), zoom: z > 18 ? 18 : z}; return {center: new L.LatLng(lat, lng), zoom: z > 18 ? 18 : z};
} }