Merge pull request #799 from NHSDigital/DTOSS-11782-other-procedures

Add and update other procedure form

Author: swebberuk

manage_breast_screening/mammograms/forms/__init__.py

@@ -4,6 +4,7 @@
 from .cyst_history_form import CystHistoryForm
 from .implanted_medical_device_history_form import ImplantedMedicalDeviceHistoryForm
 from .mastectomy_or_lumpectomy_history_form import MastectomyOrLumpectomyHistoryForm
+from .other_procedure_history_form import OtherProcedureHistoryForm
 from .record_medical_information_form import RecordMedicalInformationForm
 from .screening_appointment_form import ScreeningAppointmentForm
 from .special_appointment_forms import (
@@ -22,4 +23,5 @@
     "MarkReasonsTemporaryForm",
     "MastectomyOrLumpectomyHistoryForm",
     "ImplantedMedicalDeviceHistoryForm",
+    "OtherProcedureHistoryForm",
 ]

manage_breast_screening/mammograms/forms/other_procedure_history_form.py

@@ -0,0 +1,112 @@
+from django.forms.widgets import Textarea
+
+from manage_breast_screening.core.services.auditor import Auditor
+from manage_breast_screening.nhsuk_forms.fields import CharField, ChoiceField
+from manage_breast_screening.nhsuk_forms.fields.integer_field import YearField
+from manage_breast_screening.nhsuk_forms.forms import FormWithConditionalFields
+from manage_breast_screening.participants.models.other_procedure_history_item import (
+    OtherProcedureHistoryItem,
+)
+
+
+class OtherProcedureHistoryForm(FormWithConditionalFields):
+    PROCEDURE_DETAIL_FIELDS = {
+        OtherProcedureHistoryItem.Procedure.BREAST_REDUCTION: "breast_reduction_details",
+        OtherProcedureHistoryItem.Procedure.BREAST_SYMMETRISATION: "breast_symmetrisation_details",
+        OtherProcedureHistoryItem.Procedure.NIPPLE_CORRECTION: "nipple_correction_details",
+        OtherProcedureHistoryItem.Procedure.OTHER: "other_details",
+    }
+
+    def __init__(self, *args, participant, **kwargs):
+        self.instance = kwargs.pop("instance", None)
+
+        if self.instance:
+            kwargs["initial"] = self.initial_values(self.instance)
+
+        super().__init__(*args, **kwargs)
+
+        self.fields["procedure"] = ChoiceField(
+            choices=OtherProcedureHistoryItem.Procedure,
+            label=f"What procedure has {participant.first_name} had?",
+            error_messages={"required": "Select the procedure"},
+        )
+
+        for procedure, detail_field in self.PROCEDURE_DETAIL_FIELDS.items():
+            self.fields[detail_field] = CharField(
+                required=False,
+                label="Provide details",
+                error_messages={"required": "Provide details of the procedure"},
+                classes="nhsuk-u-width-two-thirds",
+            )
+            self.given_field_value("procedure", procedure).require_field(detail_field)
+
+        self.fields["procedure_year"] = YearField(
+            label="Year of procedure (optional)",
+            label_classes="nhsuk-label--m",
+            classes="nhsuk-input--width-4",
+            hint="Leave blank if unknown",
+            required=False,
+        )
+        self.fields["additional_details"] = CharField(
+            hint="Include any other relevant information about the device or procedure",
+            required=False,
+            label="Additional details (optional)",
+            label_classes="nhsuk-label--m",
+            widget=Textarea(attrs={"rows": 4}),
+            max_words=500,
+            error_messages={
+                "max_words": "Additional details must be 500 words or less"
+            },
+        )
+
+    def initial_values(self, instance):
+        initial = {
+            "procedure": instance.procedure,
+            "procedure_year": instance.procedure_year,
+            "additional_details": instance.additional_details,
+        }
+
+        detail_field = self.PROCEDURE_DETAIL_FIELDS.get(instance.procedure)
+        if detail_field:
+            initial[detail_field] = instance.procedure_details
+
+        return initial
+
+    def create(self, appointment, request):
+        auditor = Auditor.from_request(request)
+
+        other_procedure_history_item = OtherProcedureHistoryItem.objects.create(
+            appointment=appointment,
+            procedure=self.cleaned_data["procedure"],
+            procedure_details=self._get_selected_procedure_details(),
+            procedure_year=self.cleaned_data.get("procedure_year"),
+            additional_details=self.cleaned_data.get("additional_details", ""),
+        )
+
+        auditor.audit_create(other_procedure_history_item)
+
+        return other_procedure_history_item
+
+    def update(self, request):
+        self.instance.procedure = self.cleaned_data["procedure"]
+        self.instance.procedure_details = self._get_selected_procedure_details()
+        self.instance.procedure_year = self.cleaned_data["procedure_year"]
+        self.instance.additional_details = self.cleaned_data["additional_details"]
+        self.instance.save()
+
+        Auditor.from_request(request).audit_update(self.instance)
+
+        return self.instance
+
+    @property
+    def procedure_detail_fields(self):
+        return tuple(self.PROCEDURE_DETAIL_FIELDS.items())
+
+    def _get_selected_procedure_details(self):
+        procedure = self.cleaned_data.get("procedure")
+        try:
+            detail_field = self.PROCEDURE_DETAIL_FIELDS[procedure]
+        except KeyError as exc:
+            msg = f"Unsupported procedure '{procedure}'"
+            raise ValueError(msg) from exc
+        return self.cleaned_data.get(detail_field, "")

