Bug fixing: nonTarget tool measurements will be active when row is selected

This commit is contained in:
Evren Ozkan 2015-11-09 15:35:06 -05:00
parent 1e61e487c0
commit aa5e23cf8e
4 changed files with 21 additions and 12 deletions

View File

@ -1104,7 +1104,7 @@ if (typeof cornerstoneTools === 'undefined') {
imageNeedsUpdate = true; imageNeedsUpdate = true;
} }
if ((mouseToolInterface.pointInsideRect(eventData.element, data, coords) && !data.active) || (!mouseToolInterface.pointInsideRect(eventData.element, data, coords) && data.active)) { if (mouseToolInterface.pointInsideRect(eventData.element, data, coords) && !data.active || !mouseToolInterface.pointInsideRect(eventData.element, data, coords) && data.active) {
data.active = !data.active; data.active = !data.active;
imageNeedsUpdate = true; imageNeedsUpdate = true;
} }

View File

@ -105,19 +105,19 @@ var cornerstoneTools = (function($, cornerstone, cornerstoneMath, cornerstoneToo
} }
function suscribeLesionToolModifiedEvent(element) { function suscribeLesionToolSelectedEvent(element) {
var elementEvents = $._data(element, "events"); var elementEvents = $._data(element, "events");
var index = Object.keys(elementEvents).indexOf("LesionToolModified"); var index = Object.keys(elementEvents).indexOf("LesionToolSelected");
if (index < 0) { if (index < 0) {
// Subscribe LesionToolModified and calls measurementModified function when lesion measurement is changed or updated. // Subscribe LesionToolSelected and calls measurementModified function when lesion measurement is changed or updated.
$(element).on("LesionToolModified", measurementModified); $(element).on("LesionToolSelected", measurementModified);
} }
} }
///////// BEGIN IMAGE RENDERING /////// ///////// BEGIN IMAGE RENDERING ///////
function onImageRendered(e, eventData) { function onImageRendered(e, eventData) {
suscribeLesionToolModifiedEvent(e.currentTarget); suscribeLesionToolSelectedEvent(e.currentTarget);
// if we have no toolData for this element, return immediately as there is nothing to do // if we have no toolData for this element, return immediately as there is nothing to do
var toolData = cornerstoneTools.getToolState(e.currentTarget, toolType); var toolData = cornerstoneTools.getToolState(e.currentTarget, toolType);

View File

@ -187,10 +187,10 @@
function suscribeNonTargetToolModifiedEvent(element) { function suscribeNonTargetToolModifiedEvent(element) {
var elementEvents = $._data(element, "events"); var elementEvents = $._data(element, "events");
var index = Object.keys(elementEvents).indexOf("nonTargetToolModified"); var index = Object.keys(elementEvents).indexOf("NonTargetToolSelected");
if (index < 0) { if (index < 0) {
// Subscribe LesionToolModified and calls measurementModified function when lesion measurement is changed or updated. // Subscribe LesionToolModified and calls measurementModified function when lesion measurement is changed or updated.
$(element).on("nonTargetToolModified", measurementModified); $(element).on("NonTargetToolSelected", measurementModified);
} }
} }
@ -346,6 +346,7 @@
}); });
} }
function doubleClickCallback(e, eventData) { function doubleClickCallback(e, eventData) {
var element = eventData.element; var element = eventData.element;
var data; var data;

View File

@ -78,11 +78,19 @@ function updateLesions(e) {
eventData.type = "inactive"; eventData.type = "inactive";
} }
if(!isTarget) { if (!isTarget) {
$(element).trigger("nonTargetToolModified", eventData); $(element).trigger("NonTargetToolSelected", eventData);
// Deactivate lesion tool measurements
eventData.type = "inactive";
$(element).trigger("LesionToolSelected", eventData);
return; return;
} }
$(element).trigger("LesionToolModified", eventData); $(element).trigger("LesionToolSelected", eventData);
// Deactivate nonTarget tool measurements
eventData.type = "inactive";
$(element).trigger("NonTargetToolSelected", eventData);
}); });
} }