fix: moveNewHandle should call endHandler with success: false when th…

…e measurement is removed by `anyHandleOutsideImage` check (#1328)
cornerstonejs · Nov 18, 2020 · 8a261a2 · 8a261a2
1 parent d9a2c2b
commit 8a261a2
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/manipulators/moveNewHandle.js b/src/manipulators/moveNewHandle.js
@@ -236,6 +236,7 @@ function _moveEndHandler(
 ) {
   const eventData = evt.detail;
   const { element, currentPoints } = eventData;
+  let moveNewHandleSuccessful = true;
 
   if (options.hasMoved === false) {
     return;
@@ -289,6 +290,7 @@ function _moveEndHandler(
     anyHandlesOutsideImage(evt.detail, annotation.handles)
   ) {
     annotation.cancelled = true;
+    moveNewHandleSuccessful = false;
     removeToolState(element, toolName, annotation);
   }
 
@@ -301,7 +303,7 @@ function _moveEndHandler(
       moveEndHandler,
     },
     doneMovingCallback,
-    true
+    moveNewHandleSuccessful
   );
 }