Remove unnecessary includes in PatientStatusUpdater

The `patient_locations` association on `Patient::ProgrammeStatus`
already defines the includes that are needed to successfully generate a
status, therefore we don't need to define these again in the
`PatientStatusUpdater`.

This follows the same pattern we use for the other associations (for
example `consents`).

Author: thomasleese

app/lib/patient_status_updater.rb

@@ -46,18 +46,13 @@ def update_programme_statuses!
         batch =
           relation.includes(
             :attendance_record,
+            :consent_notifications,
             :consents,
+            :parents,
             :patient,
             :patient_locations,
             :triages,
-            :vaccination_records,
-            :parents,
-            :consent_notifications,
-            patient_locations: {
-              location: [
-                { team_locations: { sessions: :session_programme_year_groups } }
-              ]
-            }
+            :vaccination_records
           ).to_a
 
         batch.each(&:assign)