fix(dicom): Don't try to get information on non-existent PixelData

This commit makes it so that we only look for information about the
format of a DICOM's PixelData if it actually has such data, whereas
before we looked for it for any DICOM that wasn't a contour series. That
test was insufficient and broke information-only DICOM modalities like PR.

Fixes Trac#9437

Author: zjp

src/bundles/dicom/src/dicom_hierarchy.py

@@ -520,7 +520,7 @@ def __init__(
             rsi = int(rsi)
         self.rescale_intercept = rsi
         self.rescale_slope = int(self.dicom_series.rescale_slope)
-        if not self.contour_series:
+        if self.image_series and not self.contour_series:
             bits = self.sample_file.get("BitsAllocated")
             rep = self.sample_file.get("PixelRepresentation")
             self.value_type = self.numpy_value_type(
@@ -663,6 +663,18 @@ def to_models(self, open_models, derived, sgrids):
                 "be available." % (self.number, self.patient_id)
             )
 
+    @property
+    def number(self):
+        if self.sample_file.get("SeriesNumber", None) is None:
+            self.session.logger.warning("SeriesNumber not specified; setting to 0")
+            return 0
+        else:
+            return int(self.sample_file.get("SeriesNumber", 0))
+
+    @property
+    def patient_id(self):
+        return self.sample_file.get("PatientID", "")
+
     @property
     def columns(self):
         return self.sample_file.get("Columns")