Skip to content

Commit 69d473c

Browse files
committed
use constants as well in processes
1 parent a4028d1 commit 69d473c

File tree

5 files changed

+23
-11
lines changed

5 files changed

+23
-11
lines changed

dsf-bpe-process-feasibility/src/main/java/org/highmed/dsf/bpe/service/CheckFeasibilityResources.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package org.highmed.dsf.bpe.service;
22

3+
import static org.highmed.dsf.bpe.ConstantsBase.ORGANIZATION_IDENTIFIER_SYSTEM;
4+
35
import java.util.List;
46

57
import org.camunda.bpm.engine.delegate.DelegateExecution;
@@ -38,7 +40,7 @@ private void checkNumberOfParticipatingMedics(ResearchStudy researchStudy)
3840
long medics = researchStudy.getExtensionsByUrl(ConstantsFeasibility.EXTENSION_PARTICIPATING_MEDIC_URI).stream()
3941
.filter(e -> e.getValue() instanceof Reference).map(e -> (Reference) e.getValue())
4042
.map(r -> r.getIdentifier())
41-
.filter(i -> "http://highmed.org/fhir/NamingSystem/organization-identifier".equals(i.getSystem()))
43+
.filter(i -> ORGANIZATION_IDENTIFIER_SYSTEM.equals(i.getSystem()))
4244
.map(i -> i.getValue()).distinct().count();
4345

4446
if (medics < ConstantsFeasibility.MIN_PARTICIPATING_MEDICS)

dsf-bpe-process-feasibility/src/main/java/org/highmed/dsf/bpe/service/DownloadFeasibilityResources.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package org.highmed.dsf.bpe.service;
22

3+
import static org.highmed.dsf.bpe.variables.ConstantsFeasibility.EXTENSION_PARTICIPATING_TTP_URI;
4+
35
import java.util.Collections;
46
import java.util.List;
57
import java.util.Map;
@@ -150,8 +152,7 @@ private List<Group> getCohortDefinitions(Bundle bundle, String baseUrl)
150152

151153
private String getTtpIdentifier(ResearchStudy researchStudy, FhirWebserviceClient client)
152154
{
153-
Extension ext = researchStudy
154-
.getExtensionByUrl("http://highmed.org/fhir/StructureDefinition/participating-ttp");
155+
Extension ext = researchStudy.getExtensionByUrl(EXTENSION_PARTICIPATING_TTP_URI);
155156
Reference ref = (Reference) ext.getValue();
156157
return ref.getIdentifier().getValue();
157158
}

dsf-bpe-process-feasibility/src/main/java/org/highmed/dsf/bpe/service/DownloadResearchStudyResource.java

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
package org.highmed.dsf.bpe.service;
22

