TrizTech/ohif-viewer
15
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

missing useEffect unsubscribe

Browse Source
This commit is contained in:
dannyrb 2020-05-27 22:21:35 -04:00
parent 069fc3fa74
commit 1c45d643ac
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
platform/viewer/src/routes/Mode/Mode.jsx
View File

@ -92,7 +92,7 @@ export default function ModeRoute({
// TODO: This should be baked into core, not manuel? // TODO: This should be baked into core, not manuel?
// DisplaySetService would wire this up? // DisplaySetService would wire this up?
DicomMetadataStore.subscribe( const { unsubscribe } = DicomMetadataStore.subscribe(
DicomMetadataStore.EVENTS.INSTANCES_ADDED, DicomMetadataStore.EVENTS.INSTANCES_ADDED,
({ StudyInstanceUID, SeriesInstanceUID }) => { ({ StudyInstanceUID, SeriesInstanceUID }) => {
const seriesMetadata = DicomMetadataStore.getSeries( const seriesMetadata = DicomMetadataStore.getSeries(
@ -107,6 +107,8 @@ export default function ModeRoute({
StudyInstanceUIDsAsArray.forEach(StudyInstanceUID => { StudyInstanceUIDsAsArray.forEach(StudyInstanceUID => {
dataSource.retrieveSeriesMetadata({ StudyInstanceUID }); dataSource.retrieveSeriesMetadata({ StudyInstanceUID });
}); });
return unsubscribe;
}, [ }, [
mode, mode,
dataSourceName, dataSourceName,