Store vaccine method on consent forms

This adds a new column on consent forms (per programme) allowing parents
to express a preference regarding the vaccine method. In most cases this
is irrelevant as injection is the only option, but for Flu there's the
option of a nasal spray which is the preferred choice.

Jira-Issue: MAV-1230

Author: thomasleese

app/models/consent.rb

@@ -13,6 +13,7 @@
 #  response            :integer          not null
 #  route               :integer          not null
 #  submitted_at        :datetime         not null
+#  vaccine_method      :integer
 #  withdrawn_at        :datetime
 #  created_at          :datetime         not null
 #  updated_at          :datetime         not null
@@ -69,6 +70,7 @@ class Consent < ApplicationRecord
        { website: 0, phone: 1, paper: 2, in_person: 3, self_consent: 4 },
        prefix: "via",
        validate: true
+  enum :vaccine_method, { injection: 0, nasal: 1 }, prefix: true
 
   enum :reason_for_refusal,
        {
@@ -158,42 +160,44 @@ def self.from_consent_form!(consent_form, patient:, current_user:)
       parent =
         consent_form.find_or_create_parent_with_relationship_to!(patient:)
 
-      consent_given =
-        consent_form.given_programmes.map do |programme|
-          patient.consents.create!(
-            consent_form:,
-            organisation: consent_form.organisation,
-            programme:,
-            parent:,
-            notes: "",
-            response: "given",
-            route: "website",
-            health_answers: consent_form.health_answers,
-            recorded_by: current_user,
-            submitted_at: consent_form.recorded_at
-          )
-        end
-
-      consent_refused =
-        consent_form.refused_programmes.map do |programme|
-          patient.consents.create!(
-            consent_form:,
-            organisation: consent_form.organisation,
-            programme:,
-            parent:,
-            reason_for_refusal: consent_form.reason,
-            notes: consent_form.reason_notes.presence || "",
-            response: "refused",
-            route: "website",
-            health_answers: consent_form.health_answers,
-            recorded_by: current_user,
-            submitted_at: consent_form.recorded_at
-          )
-        end
+      consents =
+        consent_form
+          .consent_form_programmes
+          .includes(:programme)
+          .map do |consent_form_programme|
+            patient.consents.create!(
+              consent_form:,
+              health_answers: consent_form.health_answers,
+              notes:
+                (
+                  if consent_form_programme.response_given?
+                    ""
+                  else
+                    consent_form.reason_notes.presence || ""
+                  end
+                ),
+              organisation: consent_form.organisation,
+              parent:,
+              programme: consent_form_programme.programme,
+              reason_for_refusal:
+                (
+                  if consent_form_programme.response_given?
+                    nil
+                  else
+                    consent_form.reason
+                  end
+                ),
+              recorded_by: current_user,
+              response: consent_form_programme.response,
+              route: "website",
+              submitted_at: consent_form.recorded_at,
+              vaccine_method: consent_form_programme.vaccine_method
+            )
+          end
 
       StatusUpdater.call(patient:)
 
-      consent_given + consent_refused
+      consents
     end
   end
 

app/models/consent_form.rb

@@ -103,14 +103,6 @@ class ConsentForm < ApplicationRecord
            through: :refused_consent_form_programmes,
            source: :programme
 
-  has_many :vaccines, through: :consent_form_programmes
-  has_many :given_vaccines,
-           through: :given_consent_form_programmes,
-           source: :vaccines
-  has_many :refused_vaccines,
-           through: :refused_consent_form_programmes,
-           source: :vaccines
-
   has_one :team, through: :location
 
   has_many :eligible_schools, through: :organisation, source: :schools
@@ -466,7 +458,10 @@ def home_educated_changed?
   def update_programme_responses
     case response
     when "given"
-      consent_form_programmes.each { it.response = "given" }
+      consent_form_programmes.each do
+        it.response = "given"
+        it.vaccine_method = "injection"
+      end
     when "given_one"
       consent_form_programmes.each do |consent_form_programme|
         consent_form_programme.response =
@@ -475,6 +470,7 @@ def update_programme_responses
           else
             "refused"
           end
+        consent_form_programme.vaccine_method = "injection"
       end
     when "refused"
       consent_form_programmes.each { it.response = "refused" }

app/models/consent_form_programme.rb

@@ -6,6 +6,7 @@
 #
 #  id              :bigint           not null, primary key
 #  response        :integer
+#  vaccine_method  :integer
 #  consent_form_id :bigint           not null
 #  programme_id    :bigint           not null
 #
@@ -28,4 +29,5 @@ class ConsentFormProgramme < ApplicationRecord
   scope :ordered, -> { joins(:programme).order(:"programme.type") }
 
   enum :response, { given: 0, refused: 1 }, prefix: true
+  enum :vaccine_method, { injection: 0, nasal: 1 }, prefix: true
 end

db/migrate/20250606083902_add_vaccine_method_to_consent_and_consent_forms.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+class AddVaccineMethodToConsentAndConsentForms < ActiveRecord::Migration[8.0]
+  def change
+    add_column :consent_form_programmes, :vaccine_method, :integer
+
+    add_column :consents, :vaccine_method, :integer
+
+    # We don't support getting consent for nasal spray yet.
+    reversible do |dir|
+      dir.up do
+        ConsentFormProgramme.response_given.update_all(
+          vaccine_method: "injection"
+        )
+        Consent.response_given.update_all(vaccine_method: "injection")
+      end
+    end
+
+    remove_column :consent_forms, :contact_injection, :boolean
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.0].define(version: 2025_06_06_070424) do
+ActiveRecord::Schema[8.0].define(version: 2025_06_06_083902) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_catalog.plpgsql"
   enable_extension "pg_trgm"
