Rename ReadImageView -> ShowImageReadView

We also remove AppointmentMixin so that we no longer rely on an
appointment pk passed in the URL. The view now picks an arbitrary
appointment with a study attached as dummy data, with a TODO marking it
for replacement once retrieval of the appropriate Reading object is
implemented.

Author: malcolmbaig

manage_breast_screening/participants/models/appointment.py

@@ -111,6 +111,10 @@ def filter_counts_for_clinic(cls, clinic):
             counts[filter] = clinic.appointments.for_filter(filter).count()
         return counts
 
+    @classmethod
+    def with_study(cls):
+        return cls.objects.filter(study__isnull=False)
+
     @property
     def provider(self):
         return self.clinic_slot.provider

manage_breast_screening/reading/urls.py

@@ -8,7 +8,7 @@
     path("", views.show_reading_dashboard_view, name="show_reading_dashboard"),
     path(
         "sessions/<uuid:session_pk>/reads/<uuid:pk>/",
-        views.ReadImageView.as_view(),
+        views.ShowImageReadView.as_view(),
         name="image_read",
     ),
     path(

manage_breast_screening/reading/views.py

@@ -11,7 +11,7 @@
 from manage_breast_screening.mammograms.presenters.medical_history.check_medical_information_presenter import (
     CheckMedicalInformationPresenter,
 )
-from manage_breast_screening.mammograms.views.mixins import AppointmentMixin
+from manage_breast_screening.participants.models import Appointment
 
 logger = getLogger(__name__)
 
@@ -22,25 +22,29 @@ def show_reading_dashboard_view(request):
     return render(request, "show_readings.jinja")
 
 
-class ReadImageView(PermissionRequiredMixin, AppointmentMixin, FormView):
+class ShowImageReadView(PermissionRequiredMixin, FormView):
     form_class = Form
     template_name = "read_image.jinja"
     permission_required = Permission.READ_IMAGES
 
     def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
 
+        # TODO: dummy data — replace once retrieval of the appropriate Reading object is implemented.
+        appointment = Appointment.with_study().first()
+        participant = appointment.participant
+
         images = []
 
         context.update(
             {
-                "heading": self.participant.full_name,
+                "heading": participant.full_name,
                 "caption": "Review images",
                 "images": images,
                 "presented_medical_information": CheckMedicalInformationPresenter(
-                    self.appointment
+                    appointment
                 ),
-                "notes_for_reader": self.appointment.study.additional_details,
+                "notes_for_reader": appointment.study.additional_details,
                 "is_urgent": True,
                 "is_second_read": True,
                 "is_previously_skipped": True,