wip

Author: thomasleese

Gemfile

@@ -48,13 +48,14 @@ gem "jwt"
 gem "mechanize"
 gem "notifications-ruby-client"
 gem "okcomputer"
-gem "omniauth_openid_connect"
 gem "omniauth-rails_csrf_protection"
+gem "omniauth_openid_connect"
 gem "pagy"
 gem "phonelib"
 gem "pundit"
 gem "rails_semantic_logger"
 gem "rainbow"
+gem "redis"
 gem "ruby-progressbar"
 gem "rubyzip"
 gem "sentry-rails"

Gemfile.lock

@@ -520,6 +520,8 @@ GEM
       erb
       psych (>= 4.0.0)
     redcarpet (3.6.1)
+    redis (5.4.1)
+      redis-client (>= 0.22.0)
     redis-client (0.25.3)
       connection_pool
     redis-prescription (2.6.0)
@@ -825,6 +827,7 @@ DEPENDENCIES
   rails (~> 8.0.3)
   rails_semantic_logger
   rainbow
+  redis
   rladr
   rspec
   rspec-html-matchers

app/controllers/api/testing/teams_controller.rb

@@ -67,6 +67,8 @@ def destroy
       VaccinationRecord.where(performed_ods_code: team.organisation.ods_code)
     )
 
+    TeamCachedCounts.new(team).reset_all!
+
     unless keep_itself
       log_destroy(Session.where(team:))
 

app/controllers/consent_forms_controller.rb

@@ -35,6 +35,8 @@ def edit_match
   def update_match
     @consent_form.match_with_patient!(@patient, current_user:)
 
+    reset_count!
+
     session =
       @patient
         .sessions
@@ -108,6 +110,8 @@ def create_patient
       school_move.confirm!
 
       @consent_form.match_with_patient!(patient, current_user:)
+
+      reset_count!
     end
 
     if patient.nhs_number.nil?
@@ -143,4 +147,8 @@ def set_patient
   def archive_params
     params.expect(consent_form: :notes).merge(archived_at: Time.current)
   end
+
+  def reset_count!
+    TeamCachedCounts.new(current_team).reset_unmatched_consent_responses!
+  end
 end

app/controllers/imports/issues_controller.rb

@@ -70,7 +70,8 @@ def set_patient
   def set_form
     apply_changes = params.dig(:import_duplicate_form, :apply_changes)
 
-    @form = ImportDuplicateForm.new(object: @record, apply_changes:)
+    @form =
+      ImportDuplicateForm.new(current_team:, object: @record, apply_changes:)
   end
 
   def set_type

app/controllers/parent_interface/consent_forms_controller.rb

@@ -55,6 +55,8 @@ def record
 
       session.delete(:consent_form_id)
 
+      TeamCachedCounts.new(@team).reset_unmatched_consent_responses!
+
       send_consent_form_confirmation(@consent_form)
 
       ConsentFormMatchingJob.perform_later(@consent_form)

app/forms/import_duplicate_form.rb

@@ -3,7 +3,7 @@
 class ImportDuplicateForm
   include ActiveModel::Model
 
-  attr_accessor :object, :apply_changes
+  attr_accessor :current_team, :object, :apply_changes
 
   validates :apply_changes, inclusion: { in: :apply_changes_options }
 
@@ -21,6 +21,8 @@ def save
       end
     end
 
+    reset_count!
+
     true
   rescue ActiveRecord::RecordInvalid
     errors.add(:base, "Failed to save changes")
@@ -64,4 +66,8 @@ def discard_pending_changes!
   def keep_both_changes!
     object.apply_pending_changes_to_new_record! if can_keep_both? && can_apply?
   end
+
+  def reset_count!
+    TeamCachedCounts.new(current_team).reset_import_issues!
+  end
 end

app/forms/school_move_form.rb

@@ -20,6 +20,12 @@ def save
       @school_move.ignore!
     end
 
+    TeamCachedCounts.new(team).reset_school_moves!
+
     true
   end
+
+  private
+
+  def team = current_user.selected_team
 end

app/jobs/commit_patient_changesets_job.rb

@@ -29,6 +29,8 @@ def perform(import)
 
       import.postprocess_rows!
 
+      reset_counts(import)
+
       import.update_columns(
         processed_at: Time.zone.now,
         status: :processed,
@@ -186,4 +188,10 @@ def has_auto_confirmable_school_move?(school_move, import)
         academic_year: import.academic_year
       ) || school_move.patient.archived?(team: import.team)
   end
+
+  def reset_counts(import)
+    cached_counts = TeamCachedCounts.new(import.team)
+    cached_counts.reset_import_issues!
+    cached_counts.reset_school_moves!
+  end
 end

app/jobs/consent_form_matching_job.rb

@@ -65,6 +65,8 @@ def match_with_exact_nhs_number
     patient.update_from_pds!(pds_patient)
     send_parental_contact_warning_if_needed(patient, @consent_form)
     @consent_form.match_with_patient!(patient, current_user: nil)
+    reset_counts
+    true
   end
 
   def location_patients
@@ -106,5 +108,10 @@ def match_patient(patient)
 
     send_parental_contact_warning_if_needed(patient, @consent_form)
     @consent_form.match_with_patient!(patient, current_user: nil)
+    reset_counts
+  end
+
+  def reset_counts
+    TeamCachedCounts.new(@consent_form.team).reset_unmatched_consent_responses!
   end
 end