refact:mapped all fields under the standalone context builder

Author: nandkishorr

care/emr/reports/context_builder/data_points/account.py

@@ -9,7 +9,7 @@
     AccountChargeItemContextBuilder,
 )
 from care.emr.reports.context_builder.data_points.encounter import (
-    MinimumEncounterReportContext,
+    EncounterReportContext,
 )
 from care.emr.reports.context_builder.data_points.facility import FacilityContextBuilder
 from care.emr.reports.context_builder.data_points.invoice import (
@@ -19,7 +19,7 @@
     MonetaryComponentContextBuilder,
 )
 from care.emr.reports.context_builder.data_points.patient import (
-    PatientMinimumContextBuilder,
+    PatientContextBuilder,
 )
 from care.emr.reports.context_builder.data_points.payment_reconciliation import (
     PaymentReconciliationContextBuilder,
@@ -42,7 +42,17 @@
 }
 
 
-class BaseAccountContextBuilder(SingleObjectContextBuilder):
+class AccountContextBuilder(SingleObjectContextBuilder):
+    standalone_context = True
+    __slug__ = "account_base"
+    __associating_model__ = Account
+    __display_name__ = "Account Report"
+    __description__ = "Report context for account-based reports"
+    context_key = "account"
+
+    def get_context(self):
+        return getattr(self.parent_context, self.parent_attribute)
+
     external_id = Field(
         display="Account External ID",
         preview_value="beff3ce1-e1be-41bc-8fb9-07ce2ebe42a6",
@@ -139,22 +149,13 @@ class BaseAccountContextBuilder(SingleObjectContextBuilder):
     primary_encounter = Field(
         display="Primary Encounter",
         preview_value="",
-        target_context=MinimumEncounterReportContext,
+        target_context=EncounterReportContext,
         description="Primary encounter associated with the account",
     )
 
-
-class AccountContextBuilder(BaseAccountContextBuilder):
-    standalone_context = True
-    __slug__ = "account_base"
-    __associating_model__ = Account
-    __display_name__ = "Account Report"
-    __description__ = "Report context for account-based reports"
-    context_key = "account"
-
     patient = Field(
         display="Patient Details",
-        target_context=PatientMinimumContextBuilder,
+        target_context=PatientContextBuilder,
         preview_value="",
         description="Details of the patient associated with the account",
     )

care/emr/reports/context_builder/data_points/encounter.py

@@ -25,7 +25,7 @@
 )
 from care.emr.reports.context_builder.data_points.patient import (
     IdentifiersContextBuilder,
-    PatientMinimumContextBuilder,
+    PatientContextBuilder,
     PatientTagContextBuilder,
     TagFilter,
 )
@@ -226,7 +226,24 @@ def get_context(self):
     )
 
 
-class BaseEncounterReportContext(SingleObjectContextBuilder):
+class PatientFacilityIdentifiersContextBuilder(IdentifiersContextBuilder):
+    def get_context(self):
+        return self.parent_context.patient.facility_identifiers.get(
+            str(self.parent_context.facility.id), []
+        )
+
+
+class EncounterReportContext(SingleObjectContextBuilder):
+    standalone_context = True
+    __slug__ = "encounter_base"
+    __associating_model__ = Encounter
+    __display_name__ = "Encounter Report"
+    __description__ = "Report context for encounter-based reports"
+    context_key = "encounter"
+
+    def get_context(self):
+        return getattr(self.parent_context, self.parent_attribute)
+
     status = Field(
         display="Encounter Status",
         mapping=lambda e: STATUS_DISPLAY.get(
@@ -284,12 +301,6 @@ class BaseEncounterReportContext(SingleObjectContextBuilder):
         preview_value="Patient is advised to follow up in 2 weeks.",
         description="Discharge summary advice for the encounter",
     )
-    patient_facility_tags = Field(
-        target_context=EncounterPatientFacilityTagContextBuilder,
-        display="Patient Facility Tags",
-        preview_value="",
-        description="Facility-specific tags associated with the patient for the encounter",
-    )
     encounter_tags = Field(
         target_context=EncounterTagContextBuilder,
         display="Encounter Tags",
@@ -327,30 +338,9 @@ class BaseEncounterReportContext(SingleObjectContextBuilder):
         description="Encounter extensions as JSON (e.g. encounter.extensions.encounter_attender)",
     )
 
-
-class MinimumEncounterReportContext(BaseEncounterReportContext):
-    def get_context(self):
-        return getattr(self.parent_context, self.parent_attribute)
-
-
-class PatientFacilityIdentifiersContextBuilder(IdentifiersContextBuilder):
-    def get_context(self):
-        return self.parent_context.patient.facility_identifiers.get(
-            str(self.parent_context.facility.id), []
-        )
-
-
-class EncounterReportContext(BaseEncounterReportContext):
-    standalone_context = True
-    __slug__ = "encounter_base"
-    __associating_model__ = Encounter
-    __display_name__ = "Encounter Report"
-    __description__ = "Report context for encounter-based reports"
-    context_key = "encounter"
-
     patient = Field(
         display="Patient Details",
-        target_context=PatientMinimumContextBuilder,
+        target_context=PatientContextBuilder,
         preview_value="",
         description="Details of the patient associated with the encounter",
     )
@@ -414,6 +404,12 @@ class EncounterReportContext(BaseEncounterReportContext):
         preview_value="",
         description="Facility identifiers associated with the encounter",
     )
+    patient_facility_tags = Field(
+        target_context=EncounterPatientFacilityTagContextBuilder,
+        display="Patient Facility Tags",
+        preview_value="",
+        description="Facility-specific tags associated with the patient for the encounter",
+    )
 
 
 DataPointRegistry.register(EncounterReportContext)

care/emr/reports/context_builder/data_points/patient.py

@@ -103,7 +103,17 @@ def get_context(self):
         return TagConfig.objects.filter(id__in=self.parent_context.instance_tags)
 
 
-class BasePatientContextBuilder(SingleObjectContextBuilder):
+class PatientContextBuilder(SingleObjectContextBuilder):
+    standalone_context = True
+    __slug__ = "patient_base"
+    __associating_model__ = Patient
+    __display_name__ = "Patient Report"
+    __description__ = "Report context for patient-based reports"
+    context_key = "patient"
+
+    def get_context(self):
+        return getattr(self.parent_context, self.parent_attribute)
+
     name = Field(
         display="Patient Name",
         preview_value="John Doe",
@@ -181,18 +191,4 @@ class BasePatientContextBuilder(SingleObjectContextBuilder):
     )
 
 
-class PatientMinimumContextBuilder(BasePatientContextBuilder):
-    def get_context(self):
-        return getattr(self.parent_context, self.parent_attribute)
-
-
-class PatientContextBuilderBase(BasePatientContextBuilder):
-    standalone_context = True
-    __slug__ = "patient_base"
-    __associating_model__ = Patient
-    __display_name__ = "Patient Report"
-    __description__ = "Report context for patient-based reports"
-    context_key = "patient"
-
-
-DataPointRegistry.register(PatientContextBuilderBase)
+DataPointRegistry.register(PatientContextBuilder)