image tools: event handlers should not return 'false' when it's disabled or toolData is unavailable (PR #190)

Pull request
https://github.com/chafey/cornerstoneTools/pull/190

Packages/ohif-cornerstone/client/cornerstoneTools.js

@@ -2424,7 +2424,7 @@ if (typeof cornerstoneTools === 'undefined') {
         }
 
         if (e.data && e.data.mouseButtonMask && !cornerstoneTools.isMouseButtonEnabled(eventData.which, e.data.mouseButtonMask)) {
-            return false;
+            return;
         }
 
         var config = cornerstoneTools.arrowAnnotate.getConfiguration();
@@ -2473,7 +2473,7 @@ if (typeof cornerstoneTools === 'undefined') {
         }
 
         if (e.data && e.data.mouseButtonMask && !cornerstoneTools.isMouseButtonEnabled(eventData.which, e.data.mouseButtonMask)) {
-            return false;
+            return;
         }
 
         var config = cornerstoneTools.arrowAnnotate.getConfiguration();
@@ -2483,7 +2483,7 @@ if (typeof cornerstoneTools === 'undefined') {
 
         // now check to see if there is a handle we can move
         if (!toolData) {
-            return false;
+            return;
         }
 
         if (eventData.handlePressed) {
@@ -5787,7 +5787,7 @@ if (typeof cornerstoneTools === 'undefined') {
         }
 
         if (e.data && e.data.mouseButtonMask && !cornerstoneTools.isMouseButtonEnabled(eventData.which, e.data.mouseButtonMask)) {
-            return false;
+            return;
         }
 
         var config = cornerstoneTools.seedAnnotate.getConfiguration();
@@ -5797,7 +5797,7 @@ if (typeof cornerstoneTools === 'undefined') {
 
         // now check to see if there is a handle we can move
         if (!toolData) {
-            return false;
+            return;
         }
 
         for (var i = 0; i < toolData.data.length; i++) {
@@ -6440,7 +6440,7 @@ if (typeof cornerstoneTools === 'undefined') {
         }
 
         if (e.data && e.data.mouseButtonMask && !cornerstoneTools.isMouseButtonEnabled(eventData.which, e.data.mouseButtonMask)) {
-            return false;
+            return;
         }
 
         var config = cornerstoneTools.textMarker.getConfiguration();
@@ -6450,7 +6450,7 @@ if (typeof cornerstoneTools === 'undefined') {
 
         // now check to see if there is a handle we can move
         if (!toolData) {
-            return false;
+            return;
         }
 
         for (var i = 0; i < toolData.data.length; i++) {
@@ -6502,7 +6502,7 @@ if (typeof cornerstoneTools === 'undefined') {
 
         // now check to see if there is a handle we can move
         if (!toolData) {
-            return false;
+            return;
         }
 
         if (eventData.handlePressed) {