3+
import static org.highmed.dsf.bpe.ConstantsBase.ORGANIZATION_IDENTIFIER_SYSTEM;
4+
import static org.highmed.dsf.bpe.ConstantsBase.ORGANIZATION_TYPE_MEDIC;
5+
36
import java.util.List;
47
import java.util.Objects;
58
import java.util.stream.Collectors;
@@ -15,6 +18,7 @@
1518
import org.hl7.fhir.r4.model.Identifier;
1619
import org.hl7.fhir.r4.model.Reference;
1720
import org.hl7.fhir.r4.model.ResearchStudy;
21+
import org.hl7.fhir.r4.model.ResourceType;
1822
import org.hl7.fhir.r4.model.Task;
1923
import org.slf4j.Logger;
2024
import org.slf4j.LoggerFactory;
@@ -88,9 +92,9 @@ private ResearchStudy getResearchStudy(IdType researchStudyid, FhirWebserviceCli
8892

8993
private ResearchStudy addMissingOrganizations(ResearchStudy researchStudy, FhirWebserviceClient client)
9094
{
91-
List<String> identifiers = organizationProvider.getOrganizationsByType("MeDIC")
95+
List<String> identifiers = organizationProvider.getOrganizationsByType(ORGANIZATION_TYPE_MEDIC)
9296
.flatMap(o -> o.getIdentifier().stream())
93-
.filter(i -> "http://highmed.org/fhir/NamingSystem/organization-identifier".equals(i.getSystem()))
97+
.filter(i -> ORGANIZATION_IDENTIFIER_SYSTEM.equals(i.getSystem()))
9498
.map(i -> i.getValue()).collect(Collectors.toList());
9599

96100
List<String> existingIdentifiers = researchStudy
@@ -108,8 +112,8 @@ private ResearchStudy addMissingOrganizations(ResearchStudy researchStudy, FhirW
108112
identifier, researchStudy.getId());
109113

110114
researchStudy.addExtension().setUrl(ConstantsFeasibility.EXTENSION_PARTICIPATING_MEDIC_URI).setValue(
111-
new Reference().setType("Organization").setIdentifier(new Identifier()
112-
.setSystem("http://highmed.org/fhir/NamingSystem/organization-identifier")
115+
new Reference().setType(ResourceType.Organization.name()).setIdentifier(new Identifier()
116+
.setSystem(ORGANIZATION_IDENTIFIER_SYSTEM)
113117
.setValue(identifier)));
114118

115119
});

dsf-bpe-process-feasibility/src/main/java/org/highmed/dsf/bpe/service/GenerateBloomFilters.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package org.highmed.dsf.bpe.service;
22

3+
import static org.highmed.dsf.bpe.ConstantsBase.ORGANIZATION_IDENTIFIER_SYSTEM;
4+
35
import java.security.Key;
46
import java.util.List;
57
import java.util.Objects;
@@ -28,6 +30,7 @@
2830
import org.hl7.fhir.r4.model.Binary;
2931
import org.hl7.fhir.r4.model.IdType;
3032
import org.hl7.fhir.r4.model.Reference;
33+
import org.hl7.fhir.r4.model.ResourceType;
3134
import org.hl7.fhir.r4.model.Task;
3235
import org.slf4j.Logger;
3336
import org.slf4j.LoggerFactory;
@@ -147,13 +150,13 @@ protected String saveResultSetAsBinaryForTtp(ResultSet resultSet, String securit
147150
{
148151
byte[] content = serializeResultSet(resultSet);
149152
Reference securityContext = new Reference();
150-
securityContext.setType("Organization").getIdentifier()
151-
.setSystem("http://highmed.org/fhir/NamingSystem/organization-identifier").setValue(securityIdentifier);
153+
securityContext.setType(ResourceType.Organization.name()).getIdentifier()
154+
.setSystem(ORGANIZATION_IDENTIFIER_SYSTEM).setValue(securityIdentifier);
152155
Binary binary = new Binary().setContentType(ConstantsBase.OPENEHR_MIMETYPE_JSON)
153156
.setSecurityContext(securityContext).setData(content);
154157

155158
IdType created = createBinaryResource(binary);
156-
return new IdType(getFhirWebserviceClientProvider().getLocalBaseUrl(), "Binary", created.getIdPart(),
159+
return new IdType(getFhirWebserviceClientProvider().getLocalBaseUrl(), ResourceType.Binary.name(), created.getIdPart(),
157160
created.getVersionIdPart()).getValue();
158161
}
159162

dsf-bpe-process-local-services/src/main/java/org/highmed/dsf/bpe/service/ExtractInputValues.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package org.highmed.dsf.bpe.service;
22

3+
import static org.highmed.dsf.bpe.ConstantsBase.EXTENSION_QUERY_URI;
4+
35
import java.util.List;
46
import java.util.UUID;
57
import java.util.stream.Collectors;
@@ -61,7 +63,7 @@ private List<Group> getCohortDefinitions(Stream<String> queries)
6163
return queries.map(q -> {
6264
Group group = new Group();
6365
group.setIdElement(new IdType(UUID.randomUUID().toString()));
64-
group.addExtension().setUrl("http://highmed.org/fhir/StructureDefinition/query").setValue(
66+
group.addExtension().setUrl(EXTENSION_QUERY_URI).setValue(
6567
new Expression().setLanguageElement(ConstantsBase.AQL_QUERY_TYPE).setExpression(q));
6668
return group;
6769
}).collect(Collectors.toList());

0 commit comments

Comments
 (0)