Add ability to edit parent details

This adds a new page which allows the nurses to edit the parent details
of a patient, including the name, relationship, phone number and email
address.

Author: thomasleese

app/components/app_child_summary_component.rb

@@ -77,6 +77,16 @@ def call
             row.with_value do
               helpers.format_parent_with_relationship(parent_relationship)
             end
+            if (
+                 href =
+                   @change_links.dig(:parent, parent_relationship.parent_id)
+               )
+              row.with_action(
+                text: "Change",
+                href:,
+                visually_hidden_text: parent_relationship.ordinal_label
+              )
+            end
           end
         end
       end

app/controllers/parents_controller.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+class ParentsController < ApplicationController
+  before_action :set_patient
+  before_action :set_parent_relationship
+  before_action :set_parent
+
+  def edit
+  end
+
+  def update
+    if @parent_relationship.update(parent_relationship_params)
+      redirect_to edit_patient_path(@patient)
+    else
+      render :edit, status: :unprocessable_entity
+    end
+  end
+
+  private
+
+  def set_patient
+    @patient = policy_scope(Patient).find(params[:patient_id])
+  end
+
+  def set_parent_relationship
+    @parent_relationship =
+      @patient
+        .parent_relationships
+        .includes(:parent)
+        .find_by!(parent_id: params[:id])
+  end
+
+  def set_parent
+    @parent = @parent_relationship.parent
+  end
+
+  def parent_relationship_params
+    params.expect(
+      parent_relationship: [
+        :type,
+        :other_name,
+        { parent_attributes: %i[id full_name email phone] }
+      ]
+    )
+  end
+end

app/models/parent_relationship.rb

@@ -49,6 +49,8 @@ class ParentRelationship < ApplicationRecord
 
   before_validation -> { self.other_name = nil unless other? }
 
+  accepts_nested_attributes_for :parent, update_only: true
+
   def label
     (other? ? other_name : human_enum_name(:type)).capitalize
   end

app/views/parents/edit.html.erb

@@ -0,0 +1,33 @@
+<% content_for :before_main do %>
+  <%= render AppBacklinkComponent.new(edit_patient_path(@patient), name: "patient") %>
+<% end %>
+
+<%= form_with model: @parent_relationship, url: patient_parent_path(@patient, @parent), method: :put do |f| %>
+  <%= f.govuk_error_summary %>
+
+  <% page_title = "Details for #{@parent_relationship.ordinal_label}" %>
+  <%= h1 page_title: do %>
+    <span class="nhsuk-caption-l"><%= @patient.full_name %></span>
+    <%= page_title %>
+  <% end %>
+
+  <%= f.fields_for :parent do |parent_f| %>
+    <%= parent_f.govuk_text_field :full_name, label: { text: "Name" } %>
+  <% end %>
+
+  <%= f.govuk_radio_buttons_fieldset :type, legend: { text: "Relationship to child", size: "s" } do %>
+    <%= f.govuk_radio_button :type, :mother, label: { text: "Mum" }, link_errors: true %>
+    <%= f.govuk_radio_button :type, :father, label: { text: "Dad" } %>
+    <%= f.govuk_radio_button :type, :guardian, label: { text: "Guardian" } %>
+    <%= f.govuk_radio_button :type, :other, label: { text: "Other" } do %>
+      <%= f.govuk_text_field :other_name, label: { text: "Relationship to the child" }, hint: { text: "For example, carer" } %>
+    <% end %>
+  <% end %>
+
+  <%= f.fields_for :parent do |parent_f| %>
+    <%= parent_f.govuk_text_field :email, label: { text: "Email address" } %>
+    <%= parent_f.govuk_text_field :phone, label: { text: "Phone number" } %>
+  <% end %>
+
+  <%= f.govuk_submit "Continue" %>
+<% end %>

app/views/patients/edit.html.erb

@@ -9,8 +9,13 @@
   <%= page_title %>
 <% end %>
 
-<%= render AppPatientCardComponent.new(@patient, change_links: {
-                                                   nhs_number: edit_nhs_number_patient_path(@patient),
-                                                 }) %>
+<% change_links = {
+     nhs_number: edit_nhs_number_patient_path(@patient),
+     parent: @patient.parent_relationships.each_with_object({}) do |parent_relationship, memo|
+       memo[parent_relationship.parent_id] = edit_patient_parent_path(@patient, parent_relationship.parent_id)
+     end,
+   } %>
+
+<%= render AppPatientCardComponent.new(@patient, change_links:) %>
 
 <%= govuk_button_link_to "Continue", patient_path(@patient) %>

