From 97dc2f90490aa8e2b95c2df5d48f4d77bb480d95 Mon Sep 17 00:00:00 2001
From: Gabriel Lebaudy <gabriel.lebaudy@gmail.com>
Date: Wed, 11 Jan 2023 20:42:16 +0100
Subject: [PATCH] fix: Require all instances to have an ImagePositionPatient
 attribute (#3107)

This fixes cases when some instances don't have this attribute, which would break on platform/core/src/utils/sortInstancesByPosition.ts:52
---
 platform/core/src/utils/isDisplaySetReconstructable.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/platform/core/src/utils/isDisplaySetReconstructable.js b/platform/core/src/utils/isDisplaySetReconstructable.js
index 55539d143..6ab7bf2fd 100644
--- a/platform/core/src/utils/isDisplaySetReconstructable.js
+++ b/platform/core/src/utils/isDisplaySetReconstructable.js
@@ -29,6 +29,11 @@ export default function isDisplaySetReconstructable(instances) {
     return { value: false };
   }
 
+  // Can't reconstruct if all instances don't have the ImagePositionPatient.
+  if (!instances.every(instance => !!instance.ImagePositionPatient)) {
+    return { value: false };
+  }
+
   const sortedInstances = sortInstancesByPosition(instances);
 
   if (isMultiframe) {