[player-tracker] rename "guid" where no guid is available; fix marker click not selecting portal

This commit is contained in:
fkloft
2014-09-13 21:30:54 +02:00
parent ad53f785d3
commit 5ccc42245b

View File

@ -75,7 +75,11 @@ window.plugin.playerTracker.stored = {};
plugin.playerTracker.onClickListener = function(event) { plugin.playerTracker.onClickListener = function(event) {
var marker = event.target; var marker = event.target;
window.renderPortalDetails(marker.options.referenceToPortal);
var ll = marker.options.referenceToPortal.split(",");
var guid = window.findPortalGuidByPositionE6(ll[0], ll[1]);
if(guid) window.renderPortalDetails(guid);
if (marker.options.desc) { if (marker.options.desc) {
plugin.playerTracker.playerPopup.setContent(marker.options.desc); plugin.playerTracker.playerPopup.setContent(marker.options.desc);
plugin.playerTracker.playerPopup.setLatLng(marker.getLatLng()); plugin.playerTracker.playerPopup.setLatLng(marker.getLatLng());
@ -141,7 +145,7 @@ window.plugin.playerTracker.processNewData = function(data) {
if(json[1] < limit) return true; if(json[1] < limit) return true;
// find player and portal information // find player and portal information
var plrname, lat, lng, guid, name, address; var plrname, lat, lng, id=null, name, address;
var skipThisMessage = false; var skipThisMessage = false;
$.each(json[2].plext.markup, function(ind, markup) { $.each(json[2].plext.markup, function(ind, markup) {
switch(markup[0]) { switch(markup[0]) {
@ -166,7 +170,7 @@ window.plugin.playerTracker.processNewData = function(data) {
lng = lng ? lng : markup[1].lngE6/1E6; lng = lng ? lng : markup[1].lngE6/1E6;
// no GUID in the data any more - but we need some unique string. use the latE6,lngE6 // no GUID in the data any more - but we need some unique string. use the latE6,lngE6
guid = guid ? guid : markup[1].latE6+","+markup[1].lngE6; id = markup[1].latE6+","+markup[1].lngE6;
name = name ? name : markup[1].name; name = name ? name : markup[1].name;
address = address ? address : markup[1].address; address = address ? address : markup[1].address;
@ -175,11 +179,11 @@ window.plugin.playerTracker.processNewData = function(data) {
}); });
// skip unusable events // skip unusable events
if(!plrname || !lat || !lng || !guid || skipThisMessage) return true; if(!plrname || !lat || !lng || !id || skipThisMessage) return true;
var newEvent = { var newEvent = {
latlngs: [[lat, lng]], latlngs: [[lat, lng]],
guids: [guid], ids: [id],
time: json[1], time: json[1],
name: name, name: name,
address: address address: address
@ -211,7 +215,7 @@ window.plugin.playerTracker.processNewData = function(data) {
// this is multiple resos destroyed at the same time. // this is multiple resos destroyed at the same time.
if(evts[cmp].time === json[1]) { if(evts[cmp].time === json[1]) {
evts[cmp].latlngs.push([lat, lng]); evts[cmp].latlngs.push([lat, lng]);
evts[cmp].guids.push(guid); evts[cmp].ids.push(id);
plugin.playerTracker.stored[plrname].events = evts; plugin.playerTracker.stored[plrname].events = evts;
return true; return true;
} }
@ -345,15 +349,15 @@ window.plugin.playerTracker.drawData = function() {
var eventPortal = [] var eventPortal = []
var closestPortal; var closestPortal;
var mostPortals = 0; var mostPortals = 0;
$.each(last.guids, function(i, guid) { $.each(last.ids, function(i, id) {
if(eventPortal[guid]) { if(eventPortal[id]) {
eventPortal[guid]++; eventPortal[id]++;
} else { } else {
eventPortal[guid] = 1; eventPortal[id] = 1;
} }
if(eventPortal[guid] > mostPortals) { if(eventPortal[id] > mostPortals) {
mostPortals = eventPortal[guid]; mostPortals = eventPortal[id];
closestPortal = guid; closestPortal = id;
} }
}); });