src/views/trials/trials-panel/reading/dicoms3D/components/tools/FusionJumpToPointTool.js

@@ -40,23 +40,17 @@ class FusionJumpToPointTool extends AnnotationDisplayTool {
       return
     }
     const { element, currentPoints } = evt.detail || {}
-    if (!element || !currentPoints) return
+    const worldPoint = currentPoints?.world
+    if (!element || !worldPoint || worldPoint.length < 3) return
 
     const enabledElement = getEnabledElement(element)
     const { viewport, renderingEngine } = enabledElement || {}
     if (!viewport || !renderingEngine) return
 
-    const sourceIsMip = this._isMipViewportId(viewport.id)
-    const interactionWorldPoint = this._getInteractionWorldPoint(viewport, currentPoints, !sourceIsMip)
-    if (!interactionWorldPoint || interactionWorldPoint.length < 3) return
-    const selectedPoint = sourceIsMip
-      ? this._resolveSelectedPoint(viewport, interactionWorldPoint)
-      : interactionWorldPoint
+    const selectedPoint = this._resolveSelectedPoint(viewport, worldPoint)
     if (!selectedPoint || selectedPoint.length < 3) return
 
-    this.setPoint(selectedPoint, viewport.id, renderingEngine.id, {
-      jumpToTargetViewports: sourceIsMip,
-    })
+    this.setPoint(selectedPoint, viewport.id, renderingEngine.id)
   }
 
   mouseDownCallback(evt) {
@@ -76,14 +70,15 @@ class FusionJumpToPointTool extends AnnotationDisplayTool {
     const { viewport, renderingEngine } = enabledElement || {}
     if (!viewport || !renderingEngine) return
 
-    const sourceIsMip = this._isMipViewportId(viewport.id)
-    const worldPoint = this._getInteractionWorldPoint(viewport, currentPoints, !sourceIsMip)
+    let worldPoint = currentPoints?.world
+    if ((!worldPoint || worldPoint.length < 3) && currentPoints?.canvas && viewport.canvasToWorld) {
+      worldPoint = viewport.canvasToWorld(currentPoints.canvas)
+    }
     if (!worldPoint || worldPoint.length < 3) return
 
     const annotation = this._getViewportCrosshairAnnotation(viewport)
-    const sourceViewportId = sourceIsMip
-      ? (this.dragSourceViewportId || annotation?.data?.sourceViewportId || viewport.id)
-      : viewport.id
+    const sourceViewportId = this.dragSourceViewportId || annotation?.data?.sourceViewportId || viewport.id
+    const sourceIsMip = this._isMipViewportId(viewport.id)
     this.setPoint(worldPoint, sourceViewportId, renderingEngine.id, {
       jumpToTargetViewports: sourceIsMip,
     })
@@ -350,27 +345,6 @@ class FusionJumpToPointTool extends AnnotationDisplayTool {
     return true
   }
 
-  _getInteractionWorldPoint(viewport, currentPoints, preferCanvasToWorld = false) {
-    const worldPoint = currentPoints?.world
-    const canvasPoint = currentPoints?.canvas
-    if (preferCanvasToWorld && canvasPoint && viewport?.canvasToWorld) {
-      const worldFromCanvas = viewport.canvasToWorld(canvasPoint)
-      if (worldFromCanvas && worldFromCanvas.length >= 3) {
-        return worldFromCanvas
-      }
-    }
-    if (worldPoint && worldPoint.length >= 3) {
-      return worldPoint
-    }
-    if (canvasPoint && viewport?.canvasToWorld) {
-      const worldFromCanvas = viewport.canvasToWorld(canvasPoint)
-      if (worldFromCanvas && worldFromCanvas.length >= 3) {
-        return worldFromCanvas
-      }
-    }
-    return null
-  }
-
   _dispatchPointEvent(worldPoint, sourceViewportId, crosshairAppearance) {
     const { dispatchEventName } = this.configuration
     if (!dispatchEventName || typeof window === 'undefined') return