fix(Reduce cognitive complexity by splitting out calls): Attempt to reduce cognitive complexity rule

wayfarer3130 · wayfarer3130 · commit 6f9887e556ef · 2021-10-22T11:44:48.000-04:00
Reduce cognitive complexity by splitting code
diff --git a/src/tools/OverlayTool.js b/src/tools/OverlayTool.js
@@ -45,14 +45,8 @@ export default class OverlayTool extends BaseTool {
     }
   }
 
-  renderToolData(evt) {
-    const eventData = evt.detail;
-    const { enabledElement, image, viewport, canvasContext } = eventData;
-
-    if (!eventData || !enabledElement || !image) {
-      return;
-    }
-
+  setupRender(image) {
+    if (!image) return;
     const overlayPlaneMetadata = external.cornerstone.metaData.get(
       'overlayPlaneModule',
       image.imageId
@@ -66,12 +60,28 @@ export default class OverlayTool extends BaseTool {
       return;
     }
 
+    return overlayPlaneMetadata;
+  }
+
+  setupViewport(viewport) {
     if (viewport.overlayColor === undefined) {
       viewport.overlayColor =
         globalConfiguration.configuration.overlayColor || 'white';
     }
     // Allow turning off overlays by setting overlayColor to false
     if (viewport.overlayColor === false) return;
+    return true;
+  }
+
+  renderToolData(evt) {
+    const eventData = evt.detail;
+    const { enabledElement, image, viewport, canvasContext } = eventData;
+    const overlayPlaneMetadata = this.setupRender(image);
+
+    if (!eventData || !enabledElement || !overlayPlaneMetadata) {
+      return;
+    }
+    if (!this.setupViewport(viewport)) return;
 
     const imageWidth = image.columns;
     const imageHeight = image.rows;