config/locales/en.yml

@@ -342,6 +342,10 @@ en:
             contact_method_other_details:
               blank: Enter details about how to contact you
               too_long: Enter details that are less than 300 characters long
+            email:
+              blank: Enter an email address
+            phone:
+              blank: Enter a phone number
         patient:
           attributes:
             nhs_number:

config/routes.rb

@@ -146,6 +146,8 @@
   resources :patients, only: %i[index show edit update] do
     post "", action: :index, on: :collection
 
+    resources :parents, only: %i[edit update]
+
     member do
       get "log"
 

spec/features/edit_parent_spec.rb

@@ -0,0 +1,108 @@
+# frozen_string_literal: true
+
+describe "Edit parent" do
+  before { given_a_patient_with_a_parent_exists }
+
+  scenario "User edits the name of a parent" do
+    when_i_visit_the_patient_page
+    and_i_click_on_edit_child_record
+    and_i_click_on_change_parent
+    then_i_see_the_edit_parent_page
+
+    when_i_change_the_name_of_the_parent
+    then_i_see_the_new_name_of_the_parent
+  end
+
+  scenario "User edits the relationship of a parent" do
+    when_i_visit_the_patient_page
+    and_i_click_on_edit_child_record
+    and_i_click_on_change_parent
+    then_i_see_the_edit_parent_page
+
+    when_i_change_the_relationship_of_the_parent_to_mother
+    then_i_see_the_new_relationship_of_the_parent_of_mother
+
+    when_i_click_on_change_parent
+    and_i_change_the_relationship_of_the_parent_to_other
+    then_i_see_the_new_relationship_of_the_parent_of_other
+  end
+
+  scenario "User edits the contact details of a parent" do
+    when_i_visit_the_patient_page
+    and_i_click_on_edit_child_record
+    and_i_click_on_change_parent
+    then_i_see_the_edit_parent_page
+
+    when_i_change_the_contact_details_of_the_parent
+    then_i_see_the_new_contact_details_of_the_parent
+  end
+
+  def given_a_patient_with_a_parent_exists
+    organisation = create(:organisation)
+    @nurse = create(:nurse, organisation:)
+
+    @patient = create(:patient, organisation:)
+
+    @parent = create(:parent)
+
+    create(:parent_relationship, patient: @patient, parent: @parent)
+  end
+
+  def when_i_visit_the_patient_page
+    sign_in @nurse
+    visit patient_path(@patient)
+  end
+
+  def and_i_click_on_edit_child_record
+    click_on "Edit child record"
+  end
+
+  def when_i_click_on_change_parent
+    click_on "Change first parent or guardian"
+  end
+
+  alias_method :and_i_click_on_change_parent, :when_i_click_on_change_parent
+
+  def then_i_see_the_edit_parent_page
+    expect(page).to have_content("Details for first parent or guardian")
+  end
+
+  def when_i_change_the_name_of_the_parent
+    fill_in "Name", with: "Selina Meyer"
+    click_on "Continue"
+  end
+
+  def then_i_see_the_new_name_of_the_parent
+    expect(page).to have_content("Selina Meyer")
+  end
+
+  def when_i_change_the_relationship_of_the_parent_to_mother
+    choose "Mum"
+    click_on "Continue"
+  end
+
+  def then_i_see_the_new_relationship_of_the_parent_of_mother
+    expect(page).to have_content("Mum")
+  end
+
+  def and_i_change_the_relationship_of_the_parent_to_other
+    choose "Other"
+    fill_in "Relationship to the child", with: "Someone"
+    click_on "Continue"
+  end
+
+  def then_i_see_the_new_relationship_of_the_parent_of_other
+    expect(page).to have_content("Someone")
+  end
+
+  def when_i_change_the_contact_details_of_the_parent
+    fill_in "Email address", with: "selina@meyer.com"
+    fill_in "Phone number", with: "07700 900 000"
+    click_on "Continue"
+  end
+
+  def then_i_see_the_new_contact_details_of_the_parent
+    expect(page).to have_content("selina@meyer.com")
+    expect(page).to have_content("07700 900000")
+  end
+end