Skip to content

Commit

Permalink
#413: Properly populate PurposeOfUse attribute in audit records of so…
Browse files Browse the repository at this point in the history
…me HL7v3-based transactions
  • Loading branch information
unixoid authored and Stanojevic Boris committed May 9, 2023
1 parent b639ff5 commit 27b98c6
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -157,10 +157,10 @@ public List<TypeValuePairType> dicomDetails(String repositoryId,
return details(DOCUMENT_UNIQUE_ID, documentInstanceId, seriesInstanceId, repositoryId, homeCommunityId, isXcaHomeCommunityId);
}

private List<TypeValuePairType> details(String insranceKey, String instanceId, String seriesInstanceId, String repositoryId, String homeCommunityId, boolean xcaHomeCommunityId) {
private List<TypeValuePairType> details(String instanceKey, String instanceId, String seriesInstanceId, String repositoryId, String homeCommunityId, boolean xcaHomeCommunityId) {
var tvp = new ArrayList<TypeValuePairType>(0);
if (instanceId != null) {
tvp.add(getTypeValuePair(insranceKey, instanceId));
tvp.add(getTypeValuePair(instanceKey, instanceId));
}
if (seriesInstanceId != null) {
tvp.add(getTypeValuePair(SERIES_INSTANCE_UNIQUE_ID, seriesInstanceId));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ class Iti55AuditStrategy extends Iti47AuditStrategy {

@Override
AuditMessage[] makeAuditMessage(AuditContext auditContext, Hl7v3AuditDataset auditDataset) {
QueryInformationBuilder builder = new QueryInformationBuilder<>(auditContext, auditDataset, Hl7v3EventTypeCode.CrossGatewayPatientDiscovery)
QueryInformationBuilder builder = new QueryInformationBuilder<>(auditContext, auditDataset, Hl7v3EventTypeCode.CrossGatewayPatientDiscovery, auditDataset.getPurposesOfUse())
// No patient identifiers are included for the Initiating Gateway
if (isServerSide()) {
builder.addPatients(auditDataset.patientIds)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ class Iti56AuditStrategy extends Hl7v3AuditStrategy {

@Override
AuditMessage[] makeAuditMessage(AuditContext auditContext, Hl7v3AuditDataset auditDataset) {
new QueryInformationBuilder<>(auditContext, auditDataset, Hl7v3EventTypeCode.PatientLocationQuery)
new QueryInformationBuilder<>(auditContext, auditDataset, Hl7v3EventTypeCode.PatientLocationQuery, auditDataset.getPurposesOfUse())
.addPatients(auditDataset.patientIds)
.setQueryParameters("PatientLocationQueryRequest", PatientLocationQuery, auditDataset.requestPayload)
.getMessages()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ class Pcc1AuditStrategy extends Hl7v3AuditStrategy {

@Override
AuditMessage[] makeAuditMessage(AuditContext auditContext, Hl7v3AuditDataset auditDataset) {
new QueryInformationBuilder<>(auditContext, auditDataset, Hl7v3EventTypeCode.QueryExistingData)
new QueryInformationBuilder<>(auditContext, auditDataset, Hl7v3EventTypeCode.QueryExistingData, auditDataset.getPurposesOfUse())
.setQueryParameters(auditDataset.messageId, QueryExistingData, auditDataset.requestPayload)
.addPatients(auditDataset.patientIds)
.getMessages()
Expand Down
5 changes: 5 additions & 0 deletions src/site/changes.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,11 @@
<author email="christian.ohr@gmail.com">Christian Ohr</author>
</properties>
<body>
<release version="tbd" description="IPF tbd" date="tbd">
<action issue="413" dev="zortx233" type="fix">
Properly populate PurposeOfUse attribute in audit records of some HL7v3-based transactions
</action>
</release>
<release version="4.6.0" description="IPF 4.6.0" date="21 Apr 2023">
<action issue="412" dev="ohr" type="fix">
Don't log in HL7v3 continuation webservice when returning zero results
Expand Down

0 comments on commit 27b98c6

Please sign in to comment.