Remove duplication in SchoolMove

This removes duplicate calls to `PatientTeamUpdater` and
`PatientStatusUpdater` in the `SchoolMove#confirm!` method.

Previously we were calling these in both the `update_locations!`, and
`update_patient_statuses!` and `update_patient_teams!` methods.

I expect that removing the duplication should introduce a small
performance improvement on school moves.

Author: thomasleese

app/models/school_move.rb

@@ -79,7 +79,6 @@ def assign_from(school:, home_educated:, team:)
 
   def confirm!(user: nil)
     ActiveRecord::Base.transaction do
-      # We need to set this before updating patient locations
       move_across_teams = from_another_team?
 
       update_patient!
@@ -90,6 +89,7 @@ def confirm!(user: nil)
 
       update_patient_teams!
       update_patient_statuses!
+
       if move_across_teams
         create_important_notice!(log_entry)
         update_important_notices!
@@ -191,9 +191,6 @@ def update_locations!
         columns: %i[date_range]
       }
     )
-
-    PatientTeamUpdater.call(patient:)
-    PatientStatusUpdater.call(patient:)
   end
 
   def create_log_entry!(user:)