fix: referenceId and Synchronizer - Improve error handling and code r…

…obustness (#1612)

-  Removed unused focalPoint parameter from the volumeId query string construction in `BaseVolumeViewport.ts`.
-  Added a check for `eventSource` in `Synchronizer.ts` to prevent errors when it is undefined.
-  Updated error handling in `Synchronizer.ts` to return null instead of throwing an error when `RenderingEngine` or `viewport` is not found.
-  Ensured that `viewport.element` is returned safely by checking for the existence of the viewport.

packages/core/src/RenderingEngine/BaseVolumeViewport.ts

@@ -1790,7 +1790,7 @@ abstract class BaseVolumeViewport extends Viewport {
     const querySeparator = volumeId.includes('?') ? '&' : '?';
     return `volumeId:${volumeId}${querySeparator}sliceIndex=${sliceIndex}&viewPlaneNormal=${viewPlaneNormal.join(
       ','
-    )}&focalPoint=${focalPoint.join(',')}`;
+    )}`;
   }
 
   private _addVolumeId(volumeId: string): void {

packages/tools/src/store/SynchronizerManager/Synchronizer.ts

@@ -359,6 +359,11 @@ class Synchronizer {
 
     viewports.forEach((vp) => {
       const eventSource = this.getEventSource(vp);
+
+      if (!eventSource) {
+        return;
+      }
+
       eventSource.removeEventListener(
         Enums.Events.ELEMENT_DISABLED,
         disableHandler
@@ -380,9 +385,13 @@ class Synchronizer {
     const { renderingEngineId, viewportId } = viewportInfo;
     const renderingEngine = getRenderingEngine(renderingEngineId);
     if (!renderingEngine) {
-      throw new Error(`No RenderingEngine for Id: ${renderingEngineId}`);
+      return null;
+    }
+    const viewport = renderingEngine.getViewport(viewportId);
+    if (!viewport) {
+      return null;
     }
-    return renderingEngine.getViewport(viewportId).element;
+    return viewport.element;
   }
 }