From 1c45d643acd1dbb2eaea3abe9fe020f4d01f8933 Mon Sep 17 00:00:00 2001
From: dannyrb <danny.ri.brown@gmail.com>
Date: Wed, 27 May 2020 22:21:35 -0400
Subject: [PATCH] missing useEffect unsubscribe

---
 platform/viewer/src/routes/Mode/Mode.jsx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/platform/viewer/src/routes/Mode/Mode.jsx b/platform/viewer/src/routes/Mode/Mode.jsx
index 80a2cf874..a728934ea 100644
--- a/platform/viewer/src/routes/Mode/Mode.jsx
+++ b/platform/viewer/src/routes/Mode/Mode.jsx
@@ -92,7 +92,7 @@ export default function ModeRoute({
 
     // TODO: This should be baked into core, not manuel?
     // DisplaySetService would wire this up?
-    DicomMetadataStore.subscribe(
+    const { unsubscribe } = DicomMetadataStore.subscribe(
       DicomMetadataStore.EVENTS.INSTANCES_ADDED,
       ({ StudyInstanceUID, SeriesInstanceUID }) => {
         const seriesMetadata = DicomMetadataStore.getSeries(
@@ -107,6 +107,8 @@ export default function ModeRoute({
     StudyInstanceUIDsAsArray.forEach(StudyInstanceUID => {
       dataSource.retrieveSeriesMetadata({ StudyInstanceUID });
     });
+
+    return unsubscribe;
   }, [
     mode,
     dataSourceName,