Rename register status

The model class is called registration status so this helps to keep the
wording consistent.

Jira-Issue: MAV-2581

Author: thomasleese

app/components/app_patient_search_form_component.rb

@@ -86,16 +86,16 @@ class AppPatientSearchFormComponent < ViewComponent::Base
           <% end %>
         <% end %>
 
-        <% if register_statuses.any? %>
-          <%= f.govuk_radio_buttons_fieldset :register_status,
+        <% if registration_statuses.any? %>
+          <%= f.govuk_radio_buttons_fieldset :registration_status,
                                              legend: { text: "Registration status", size: "s" },
                                              small: true do %>
-            <%= f.govuk_radio_button :register_status, "", checked: form.register_status.blank?, label: { text: "Any" } %>
-            <% register_statuses.each do |status| %>
-              <%= f.govuk_radio_button :register_status,
+            <%= f.govuk_radio_button :registration_status, "", checked: form.registration_status.blank?, label: { text: "Any" } %>
+            <% registration_statuses.each do |status| %>
+              <%= f.govuk_radio_button :registration_status,
                                        status,
-                                       checked: form.register_status == status,
-                                       label: { text: t(status, scope: %i[status register label]) } %>
+                                       checked: form.registration_status == status,
+                                       label: { text: t(status, scope: %i[status registration label]) } %>
             <% end %>
           <% end %>
         <% end %>
