Skip to content

Commit f7e9408

Browse files
committed
Fix Bundle PUT URL Not Set To Relative URL
1 parent 060b29d commit f7e9408

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

src/main/java/de/medizininformatikinitiative/torch/model/management/ResourceBundle.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -257,9 +257,7 @@ public Bundle toFhirBundle() {
257257
bundle.setId(UUID.randomUUID().toString());
258258

259259
cache.values().forEach(resource -> {
260-
if (resource.isPresent()) {
261-
bundle.addEntry(createBundleEntry(resource.get()));
262-
}
260+
resource.ifPresent(value -> bundle.addEntry(createBundleEntry(value)));
263261
}
264262
);
265263
return bundle;
@@ -269,7 +267,7 @@ private Bundle.BundleEntryComponent createBundleEntry(Resource resource) {
269267
Bundle.BundleEntryComponent entryComponent = new Bundle.BundleEntryComponent();
270268
entryComponent.setResource(resource);
271269
Bundle.BundleEntryRequestComponent request = new Bundle.BundleEntryRequestComponent();
272-
request.setUrl(resource.getResourceType() + "/" + resource.getId());
270+
request.setUrl(ResourceUtils.getRelativeURL(resource));
273271
request.setMethod(PUT);
274272
entryComponent.setRequest(request);
275273
return entryComponent;

src/test/java/de/medizininformatikinitiative/torch/management/ResourceBundleTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ class ResourceBundleTest {
3434

3535
@BeforeEach
3636
void setUp() {
37-
patient1.setId("patient1");
37+
patient1.setId("http://blaze.com/fhir/Patient/patient1");
3838
patient2.setId("patient2");
3939
patient3.setId("patient3");
4040
Set<String> attributeGroups1 = Set.of("group1", "group2");
@@ -52,7 +52,7 @@ void toFhirBundleTest() {
5252
cache.put(wrapper1);
5353

5454
var fhirBundle = cache.toFhirBundle();
55-
55+
assertThat(fhirBundle.getEntry().getFirst().getRequest().getUrl()).isEqualTo("Patient/patient1");
5656
assertThat(fhirBundle)
5757
.containsNEntries(1)
5858
.extractResources()

0 commit comments

Comments
 (0)