Merge pull request #4384 from nhsuk/prescribers-can-create-psd-instructions-in-bulk

Prescribers can create PSD instructions in bulk

Author: jhenderson

app/assets/stylesheets/components/_action-list.scss

@@ -33,3 +33,59 @@
   margin-right: 0;
   padding-right: 0;
 }
+
+.nhsuk-action-link {
+  @include nhsuk-responsive-margin(6, "bottom");
+}
+
+.nhsuk-action-link__link {
+  display: inline-block; // [1]
+  padding-left: 38px; // [2]
+  position: relative; // [3]
+  text-decoration: none; // [4]
+
+  @include nhsuk-font(22, $weight: bold);
+
+  &:not(:focus):hover {
+    .nhsuk-action-link__text {
+      text-decoration: underline; // [6]
+    }
+  }
+
+  @include nhsuk-media-query($until: tablet) {
+    padding-left: 26px; // [2]
+  }
+
+  @include nhsuk-media-query($media-type: print) {
+    color: $nhsuk-print-text-color;
+
+    &:visited {
+      color: $nhsuk-print-text-color;
+    }
+  }
+
+  .nhsuk-icon__arrow-right-circle {
+    // stylelint-disable-next-line declaration-no-important
+    fill: $color_nhsuk-green !important;
+    height: 36px;
+    left: -3px;
+    position: absolute;
+    top: -3px;
+    width: 36px;
+
+    @include nhsuk-print-color($nhsuk-print-text-color);
+
+    @include nhsuk-media-query($until: tablet) {
+      height: 24px;
+      left: -2px;
+      margin-bottom: 0;
+      top: 1px;
+      width: 24px;
+    }
+  }
+
+  &:focus .nhsuk-icon__arrow-right-circle {
+    // stylelint-disable-next-line declaration-no-important
+    fill: $color_nhsuk-black !important;
+  }
+}

app/components/app_patient_session_search_result_card_component.rb

@@ -37,7 +37,7 @@ class AppPatientSessionSearchResultCardComponent < ViewComponent::Base
               end
             end
 
-            if (note = patient_session.latest_note)
+            if context != :patient_specific_direction && (note = patient_session.latest_note)
               summary_list.with_row do |row|
                 row.with_key { "Notes" }
                 row.with_value { render note_to_log_event(note) }
@@ -57,7 +57,16 @@ class AppPatientSessionSearchResultCardComponent < ViewComponent::Base
   def initialize(patient_session, context:, programmes: [])
     super
 
-    unless context.in?(%i[patients consent triage register record])
+    unless context.in?(
+             %i[
+               patients
+               consent
+               triage
+               register
+               record
+               patient_specific_direction
+             ]
+           )
       raise "Unknown context: #{context}"
     end
 
@@ -157,6 +166,8 @@ def status_tags
       [consent_status_tag]
     when :triage
       [triage_status_tag]
+    when :patient_specific_direction
+      [patient_specific_direction_status_tag]
     else
       [vaccination_status_tag]
     end
@@ -203,8 +214,9 @@ def register_status_tag
       key: :register,
       value:
         render(
-          AppRegisterStatusTagComponent.new(
-            patient_session.registration_status&.status || "unknown"
+          AppStatusTagComponent.new(
+            patient_session.registration_status&.status || "unknown",
+            context: :register
           )
         )
     }
@@ -240,6 +252,19 @@ def triage_status_value(triage_status, programme)
     { status: status }
   end
 
+  def patient_specific_direction_status_tag
+    {
+      key: :patient_specific_direction,
+      value:
+        render(
+          AppStatusTagComponent.new(
+            psd_exists?(programmes.first) ? :added : :not_added,
+            context: :patient_specific_direction
+          )
+        )
+    }
+  end
+
   def note_to_log_event(note)
     truncated_body = note.body.truncate_words(80, omission: "…")
 
@@ -260,4 +285,10 @@ def note_to_log_event(note)
 
     AppLogEventComponent.new(body:, at: note.created_at, by: note.created_by)
   end
+
+  def psd_exists?(programme)
+    patient.patient_specific_directions.any? do
+      it.programme_id == programme.id && it.academic_year == academic_year
+    end
+  end
 end

app/components/app_register_status_tag_component.rb

@@ -2,7 +2,7 @@
 
 class AppSearchResultsComponent < ViewComponent::Base
   erb_template <<-ERB
-    <h3 class="nhsuk-heading-m nhsuk-u-margin-bottom-2">Search results</h3>
+    <h3 class="nhsuk-heading-m nhsuk-u-margin-bottom-2"><%= heading %></h3>
 
     <p class="nhsuk-caption-m nhsuk-u-margin-bottom-4">
       <% if has_results? %>
@@ -19,16 +19,17 @@ class AppSearchResultsComponent < ViewComponent::Base
     <% end %>
   ERB
 
-  def initialize(pagy, label:)
+  def initialize(pagy, label:, heading: "Search results")
     super
 
     @pagy = pagy
     @label = label
+    @heading = heading
   end
 
   private
 
-  attr_reader :pagy, :label
+  attr_reader :pagy, :label, :heading
 
   def has_results? = pagy.count.positive?
 end