@@ -236,7 +236,7 @@ def initialize(
     url:,
     programmes: [],
     consent_statuses: [],
-    register_statuses: [],
+    registration_statuses: [],
     triage_statuses: [],
     vaccination_statuses: [],
     patient_specific_direction_statuses: [],
@@ -252,7 +252,7 @@ def initialize(
 
     @programmes = programmes
     @consent_statuses = consent_statuses
-    @register_statuses = register_statuses
+    @registration_statuses = registration_statuses
     @triage_statuses = triage_statuses
     @vaccination_statuses = vaccination_statuses
     @patient_specific_direction_statuses = patient_specific_direction_statuses
@@ -270,7 +270,7 @@ def initialize(
               :url,
               :programmes,
               :consent_statuses,
-              :register_statuses,
+              :registration_statuses,
               :triage_statuses,
               :vaccination_statuses,
               :patient_specific_direction_statuses,
@@ -296,7 +296,7 @@ def open_details?
   def show_buttons_in_details?
     !(
       consent_statuses.any? || vaccination_statuses.any? ||
-        register_statuses.any? || triage_statuses.any? || year_groups.any?
+        registration_statuses.any? || triage_statuses.any? || year_groups.any?
     )
   end
 

app/components/app_patient_session_search_result_card_component.rb

@@ -170,7 +170,7 @@ def status_tags
     when :record
       []
     when :register
-      [register_status_tag, vaccination_status_tag]
+      [registration_status_tag, vaccination_status_tag]
     when :consent
       [consent_status_tag]
     when :triage
@@ -196,14 +196,14 @@ def vaccination_status_tag
     }
   end
 
-  def register_status_tag
+  def registration_status_tag
     {
-      key: :register,
+      key: :registration,
       value:
         render(
           AppStatusTagComponent.new(
             patient.registration_status(session:)&.status || "unknown",
-            context: :register
+            context: :registration
           )
         )
     }

app/components/app_session_actions_component.rb

@@ -96,7 +96,7 @@ def register_attendance_row
 
     status = "unknown"
     count = patients.has_registration_status(status, session:).count
-    href = session_register_path(session, register_status: status)
+    href = session_register_path(session, registration_status: status)
 
     generate_row(:children_to_register, count:, href:)
   end

app/controllers/concerns/patient_search_form_concern.rb

@@ -30,7 +30,7 @@ def patient_search_form_params
       :missing_nhs_number,
       :patient_specific_direction_status,
       :q,
-      :register_status,
+      :registration_status,
       :still_to_vaccinate,
       :triage_status,
       :vaccination_status,

app/forms/patient_search_form.rb

@@ -14,7 +14,7 @@ class PatientSearchForm < SearchForm
   attribute :patient_specific_direction_status, :string
   attribute :programme_types, array: true
   attribute :q, :string
-  attribute :register_status, :string
+  attribute :registration_status, :string
   attribute :still_to_vaccinate, :boolean
   attribute :triage_status, :string
   attribute :vaccination_status, :string
@@ -84,7 +84,7 @@ def apply(scope)
     scope = filter_programmes(scope)
     scope = filter_consent_statuses(scope)
     scope = filter_vaccination_statuses(scope)
-    scope = filter_register_status(scope)
+    scope = filter_registration_status(scope)
     scope = filter_triage_status(scope)
     scope = filter_vaccine_criteria(scope)
     scope = filter_patient_specific_direction_status(scope)
@@ -237,8 +237,8 @@ def filter_patient_specific_direction_status(scope)
     end
   end
 
-  def filter_register_status(scope)
-    if (status = register_status&.to_sym).present?
+  def filter_registration_status(scope)
+    if (status = registration_status&.to_sym).present?
       scope.has_registration_status(status, session:)
     else
       scope

app/views/patient_sessions/_header.html.erb

@@ -42,7 +42,7 @@
     <% attendance_record = @patient.attendance_records.find_or_initialize_by(location: @session.location, date:) %>
 
     <li class="app-action-list__item">
-      <%= render AppStatusTagComponent.new(@patient.registration_status(session: @session)&.status || "unknown", context: :register) %>
+      <%= render AppStatusTagComponent.new(@patient.registration_status(session: @session)&.status || "unknown", context: :registration) %>
     </li>
 
     <% attendance_record.session = @session %>

app/views/sessions/register/show.html.erb

@@ -16,7 +16,7 @@
             url: session_register_path(@session),
             programmes: @session.programmes,
             vaccination_statuses: Patient::VaccinationStatus.statuses.keys,
-            register_statuses: @statuses,
+            registration_statuses: @statuses,
             year_groups: @session.year_groups,
           ) %>
     </div>

config/locales/status.en.yml

@@ -3,7 +3,7 @@ en:
     label:
       consent: Consent status
       patient_specific_direction: PSD status
-      register: Registration status
+      registration: Registration status
       triage: Triage status
       vaccination: Programme status
     consent:
@@ -32,7 +32,7 @@ en:
       colour:
         added: green
         not_added: grey
-    register:
+    registration:
       label:
         attending: Attending session
         completed: Completed session

spec/forms/patient_search_form_spec.rb

@@ -32,7 +32,7 @@
   let(:vaccination_status) { nil }
   let(:programme_types) { nil }
   let(:q) { "query" }
-  let(:register_status) { nil }
+  let(:registration_status) { nil }
   let(:triage_status) { nil }
   let(:vaccine_criteria) { nil }
   let(:still_to_vaccinate) { nil }
@@ -53,7 +53,7 @@
       patient_specific_direction_status:,
       programme_types:,
       q:,
-      register_status:,
+      registration_status:,
       triage_status:,
       vaccine_criteria:,
       year_groups:
@@ -79,7 +79,7 @@
       let(:missing_nhs_number) { nil }
       let(:vaccination_status) { nil }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -129,7 +129,7 @@
       let(:missing_nhs_number) { nil }
       let(:vaccination_status) { nil }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -175,7 +175,7 @@
       let(:missing_nhs_number) { nil }
       let(:vaccination_status) { nil }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -231,7 +231,7 @@
       let(:vaccination_status) { nil }
       let(:programme_types) { programmes.map(&:type) }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -328,7 +328,7 @@
       let(:vaccination_status) { nil }
       let(:programme_types) { programmes.map(&:type) }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -361,7 +361,7 @@
       let(:vaccination_status) { "vaccinated" }
       let(:programme_types) { programmes.map(&:type) }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -386,7 +386,7 @@
       let(:missing_nhs_number) { nil }
       let(:vaccination_status) { nil }
       let(:q) { nil }
-      let(:register_status) { "attending" }
+      let(:registration_status) { "attending" }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -407,7 +407,7 @@
       let(:vaccination_status) { nil }
       let(:programme_types) { programmes.map(&:type) }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { "required" }
       let(:year_groups) { nil }
 
@@ -475,7 +475,7 @@
       let(:vaccination_status) { nil }
       let(:programme_types) { nil }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -519,7 +519,7 @@
       let(:vaccination_status) { nil }
       let(:programme_types) { programmes.map(&:type) }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:vaccine_criteria) { "nasal" }
       let(:year_groups) { nil }
@@ -563,7 +563,7 @@
       let(:missing_nhs_number) { nil }
       let(:vaccination_status) { nil }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
 
@@ -636,7 +636,7 @@
       let(:vaccination_status) { nil }
       let(:programme_types) { nil }
       let(:q) { nil }
-      let(:register_status) { nil }
+      let(:registration_status) { nil }
       let(:triage_status) { nil }
       let(:year_groups) { nil }
       let(:still_to_vaccinate) { "1" }