[NRL-1606] Set patient_id in report to non-included if flag is not set

mattdean3-nhs · mattdean3-nhs · commit 63e5d4530665 · 2025-09-23T08:58:59.000+01:00
diff --git a/reports/get_pointer_masterids_for_custodian.py b/reports/get_pointer_masterids_for_custodian.py
@@ -56,14 +56,15 @@ def _get_masterids_for_custodians(table_name: str, custodians: str | tuple[str])
             pointer_type = item.get("type_id", {}).get("S", "no-type")
             master_id = item.get("master_identifier", {}).get("S", "no-master-id")
             custodian = item.get("custodian", {}).get("S", "no-custodian")
+            patient_id = item.get("nhs_number", {}).get("S", "no-patient-id")
 
             pointers_info.append(
                 {
                     "nrl-id": pointer_id,
                     "pointer-type": pointer_type,
                     "master_identifier": master_id,
                     "custodian": custodian,
-                    "patient_id": item.get("nhs_number", {}).get("S", "no-patient-id"),
+                    "patient_id": patient_id if INCLUDE_PATIENT_IDS else "not-included",
                 }
             )
 

Original file line number	Diff line number	Diff line change
`@@ -56,14 +56,15 @@ def _get_masterids_for_custodians(table_name: str, custodians: str \| tuple[str])`
`56`	`56`	`pointer_type = item.get("type_id", {}).get("S", "no-type")`
`57`	`57`	`master_id = item.get("master_identifier", {}).get("S", "no-master-id")`
`58`	`58`	`custodian = item.get("custodian", {}).get("S", "no-custodian")`
	`59`	`+ patient_id = item.get("nhs_number", {}).get("S", "no-patient-id")`
`59`	`60`
`60`	`61`	`pointers_info.append(`
`61`	`62`	`{`
`62`	`63`	`"nrl-id": pointer_id,`
`63`	`64`	`"pointer-type": pointer_type,`
`64`	`65`	`"master_identifier": master_id,`
`65`	`66`	`"custodian": custodian,`
`66`		`- "patient_id": item.get("nhs_number", {}).get("S", "no-patient-id"),`
	`67`	`+ "patient_id": patient_id if INCLUDE_PATIENT_IDS else "not-included",`
`67`	`68`	`}`
`68`	`69`	`)`
`69`	`70`