app/components/app_search_results_component.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+class AppStatusTagComponent < ViewComponent::Base
+  def initialize(status, context:)
+    super
+
+    @status = status
+    @context = context
+  end
+
+  def call = tag.strong(text, class: ["nhsuk-tag nhsuk-tag--#{colour}"])
+
+  private
+
+  def text
+    I18n.t(@status, scope: [:status, @context, :label])
+  end
+
+  def colour
+    I18n.t(@status, scope: [:status, @context, :colour])
+  end
+end

app/components/app_status_tag_component.rb

@@ -0,0 +1,72 @@
+# frozen_string_literal: true
+
+class Sessions::PatientSpecificDirectionsController < ApplicationController
+  include PatientSearchFormConcern
+
+  before_action :set_session
+  before_action :set_patient_search_form
+
+  layout "full"
+
+  def show
+    scope =
+      @session.patient_sessions.includes_programmes.includes(
+        patient: {
+          patient_specific_directions: :programme
+        }
+      )
+    @eligible_for_bulk_psd_count = patient_sessions_allowed_psd.count
+    patient_sessions = @form.apply(scope)
+    @pagy, @patient_sessions = pagy(patient_sessions)
+  end
+
+  def create
+    ActiveRecord::Base.transaction do
+      PatientSpecificDirection.import!(
+        psds_to_create,
+        on_duplicate_key_ignore: true
+      )
+    end
+
+    redirect_to session_patient_specific_directions_path(@session),
+                flash: {
+                  success: "PSDs added"
+                }
+  end
+
+  def bulk_add
+    @eligible_for_bulk_psd_count = patient_sessions_allowed_psd.count
+  end
+
+  private
+
+  def set_session
+    @session = policy_scope(Session).find_by!(slug: params[:session_slug])
+  end
+
+  def programme
+    @session.programmes.includes(:vaccines).first
+  end
+
+  def psds_to_create
+    patient_sessions_allowed_psd.map do |patient_session|
+      PatientSpecificDirection.new(
+        academic_year: @session.academic_year,
+        patient_id: patient_session.patient_id,
+        programme_id: programme.id,
+        vaccine_id: programme.vaccines.first.id,
+        created_by_user_id: current_user.id,
+        vaccine_method: :nasal,
+        delivery_site: :nose
+      )
+    end
+  end
+
+  def patient_sessions_allowed_psd
+    @patient_sessions_allowed_psd ||=
+      @session
+        .patient_sessions
+        .has_consent_status(:given, programme:)
+        .without_patient_specific_direction(programme:)
+  end
+end

app/controllers/sessions/patient_specific_directions_controller.rb

@@ -257,6 +257,18 @@ class PatientSession < ApplicationRecord
           end
         end
 
+  scope :without_patient_specific_direction,
+        ->(programme:) do
+          joins(:session).where.not(
+            PatientSpecificDirection
+              .where("patient_id = patient_sessions.patient_id")
+              .where("academic_year = sessions.academic_year")
+              .where(programme:)
+              .arel
+              .exists
+          )
+        end
+
   scope :destroy_all_if_safe,
         -> do
           includes(

app/models/patient_session.rb

@@ -36,7 +36,7 @@
     <% end %>
 
     <li class="app-action-list__item">
-      <%= render AppRegisterStatusTagComponent.new(@patient_session.registration_status&.status || "unknown") %>
+      <%= render AppStatusTagComponent.new(@patient_session.registration_status&.status || "unknown", context: :register) %>
     </li>
 
     <% if policy(session_attendance).edit? && @session.requires_registration? %>

app/views/patient_sessions/_header.html.erb

@@ -24,6 +24,14 @@
         selected: request.path == session_consent_path(@session),
       )
 
+      if @session.psd_enabled?
+        nav.with_item(
+          href: session_patient_specific_directions_path(@session),
+          text: t("sessions.tabs.patient_specific_directions"),
+          selected: request.path == session_patient_specific_directions_path(@session),
+        )
+      end
+    
       nav.with_item(
         href: session_triage_path(@session),
         text: t("sessions.tabs.triage"),

app/views/sessions/_header.html.erb

@@ -0,0 +1,22 @@
+<% content_for :before_main do %>
+  <%= render AppBreadcrumbComponent.new(items: [
+                                          { text: t("dashboard.index.title"), href: dashboard_path },
+                                          { text: t("sessions.index.title"), href: sessions_path },
+                                          { text: @session.location.name, href: session_path(@session) },
+                                        ]) %>
+<% end %>
+
+<%= h1 "Are you sure you want to add #{@eligible_for_bulk_psd_count} new PSDs?" %>
+
+<div class="nhsuk-grid-row">
+  <div class="nhsuk-grid-column-two-thirds">
+    <p class="nhsuk-body">This cannot be undone.</p>
+    <div class="app-button-group">
+      <%= govuk_button_to "Yes, add PSDs", session_patient_specific_directions_path(@session) %>
+      <%= govuk_link_to "No, return to session",
+                        bulk_add_session_patient_specific_directions_path(@session),
+                        method: :post,
+                        prevent_double_click: true %>
+    </div>
+  </div>
+</div>