From d682ba17c7eb890afe8180e957d0e2aebaa298f6 Mon Sep 17 00:00:00 2001 From: fkloft Date: Thu, 16 Apr 2015 00:11:03 +0200 Subject: [PATCH] [missions] bugfix: mwpid now includes waypoint index otherwise, you wouldn't be able to mark waypoints for hidden missions or duplicate portals (i.e. when a portal appears more than once in a mission) --- plugins/missions.user.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/plugins/missions.user.js b/plugins/missions.user.js index 3f51e792..606673c2 100644 --- a/plugins/missions.user.js +++ b/plugins/missions.user.js @@ -487,7 +487,7 @@ window.plugin.missions = { else title.textContent = 'Unknown'; - var mwpid = mission.guid + '-' + waypoint.guid; + var mwpid = mission.guid + '-' + index + '-' + waypoint.guid; var checked = this.checkedWaypoints[mwpid]; var label = container.appendChild(document.createElement('label')); @@ -495,7 +495,7 @@ window.plugin.missions = { var checkbox = label.appendChild(document.createElement('input')); checkbox.type = 'checkbox'; checkbox.addEventListener('change', function() { - plugin.missions.toggleWaypoint(mission.guid, waypoint.guid); + plugin.missions.toggleWaypoint(mission.guid, mwpid); }, false); checkbox.dataset['mission_mwpid'] = mwpid; @@ -524,8 +524,7 @@ window.plugin.missions = { return container; }, - toggleWaypoint: function(mid, wpid, dontsave) { - var mwpid = mid + '-' + wpid; + toggleWaypoint: function(mid, mwpid, dontsave) { if(this.checkedWaypoints[mwpid]) delete this.checkedWaypoints[mwpid]; else