fix some oopsies

MeasureAuthoringTool · Sep 26, 2023 · 8cf7c38 · 8cf7c38
1 parent 2eb491d
commit 8cf7c38
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/main/java/cms/gov/madie/measure/services/TestCaseService.java b/src/main/java/cms/gov/madie/measure/services/TestCaseService.java
@@ -308,10 +308,10 @@ public TestCase updateTestCase(
           QiCoreJsonUtil.enforcePatientId(validatedTestCase, madieJsonResourcesBaseUri));
       validatedTestCase.setJson(
           QiCoreJsonUtil.updateResourceFullUrls(validatedTestCase, madieJsonResourcesBaseUri));
+      validatedTestCase.setJson(
+          QiCoreJsonUtil.replacePatientRefs(
+              validatedTestCase.getJson(), validatedTestCase.getPatientId().toString()));
     }
-    validatedTestCase.setJson(
-        QiCoreJsonUtil.replacePatientRefs(
-            validatedTestCase.getJson(), validatedTestCase.getPatientId().toString()));
     measure.getTestCases().add(validatedTestCase);
 
     measureRepository.save(measure);

diff --git a/src/main/java/cms/gov/madie/measure/utils/QiCoreJsonUtil.java b/src/main/java/cms/gov/madie/measure/utils/QiCoreJsonUtil.java
@@ -316,8 +316,6 @@ public static String updateResourceFullUrls(TestCase testCase, String madieJsonR
     return testCase.getJson();
   }
 
-  // update full urls for non-patient resources
-
   protected static String jsonNodeToString(ObjectMapper objectMapper, JsonNode rootNode) {
     ByteArrayOutputStream bout = new ByteArrayOutputStream();
     try {