@@ -156,6 +156,7 @@
     t.bigint "programme_id", null: false
     t.bigint "consent_form_id", null: false
     t.integer "response"
+    t.integer "vaccine_method"
     t.index ["consent_form_id"], name: "index_consent_form_programmes_on_consent_form_id"
     t.index ["programme_id", "consent_form_id"], name: "idx_on_programme_id_consent_form_id_2113cb7f37", unique: true
   end
@@ -236,6 +237,7 @@
     t.datetime "invalidated_at"
     t.boolean "notify_parents"
     t.datetime "submitted_at", null: false
+    t.integer "vaccine_method"
     t.index ["organisation_id"], name: "index_consents_on_organisation_id"
     t.index ["parent_id"], name: "index_consents_on_parent_id"
     t.index ["patient_id"], name: "index_consents_on_patient_id"

spec/factories/consent_form_programmes.rb

@@ -6,6 +6,7 @@
 #
 #  id              :bigint           not null, primary key
 #  response        :integer
+#  vaccine_method  :integer
 #  consent_form_id :bigint           not null
 #  programme_id    :bigint           not null
 #

spec/factories/consents.rb

@@ -13,6 +13,7 @@
 #  response            :integer          not null
 #  route               :integer          not null
 #  submitted_at        :datetime         not null
+#  vaccine_method      :integer
 #  withdrawn_at        :datetime
 #  created_at          :datetime         not null
 #  updated_at          :datetime         not null

spec/models/consent_form_spec.rb

@@ -67,7 +67,7 @@
         session:,
         programmes:,
         use_preferred_name:,
-        wizard_step:,
+        wizard_step:
       )
     end
 

spec/models/consent_spec.rb

@@ -13,6 +13,7 @@
 #  response            :integer          not null
 #  route               :integer          not null
 #  submitted_at        :datetime         not null
+#  vaccine_method      :integer
 #  withdrawn_at        :datetime
 #  created_at          :datetime         not null
 #  updated_at          :datetime         not null
@@ -94,7 +95,7 @@
   end
 
   describe "#from_consent_form!" do
-    describe "the created consent object" do
+    context "with on programme" do
       subject(:consent) do
         described_class.from_consent_form!(
           consent_form,
@@ -111,7 +112,7 @@
         expect(consent.recorded_by).to eq(current_user)
       end
 
-      it "copies over attributes from consent_form" do
+      it "copies over attributes from consent form" do
         expect(consent).to(
           have_attributes(
             programme: consent_form.programmes.first,
@@ -160,9 +161,21 @@
           )
         end
       end
+
+      context "when consenting to nasal spray" do
+        before do
+          consent_form.consent_form_programmes.first.update!(
+            vaccine_method: "nasal"
+          )
+        end
+
+        it "stores this preference on the consent" do
+          expect(consent).to be_vaccine_method_nasal
+        end
+      end
     end
 
-    context "when only consenting to one programme" do
+    context "with multiple programmes" do
       subject(:consents) do
         described_class.from_consent_form!(
           consent_form,