diff --git a/LesionTracker/client/components/viewer.js b/LesionTracker/client/components/viewer.js index df895021d..c2522b99a 100644 --- a/LesionTracker/client/components/viewer.js +++ b/LesionTracker/client/components/viewer.js @@ -44,7 +44,7 @@ Template.viewer.onCreated(function() { }, nonTarget: function() { toolManager.setActiveTool("nonTarget"); - }, + } }; // The hotkey can also be an array (e.g. ["NUMPAD0", "0"]) diff --git a/Packages/lesiontracker/client/compatibility/LesionManager.js b/Packages/lesiontracker/client/compatibility/LesionManager.js index 2ef59570b..d90557dbb 100644 --- a/Packages/lesiontracker/client/compatibility/LesionManager.js +++ b/Packages/lesiontracker/client/compatibility/LesionManager.js @@ -37,7 +37,7 @@ var LesionManager = (function() { // Find the specific lesion to be updated var existingMeasurement; - if (lesionData.id) { + if (lesionData.id && lesionData.id !== 'notready') { existingMeasurement = Measurements.findOne(lesionData.id); } else { existingMeasurement = Measurements.findOne({ diff --git a/Packages/lesiontracker/client/compatibility/deleteLesionKeyboardTool.js b/Packages/lesiontracker/client/compatibility/deleteLesionKeyboardTool.js index d8884ed73..c7dad92e5 100644 --- a/Packages/lesiontracker/client/compatibility/deleteLesionKeyboardTool.js +++ b/Packages/lesiontracker/client/compatibility/deleteLesionKeyboardTool.js @@ -22,6 +22,7 @@ }); } + // TODO = Check if we have the same function already in Cornerstone Tools function getNearbyToolData(element, coords, toolTypes) { var allTools = toolManager.getTools(); var pointNearTool = false; @@ -77,11 +78,17 @@ var toolTypes = ["lesion", "nonTarget"]; var nearbyToolData = getNearbyToolData(eventData.element, eventData.currentPoints.canvas, toolTypes); - if (nearbyToolData) { - removeMeasurementTimepoint(nearbyToolData.nearbyTool, - nearbyToolData.nearbyToolIndex, - nearbyToolData.nearbyToolType); + if (!nearbyToolData) { + return; } + + // TODO= Refactor this so the confirmation dialog is an + // optional settable callback in the tool's configuration + showConfirmDialog(function() { + removeMeasurementTimepoint(nearbyToolData.nearbyTool, + nearbyToolData.nearbyToolIndex, + nearbyToolData.nearbyToolType); + }); } } diff --git a/Packages/lesiontracker/client/components/confirmDeleteDialog/confirmDeleteDialog.html b/Packages/lesiontracker/client/components/confirmDeleteDialog/confirmDeleteDialog.html index 81f72abac..978fd1fe5 100644 --- a/Packages/lesiontracker/client/components/confirmDeleteDialog/confirmDeleteDialog.html +++ b/Packages/lesiontracker/client/components/confirmDeleteDialog/confirmDeleteDialog.html @@ -2,7 +2,7 @@
Are you sure you would like to remove this measurement?
- - + +