Merge pull request #6449 from NHSDigital/needs-consent-sub-statuses-attempt-no-contact-details

Add `needs_consent_no_contact_details` status and supporting logic

Author: jhenderson

app/components/app_patient_session_consent_component.html.erb

@@ -19,6 +19,8 @@
       <p><%= who_refused %> refused to give consent.</p>
     <% elsif consent_status_generator.status == :given %>
       <p><%= patient.full_name %> is ready for the vaccinator.</p>
+    <% elsif consent_status_value == :no_contact_details %>
+      <p>We cannot send consent requests because we have no parent contact details.</p>
     <% end %>
 
     <div class="nhsuk-button-group">

app/components/app_patient_session_consent_component.rb

@@ -5,11 +5,12 @@ def initialize(patient:, session:, programme:)
     @patient = patient
     @session = session
     @programme = programme
+    @parents = patient.parents
   end
 
   private
 
-  attr_reader :patient, :session, :programme
+  attr_reader :patient, :session, :programme, :parents
 
   delegate :academic_year, :team, to: :session
 
@@ -61,7 +62,8 @@ def consent_status_generator
         academic_year:,
         patient:,
         consents:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 
@@ -73,7 +75,8 @@ def triage_status_generator
         patient:,
         consents:,
         triages:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 
@@ -116,7 +119,7 @@ def vaccination_records
   def can_send_consent_request?
     consent_status_value == :no_response &&
       patient.send_notifications?(team: @session.team) &&
-      session.can_receive_consent? && patient.parents.any?
+      session.can_receive_consent? && patient.parents.any?(&:contactable?)
   end
 
   def grouped_consents

app/components/app_patient_session_triage_component.rb

@@ -13,6 +13,7 @@ def initialize(
     @programme = programme
     @current_user = current_user
     @triage_form = triage_form || default_triage_form
+    @parents = patient.parents
   end
 
   def render?
@@ -22,7 +23,12 @@ def render?
 
   private
 
-  attr_reader :patient, :session, :programme, :current_user, :triage_form
+  attr_reader :patient,
+              :session,
+              :programme,
+              :current_user,
+              :triage_form,
+              :parents
 
   delegate :academic_year, :team, to: :session
 
@@ -70,7 +76,8 @@ def triage_status_generator
         patient:,
         consents:,
         triages:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 
@@ -81,7 +88,8 @@ def consent_status_generator
         academic_year:,
         patient:,
         consents:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 

app/forms/triage_form.rb

@@ -129,7 +129,8 @@ def consent_status_generator
         academic_year:,
         patient:,
         consents: patient.consents,
-        vaccination_records: []
+        vaccination_records: [],
+        parents: []
       )
   end
 

app/lib/patient_status_updater.rb

@@ -48,7 +48,8 @@ def update_programme_statuses!
         :patient,
         :patient_locations,
         :triages,
-        :vaccination_records
+        :vaccination_records,
+        :parents
       )
       .find_in_batches(batch_size: 10_000) do |batch|
         batch.each(&:assign)

app/lib/status_generator/consent.rb

@@ -6,13 +6,15 @@ def initialize(
     academic_year:,
     patient:,
     consents:,
-    vaccination_records:
+    vaccination_records:,
+    parents:
   )
     @programme_type = programme_type
     @academic_year = academic_year
     @patient = patient
     @consents = consents
     @vaccination_records = vaccination_records
+    @parents = parents
   end
 
   def programme
@@ -28,6 +30,8 @@ def status
       :follow_up_requested
     elsif status_should_be_conflicts?
       :conflicts
+    elsif status_should_be_no_contact_details?
+      :no_contact_details
     elsif status_should_be_no_response?
       :no_response
     else
@@ -57,7 +61,8 @@ def disease_types
               :academic_year,
               :patient,
               :consents,
-              :vaccination_records
+              :vaccination_records,
+              :parents
 
   def vaccinated?
     return @vaccinated if defined?(@vaccinated)
@@ -122,6 +127,10 @@ def status_should_be_conflicts?
 
   def status_should_be_no_response? = !vaccinated?
 
+  def status_should_be_no_contact_details?
+    parents.none?(&:contactable?)
+  end
+
   def agreed_vaccine_methods
     @agreed_vaccine_methods ||=
       consents_for_status.map(&:vaccine_methods).inject(&:intersection)

app/lib/status_generator/programme.rb