manage_breast_screening/mammograms/jinja2/mammograms/medical_information/medical_history/forms/other_procedure_history.jinja

@@ -0,0 +1,18 @@
+{% extends "layout-form.jinja" %}
+{% from "nhsuk/components/button/macro.jinja" import button %}
+{% from "nhsuk/components/fieldset/macro.jinja" import fieldset %}
+
+{% block form %}
+  {{ form.procedure.as_field_group() }}
+
+  {{ form.procedure_year.as_field_group() }}
+
+  {{ form.additional_details.as_field_group() }}
+
+  <div class="nhsuk-button-group">
+    {{ button({
+      "text": "Save"
+    }) }}
+  </div>
+{% endblock %}
+

manage_breast_screening/mammograms/jinja2/mammograms/record_medical_information.jinja

@@ -75,8 +75,12 @@
 
   {% set other_procedure_history_html %}
     {% for presented_item in presenter.other_procedure_history %}
+      <a style="float: right" class="nhsuk-link nhsuk-link--no-visited-state" href="{{ presented_item.change_link.href}}">
+        {{ presented_item.change_link.text }}<span class="nhsuk-u-visually-hidden">{{ presented_item.change_link.visually_hidden_text }}</span>
+      </a><br>
       {{ summaryList(presented_item.summary_list_params) }}
     {% endfor %}
+    <a href="{{ presenter.add_other_procedure_history_link.href }}" class="nhsuk-link nhsuk-link--no-visited-state">{{ presenter.add_other_procedure_history_link.text }}</a><br>
   {% endset %}
 
   {% set cyst_history_html %}

manage_breast_screening/mammograms/presenters/medical_information_presenter.py

@@ -55,10 +55,12 @@ def __init__(self, appointment):
             appointment.breast_augmentation_history_items.all(),
             BreastAugmentationHistoryItemPresenter,
         )
-        self.other_procedure_history = [
-            OtherProcedureHistoryItemPresenter(item)
-            for item in appointment.other_procedure_history_items.all()
-        ]
+
+        self.other_procedure_history = self._present_items(
+            appointment.other_procedure_history_items.all(),
+            OtherProcedureHistoryItemPresenter,
+        )
+
         self.benign_lump_history = [
             BenignLumpHistoryItemPresenter(item)
             for item in appointment.benign_lump_history_items.all()
@@ -226,6 +228,18 @@ def add_mastectomy_or_lumpectomy_history_link(self):
             "text": "Add mastectomy or lumpectomy history",
         }
 
+    @property
+    def add_other_procedure_history_link(self):
+        url = reverse(
+            "mammograms:add_other_procedure_history_item",
+            kwargs={"pk": self.appointment.pk},
+        )
+
+        return {
+            "href": url,
+            "text": "Add other procedure history",
+        }
+
     def _present_items(self, items, presenter_class):
         items = list(items)
         if len(items) == 1:

manage_breast_screening/mammograms/presenters/other_procedure_history_item_presenter.py

@@ -1,9 +1,12 @@
+from django.urls import reverse
+
 from manage_breast_screening.core.template_helpers import nl2br
 
 
 class OtherProcedureHistoryItemPresenter:
-    def __init__(self, implanted_medical_device_history_item):
-        self._item = implanted_medical_device_history_item
+    def __init__(self, other_procedure_history_item, counter=None):
+        self._item = other_procedure_history_item
+        self.counter = counter
 
         self.procedure = self._item.get_procedure_display()
         self.procedure_details = self._item.procedure_details or "N/A"
@@ -33,3 +36,21 @@ def summary_list_params(self):
                 },
             ],
         }
+
+    @property
+    def change_link(self):
+        return {
+            "href": reverse(
+                "mammograms:change_other_procedure_history_item",
+                kwargs={
+                    "pk": self._item.appointment_id,
+                    "history_item_pk": self._item.pk,
+                },
+            ),
+            "text": "Change",
+            "visually_hidden_text": (
+                f" other procedure item {self.counter}"
+                if self.counter
+                else " other procedure item"
+            ),
+        }