Remove manual query for ModalitiesInStudy for now

openradx · Aug 14, 2023 · 67a84e9 · 67a84e9
1 parent 396f19f
commit 67a84e9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/TODO.md b/TODO.md
@@ -17,6 +17,8 @@
 - Split messages into toasts (client only) and messages from server (using Bootstrap alerts)
 - Rewrite Celery unit tests using the official test helpers
 - Check why this dicomweb test takes so long
+- Use pynetdicom SCPs instead of Orthancs in integrations tests
+  -- Find those tests by looking for "setup_orthancs" fixture
 
 ## High Priority
 

diff --git a/adit/core/utils/dicom_operator.py b/adit/core/utils/dicom_operator.py
@@ -195,10 +195,10 @@ def _handle_found_studies(
             # TODO: I guess this won't work as we are in the middle of a C-FIND request (we
             # are using generators now) and we can't do another C-FIND request here.
             # But we could use a new Connector instance for this. Fix this later.
-            if "ModalitiesInStudy" not in result:
-                result.ModalitiesInStudy = self._fetch_study_modalities(
-                    result.PatientID, result.StudyInstanceUID
-                )
+            # if "ModalitiesInStudy" not in result:
+            #     result.ModalitiesInStudy = self._fetch_study_modalities(
+            #         result.PatientID, result.StudyInstanceUID
+            #     )
 
             modality_query = query.ModalitiesInStudy
             if modality_query: