*based on PR #510 'Finished KPI Tests'* Finishes first round of testing

epilepsy12/admin.py

@@ -18,7 +18,6 @@ class Epilepsy12UserAdmin(UserAdmin, SimpleHistoryAdmin):
         "email",
         "surname",
         "role",
-        "organisation_employer",
         "is_active",
     )
     list_display = (
@@ -28,7 +27,6 @@ class Epilepsy12UserAdmin(UserAdmin, SimpleHistoryAdmin):
         "first_name",
         "surname",
         "is_active",
-        "twitter_handle",
         "role",
         "organisation_employer",
         "is_superuser",
@@ -52,20 +50,30 @@ class Epilepsy12UserAdmin(UserAdmin, SimpleHistoryAdmin):
             },
         ),
         ("Epilepsy12 Centre", {"fields": ("organisation_employer", "role")}),
-        ("Contacts", {"fields": ("email", "twitter_handle")}),
+        ("Contacts", {"fields": ("email",)}),
         (
             "Permissions",
             {
                 "fields": (
                     "is_active",
                     "is_staff",
+                    "is_rcpch_staff",
                     "is_rcpch_audit_team_member",
                     "is_superuser",
                     "email_confirmed",
                     "view_preference",
                 )
             },
         ),
+        (
+            "Access",
+            {
+                "fields": (
+                    "last_login",
+                    "date_joined",
+                )
+            },
+        ),
         (
             "Group Permissions",
             {
@@ -76,7 +84,6 @@ class Epilepsy12UserAdmin(UserAdmin, SimpleHistoryAdmin):
                 ),
             },
         ),
-        ("Personal", {"fields": ("bio",)}),
     )
     add_fieldsets = (
         (
@@ -89,6 +96,7 @@ class Epilepsy12UserAdmin(UserAdmin, SimpleHistoryAdmin):
                     "first_name",
                     "surname",
                     "is_staff",
+                    "is_rcpch_staff",
                     "is_active",
                     "is_rcpch_audit_team_member",
                     "role",
@@ -114,6 +122,8 @@ def get_form(self, request, obj=None, **kwargs):
             form.base_fields["title"].disabled = True
             form.base_fields["email"].disabled = True
             form.base_fields["is_staff"].disabled = True
+            form.base_fields["is_rcpch_staff"].disabled = True
+            form.base_fields["is_rcpch_audit_team_member"].disabled = True
         return form
 
 

epilepsy12/common_view_functions/aggregate_by.py

@@ -1,6 +1,7 @@
 from typing import Literal
 
 # Django imports
+from django.apps import apps
 from django.contrib.gis.db.models import (
     Q,
     F,
@@ -16,7 +17,8 @@
 
 # E12 imports
 from epilepsy12.constants import ETHNICITIES, SEX_TYPE
-from ..models import Case
+
+# from ..models import Case
 from .report_queries import (
     get_all_organisations,
     get_all_trusts,
@@ -34,12 +36,12 @@
 def cases_aggregated_by_sex(selected_organisation):
     # aggregate queries on trust level cases
 
+    Case = apps.get_model("epilepsy12", "Case")
+
     sex_long_list = [When(sex=k, then=Value(v)) for k, v in SEX_TYPE]
 
     cases_aggregated_by_sex = (
-        Case.objects.filter(
-            organisations__OrganisationName__contains=selected_organisation
-        )
+        Case.objects.filter(organisations=selected_organisation)
         .values("sex")
         .annotate(sex_display=DJANGO_CASE(*sex_long_list, output_field=CharField()))
         .values("sex_display")
@@ -52,34 +54,37 @@ def cases_aggregated_by_sex(selected_organisation):
 
 def cases_aggregated_by_deprivation_score(selected_organisation):
     # aggregate queries on trust level cases
+    Case = apps.get_model("epilepsy12", "Case")
 
-    deprivation_quintiles = ((1, 1), (2, 2), (3, 3), (4, 4), (5, 5), (None, 6))
-
-    imd_long_list = [
-        When(index_of_multiple_deprivation_quintile=k, then=Value(v))
-        for k, v in deprivation_quintiles
-    ]
+    cases_in_selected_organisation = Case.objects.filter(
+        organisations__OrganisationName__contains=selected_organisation
+    )
 
     cases_aggregated_by_deprivation = (
-        Case.objects.filter(
-            organisations__OrganisationName__contains=selected_organisation
-        )
+        # Filter just Cases in selected org
+        cases_in_selected_organisation
+        # Get list of IMD quintiles
         .values("index_of_multiple_deprivation_quintile")
+        # Converting 'None' to 6 in a new index_of_multiple_deprivation_quintile_display "column"
         .annotate(
             index_of_multiple_deprivation_quintile_display=DJANGO_CASE(
-                *imd_long_list, output_field=PositiveSmallIntegerField()
+                When(index_of_multiple_deprivation_quintile=None, then=Value(6)),
+                default="index_of_multiple_deprivation_quintile",
+                output_field=PositiveSmallIntegerField(),
             )
         )
+        # Keeps only the new column
         .values("index_of_multiple_deprivation_quintile_display")
+        # Value count the new column
         .annotate(
             cases_aggregated_by_deprivation=Count(
-                "index_of_multiple_deprivation_quintile"
-            )
+                "index_of_multiple_deprivation_quintile_display"
+            ),
         )
-        .order_by("index_of_multiple_deprivation_quintile")
+        .order_by('index_of_multiple_deprivation_quintile_display')
+
     )
-
-    # map quintile num to string repr
+
     deprivation_quintile_str_map = {
         1: "1st quintile",
         2: "2nd quintile",
@@ -89,19 +94,23 @@ def cases_aggregated_by_deprivation_score(selected_organisation):
         6: "Not known",
     }
 
-    for index, q in enumerate(cases_aggregated_by_deprivation):
-        q[
-            "index_of_multiple_deprivation_quintile_display"
-        ] = deprivation_quintile_str_map.get(
-            q.get("index_of_multiple_deprivation_quintile_display")
-        )
+    for aggregate in cases_aggregated_by_deprivation:
+        quintile = aggregate["index_of_multiple_deprivation_quintile_display"]
+
+        str_map = deprivation_quintile_str_map.get(quintile)
 
+        aggregate.update(
+            {"index_of_multiple_deprivation_quintile_display_str": str_map}
+        )
+
     return cases_aggregated_by_deprivation
 
 
 def cases_aggregated_by_ethnicity(selected_organisation):
     # aggregate queries on trust level cases
 
+    Case = apps.get_model("epilepsy12", "Case")
+
     ethnicity_long_list = [When(ethnicity=k, then=Value(v)) for k, v in ETHNICITIES]
 
     cases_aggregated_by_ethnicity = (
@@ -219,6 +228,7 @@ def return_all_aggregated_kpis_for_cohort_and_abstraction_level_annotated_by_sub
     """
     Returns aggregated KPIS for given cohort annotated by sublevel of abstraction (eg kpis in each NHS England region, labelled by region)
     """
+    Case = apps.get_model("epilepsy12", "Case")
 
     if abstraction_level == "organisation":
         abstraction_sublevels = get_all_organisations()

epilepsy12/common_view_functions/calculate_kpi_functions/score_kpi_1.py

@@ -49,25 +49,19 @@ def score_kpi_1(registration_instance) -> int:
     # score KPI
     if all_consultant_paediatrician_fields_complete:
         passed_metric = (
-            relativedelta(
-                assessment.consultant_paediatrician_input_date,
-                assessment.consultant_paediatrician_referral_date,
-            ).days
-            <= 14
-        )
+            assessment.consultant_paediatrician_input_date
+            - assessment.consultant_paediatrician_referral_date
+        ).days <= 14
         if passed_metric:
             return KPI_SCORE["PASS"]
         else:
             return KPI_SCORE["FAIL"]
 
     elif all_paediatric_neurologist_fields_complete:
         passed_metric = (
-            relativedelta(
-                assessment.paediatric_neurologist_input_date,
-                assessment.paediatric_neurologist_referral_date,
-            ).days
-            <= 14
-        )
+            assessment.paediatric_neurologist_input_date
+            - assessment.paediatric_neurologist_referral_date
+        ).days <= 14
         if passed_metric:
             return KPI_SCORE["PASS"]
         else:

epilepsy12/common_view_functions/calculate_kpi_functions/score_kpi_3.py

@@ -2,10 +2,11 @@
 
 # django imports
 from django.contrib.gis.db.models import Q
+from django.apps import apps
 
 # E12 imports
 from epilepsy12.constants import KPI_SCORE
-from epilepsy12.models import AntiEpilepsyMedicine, Episode
+
 
 def score_kpi_3(registration_instance, age_at_first_paediatric_assessment) -> int:
     """3. tertiary_input
@@ -20,6 +21,9 @@ def score_kpi_3(registration_instance, age_at_first_paediatric_assessment) -> in
 
     assessment = registration_instance.assessment
 
+    AntiEpilepsyMedicine = apps.get_model("epilepsy12", "AntiEpilepsyMedicine")
+    Episode = apps.get_model("epilepsy12", "Episode")
+
     # EVALUATE ELIGIBILITY CRITERIA
 
     # first gather relevant data
@@ -69,6 +73,7 @@ def score_kpi_3(registration_instance, age_at_first_paediatric_assessment) -> in
     else:
         return KPI_SCORE["FAIL"]
 
+
 def score_kpi_3b(registration_instance) -> int:
     """3b. epilepsy_surgery_referral
 

epilepsy12/common_view_functions/calculate_kpi_functions/score_kpi_4.py

@@ -19,13 +19,15 @@ def score_kpi_4(registration_instance) -> int:
     epilepsy_context = registration_instance.epilepsycontext
     investigations = registration_instance.investigations
 
+    # ineligible
+    if epilepsy_context.were_any_of_the_epileptic_seizures_convulsive is False:
+        return KPI_SCORE["INELIGIBLE"]
+
     # not scored / ineligible guard clauses
     if (epilepsy_context.were_any_of_the_epileptic_seizures_convulsive is None) or (
         investigations.twelve_lead_ecg_status is None
     ):
         return KPI_SCORE["NOT_SCORED"]
-    if epilepsy_context.were_any_of_the_epileptic_seizures_convulsive is False:
-        return KPI_SCORE["INELIGIBLE"]
 
     # Convulsive seizure - score ECG status
 

epilepsy12/common_view_functions/calculate_kpi_functions/score_kpi_5.py

@@ -2,11 +2,13 @@
 
 # django imports
 from django.contrib.gis.db.models import Q
+from django.apps import apps
 
 # E12 imports
-from epilepsy12.models import Syndrome
+# from epilepsy12.models import Syndrome
 from epilepsy12.constants import KPI_SCORE
 
+
 def score_kpi_5(registration_instance, age_at_first_paediatric_assessment) -> int:
     """5. MRI
 
@@ -19,13 +21,15 @@ def score_kpi_5(registration_instance, age_at_first_paediatric_assessment) -> in
     multiaxial_diagnosis = registration_instance.multiaxialdiagnosis
     investigations = registration_instance.investigations
 
+    Syndrome = apps.get_model("epilepsy12", "Syndrome")
+
     # not scored
-    if multiaxial_diagnosis.syndrome_present is None:
+    if (age_at_first_paediatric_assessment >= 2) and (
+        multiaxial_diagnosis.syndrome_present is None
+    ):
         return KPI_SCORE["NOT_SCORED"]
 
-    # ineligible
-    if age_at_first_paediatric_assessment >= 2:
-        return KPI_SCORE["INELIGIBLE"]
+    # define eligibility criteria 1
     ineligible_syndrome_present = Syndrome.objects.filter(
         Q(multiaxial_diagnosis=multiaxial_diagnosis)
         &
@@ -39,29 +43,35 @@ def score_kpi_5(registration_instance, age_at_first_paediatric_assessment) -> in
             ]
         )
     ).exists()
-    if ineligible_syndrome_present:
-        return KPI_SCORE["INELIGIBLE"]
 
-    # not scored
-    mri_dates_are_none = [
-        (investigations.mri_brain_requested_date is None),
-        (investigations.mri_brain_reported_date is None),
-    ]
-    if any(mri_dates_are_none):
-        return KPI_SCORE["NOT_SCORED"]
+    # check eligibility criteria 1 & 2
+    # 1 = none of the specified syndromes present
+    # 2 = age in years < 2
+    if (not ineligible_syndrome_present) or (age_at_first_paediatric_assessment < 2):
+        # not scored
+        mri_dates_are_none = [
+            (investigations.mri_brain_requested_date is None),
+            (investigations.mri_brain_reported_date is None),
+        ]
+        if any(mri_dates_are_none):
+            return KPI_SCORE["NOT_SCORED"]
 
-    # eligible for this measure - score kpi
-    passing_criteria_met = (
-        abs(
-            (
-                investigations.mri_brain_requested_date
-                - investigations.mri_brain_reported_date
-            ).days
+        # eligible for this measure - score kpi
+        passing_criteria_met = (
+            abs(
+                (
+                    investigations.mri_brain_requested_date
+                    - investigations.mri_brain_reported_date
+                ).days
+            )
+            <= 42
         )
-        <= 42
-    )
 
-    if passing_criteria_met:
-        return KPI_SCORE["PASS"]
+        if passing_criteria_met:
+            return KPI_SCORE["PASS"]
+        else:
+            return KPI_SCORE["FAIL"]
+
+    # ineligible
     else:
-        return KPI_SCORE["FAIL"]
+        return KPI_SCORE["INELIGIBLE"]

epilepsy12/common_view_functions/calculate_kpi_functions/score_kpi_6.py

@@ -16,13 +16,14 @@ def score_kpi_6(registration_instance, age_at_first_paediatric_assessment) -> in
 
     multiaxial_diagnosis = registration_instance.multiaxialdiagnosis
 
+    # ineligible
+    if age_at_first_paediatric_assessment < 5:
+        return KPI_SCORE["INELIGIBLE"]
+
     # not scored
     if multiaxial_diagnosis.mental_health_screen is None:
         return KPI_SCORE["NOT_SCORED"]
 
-    # ineligible
-    if age_at_first_paediatric_assessment < 5:
-        return KPI_SCORE["INELIGIBLE"]
 
     # score kpi
     if multiaxial_diagnosis.mental_health_screen: