Skip to content

Commit 994f9d9

Browse files
Merge pull request #6614 from NHSDigital/alistair/imms-api-suppress-bundle-link-mismatch
Suppress warning for `immunization.target` mismatch
2 parents 4204198 + 54660d3 commit 994f9d9

3 files changed

Lines changed: 7 additions & 2 deletions

File tree

app/lib/nhs/immunisations_api.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -412,6 +412,11 @@ def check_bundle_link_params(bundle, request_params)
412412
uri = URI(link)
413413
bundle_params = URI.decode_www_form(uri.query).to_h
414414

415+
# The Imms API used to have a bug where they referred to `immunization.target` instead of `-immunization.target`.
416+
# They have since fixed this, but they now include both versions in the `bundle.link` field. We must exclude the
417+
# deprecated version from the comparison.
418+
bundle_params.delete("immunization.target")
419+
415420
# We don't care about the order of the target values
416421
bundle_params["-immunization.target"] = bundle_params[
417422
"-immunization.target"

spec/fixtures/files/fhir/search_responses/bad_immunization_target.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"link": [
55
{
66
"relation": "self",
7-
"url": "https://int.api.service.nhs.uk/immunisation-fhir-api/FHIR/R4/Immunization?immunization+target=3IN1,MMR,MMRV,FLU,HPV,MENACWY&_include=Immunization:patient&patient.identifier=https://fhir.nhs.uk/Id/nhs-number|9793826983"
7+
"url": "https://int.api.service.nhs.uk/immunisation-fhir-api/FHIR/R4/Immunization?immunization+target=3IN1,MMR,MMRV,FLU,HPV,MENACWY&patient.identifier=https://fhir.nhs.uk/Id/nhs-number|9793826983"
88
}
99
],
1010
"entry": [

spec/fixtures/files/fhir/search_responses/immunization_target_both.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"link": [
55
{
66
"relation": "self",
7-
"url": "https://int.api.service.nhs.uk/immunisation-fhir-api/Immunization?-date.from=2025-08-01\u0026-date.to=2025-10-01\u0026-immunization.target=FLU,HPV,MENACWY,3IN1,MMR,MMRV\u0026patient.identifier=https%3A%2F%2Ffhir.nhs.uk%2FId%2Fnhs-number%7C9449308357"
7+
"url": "https://int.api.service.nhs.uk/immunisation-fhir-api/Immunization?-date.from=2025-08-01\u0026-date.to=2025-10-01\u0026-immunization.target=FLU,HPV,MENACWY,3IN1,MMR,MMRV\u0026immunization.target=FLU,HPV,MENACWY,3IN1,MMR,MMRV\u0026patient.identifier=https%3A%2F%2Ffhir.nhs.uk%2FId%2Fnhs-number%7C9449308357"
88
}
99
],
1010
"entry": [

0 commit comments

Comments
 (0)