Add changing campaign vaccines

This adds a new step for editing campaigns where the user can choose the
vaccines that belong to a campaign. When creating a campaign the
vaccines default to the active vaccines for a particular programme type,
but we may need to add discontinued vaccines to campaigns that happened
in the past, to support importing historical records.

Author: thomasleese

app/controllers/campaigns/edit_controller.rb

@@ -48,6 +48,10 @@ def dates_params
     params.require(:campaign).permit(:start_date, :end_date)
   end
 
+  def vaccines_params
+    params.require(:campaign).permit(vaccine_ids: [])
+  end
+
   def confirm_params
     { active: true }
   end

app/models/campaign.rb

@@ -79,14 +79,26 @@ class Campaign < ApplicationRecord
               }
   end
 
+  on_wizard_step :vaccines do
+    validates :vaccines, presence: true
+  end
+
   on_wizard_step :confirm do
     validates :active, presence: true
   end
 
   validate :vaccines_match_type
 
   def wizard_steps
-    %i[details dates confirm]
+    [:details, :dates, (:vaccines if active), :confirm].compact
+  end
+
+  def vaccine_ids
+    @vaccine_ids ||= vaccines.map(&:id)
+  end
+
+  def vaccine_ids=(ids)
+    self.vaccines = Vaccine.where(id: ids)
   end
 
   private

app/views/campaigns/edit/confirm.html.erb

@@ -48,6 +48,7 @@
         <%= summary_list.with_row do |row| %>
           <%= row.with_key { "Vaccines" } %>
           <%= row.with_value { @campaign.vaccines.map(&:brand).join("<br>").html_safe } %>
+          <%= row.with_action(text: "Change", href: campaign_edit_path(@campaign, :vaccines), visually_hidden_text: "vaccines") %>
         <% end %>
       <% end %>
     <% end %>

app/views/campaigns/edit/vaccines.html.erb

@@ -0,0 +1,18 @@
+<% content_for :before_main do %>
+  <%= render AppBacklinkComponent.new(
+        href: @campaign.active ? campaign_edit_path(@campaign, :confirm) : previous_wizard_path,
+        name: @campaign.active ? "edit programme" : "#{@previous_step} page of programme creation",
+      ) %>
+<% end %>
+
+<% title = "Which vaccine(s) does this programme administer?" %>
+
+<% content_for :page_title, title %>
+
+<%= form_with model: @campaign, url: wizard_path, method: :put do |f| %>
+  <%= f.govuk_error_summary %>
+
+  <%= f.govuk_collection_check_boxes :vaccine_ids, Vaccine.where(type: @campaign.type), :id, :brand, legend: { text: title, tag: "h1", size: "l" } %>
+
+  <%= f.govuk_submit %>
+<% end %>

config/locales/en.yml

@@ -204,6 +204,7 @@ en:
               blank: Choose a programme type
               inclusion: Choose a programme type
             vaccines:
+              blank: Choose the vaccines this programme administers
               match_type: Vaccines must be suitable for the programme type
         consent:
           attributes:

spec/features/create_and_edit_campaigns_spec.rb

@@ -4,7 +4,8 @@
   before { given_i_am_signed_in }
 
   scenario "User creates a Flu campaign" do
-    given_active_and_discontinued_flu_vaccines_exist
+    given_active_flu_vaccines_exist
+    and_discontinued_flu_vaccines_exist
 
     when_i_go_to_the_campaigns_page
     and_i_click_on_the_new_campaign_button
@@ -28,6 +29,7 @@
 
   scenario "User edits a Flu campaign" do
     given_a_flu_campaign_exists
+    and_discontinued_flu_vaccines_exist
 
     when_i_go_to_the_campaigns_page
     and_i_click_on_the_flu_campaign
@@ -44,13 +46,19 @@
     and_i_click_continue
     then_i_should_see_the_edit_confirm_page
 
+    when_i_click_on_change_vaccines
+    and_i_select_the_flu_vaccines
+    and_i_click_continue
+    then_i_should_see_the_edit_confirm_page
+
     when_i_confirm_the_campaign
     then_i_should_see_the_campaign_page
     and_i_should_see_the_flu_campaign
   end
 
   scenario "User creates an HPV campaign" do
-    given_active_and_discontinued_hpv_vaccines_exist
+    given_active_hpv_vaccines_exist
+    and_discontinued_hpv_vaccines_exist
 
     when_i_go_to_the_campaigns_page
     and_i_click_on_the_new_campaign_button
@@ -74,6 +82,7 @@
 
   scenario "User edits an HPV campaign" do
     given_an_hpv_campaign_exists
+    and_discontinued_hpv_vaccines_exist
 
     when_i_go_to_the_campaigns_page
     and_i_click_on_the_hpv_campaign
@@ -90,6 +99,11 @@
     and_i_click_continue
     then_i_should_see_the_edit_confirm_page
 
+    when_i_click_on_change_vaccines
+    and_i_select_the_hpv_vaccines
+    and_i_click_continue
+    then_i_should_see_the_edit_confirm_page
+
     when_i_confirm_the_campaign
     then_i_should_see_the_campaign_page
     and_i_should_see_the_hpv_campaign
@@ -100,13 +114,21 @@ def given_i_am_signed_in
     sign_in @team.users.first
   end
 
-  def given_active_and_discontinued_flu_vaccines_exist
+  def given_active_flu_vaccines_exist
     create(:vaccine, :adjuvanted_quadrivalent)
+  end
+
+  def and_discontinued_flu_vaccines_exist
     create(:vaccine, :fluad_tetra)
+    create(:vaccine, :flucelvax_tetra)
   end
 
-  def given_active_and_discontinued_hpv_vaccines_exist
+  def given_active_hpv_vaccines_exist
     create(:vaccine, :gardasil_9)
+  end
+
+  def and_discontinued_hpv_vaccines_exist
+    create(:vaccine, :cervarix)
     create(:vaccine, :gardasil)
   end
 
@@ -241,4 +263,18 @@ def when_i_click_on_change_name
   def when_i_click_on_change_start_date
     within(".nhsuk-summary-list__row", text: "Start date") { click_on "Change" }
   end
+
+  def when_i_click_on_change_vaccines
+    within(".nhsuk-summary-list__row", text: "Vaccines") { click_on "Change" }
+  end
+
+  def and_i_select_the_flu_vaccines
+    check "Fluad Tetra - aQIV"
+    check "Flucelvax Tetra - QIVc"
+  end
+
+  def and_i_select_the_hpv_vaccines
+    check "Cervarix"
+    check "Gardasil"
+  end
 end