@@ -16,7 +16,8 @@ def initialize(
     consents:,
     triages:,
     attendance_record:,
-    vaccination_records:
+    vaccination_records:,
+    parents:
   )
     @programme_type = programme_type
     @academic_year = academic_year
@@ -25,6 +26,8 @@ def initialize(
     @consents = consents
     @triages = triages
     @attendance_record = attendance_record
+    @vaccination_records = vaccination_records
+    @parents = parents
 
     @vaccination_criteria =
       VaccinationCriteria.new(
@@ -54,6 +57,8 @@ def status
       :cannot_vaccinate_absent
     elsif should_be_cannot_vaccinate_do_not_vaccinate?
       :cannot_vaccinate_do_not_vaccinate
+    elsif should_be_needs_consent_no_contact_details?
+      :needs_consent_no_contact_details
     elsif should_be_needs_consent_no_response?
       :needs_consent_no_response
     elsif should_be_cannot_vaccinate_delay_vaccination?
@@ -166,7 +171,8 @@ def consent_vaccine_methods
               :consents,
               :triages,
               :attendance_record,
-              :vaccination_criteria
+              :vaccination_criteria,
+              :parents
 
   delegate :vaccinated?,
            :vaccinated_vaccination_record,
@@ -238,6 +244,10 @@ def should_be_needs_consent_request_not_scheduled?
     false # TODO: Implement this status.
   end
 
+  def should_be_needs_consent_no_contact_details?
+    is_eligible? && consent_status == :no_contact_details
+  end
+
   def year_group = patient.year_group(academic_year:)
 
   def is_eligible?
@@ -285,7 +295,8 @@ def consent_generator
         academic_year:,
         patient:,
         consents:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 
@@ -297,7 +308,8 @@ def triage_generator
         patient:,
         consents:,
         triages:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 end

app/lib/status_generator/triage.rb

@@ -7,14 +7,16 @@ def initialize(
     patient:,
     consents:,
     triages:,
-    vaccination_records:
+    vaccination_records:,
+    parents:
   )
     @programme_type = programme_type
     @academic_year = academic_year
     @patient = patient
     @consents = consents
     @triages = triages
     @vaccination_records = vaccination_records
+    @parents = parents
   end
 
   def programme
@@ -77,7 +79,8 @@ def vaccination_history_requires_triage?
               :patient,
               :consents,
               :triages,
-              :vaccination_records
+              :vaccination_records,
+              :parents
 
   def vaccinated?
     return @vaccinated if defined?(@vaccinated)
@@ -129,7 +132,8 @@ def consent_generator
         academic_year:,
         patient:,
         consents:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 

app/models/draft_vaccination_record.rb

@@ -351,7 +351,8 @@ def vaccine_method_matches_consent_and_triage?
         academic_year:,
         patient:,
         consents: patient.consents,
-        vaccination_records: []
+        vaccination_records: [],
+        parents: []
       )
 
     triage_generator =
@@ -361,7 +362,8 @@ def vaccine_method_matches_consent_and_triage?
         patient:,
         consents: patient.consents,
         triages: patient.triages,
-        vaccination_records: []
+        vaccination_records: [],
+        parents: []
       )
 
     approved_vaccine_methods =

app/models/patient/programme_status.rb

@@ -63,6 +63,8 @@ class Patient::ProgrammeStatus < ApplicationRecord
           through: :patient,
           source: :attendance_records
 
+  has_many :parents, through: :patient
+
   GROUPS = %w[
     not_eligible
     needs_consent
@@ -80,7 +82,8 @@ class Patient::ProgrammeStatus < ApplicationRecord
     "needs_consent_request_scheduled" => 11,
     "needs_consent_request_failed" => 12,
     "needs_consent_no_response" => 13,
-    "needs_consent_follow_up_requested" => 14
+    "needs_consent_follow_up_requested" => 14,
+    "needs_consent_no_contact_details" => 15
   }.freeze
 
   HAS_REFUSAL_STATUSES = {
@@ -126,7 +129,8 @@ class Patient::ProgrammeStatus < ApplicationRecord
          refused: 2,
          conflicts: 3,
          not_required: 4,
-         follow_up_requested: 5
+         follow_up_requested: 5,
+         no_contact_details: 6
        },
        default: :no_response,
        prefix: :consent,
@@ -195,7 +199,8 @@ def generator
         consents:,
         triages:,
         attendance_record:,
-        vaccination_records:
+        vaccination_records:,
+        parents:
       )
   end
 end