Remove location_id column

This removes the unused column from the `patient_locations` table which
has since been replaced by `school_id`.

Jira-Issue: MAV-7116
Jira-Issue: MAV-7113

Author: thomasleese

app/models/patient_location.rb

@@ -16,7 +16,6 @@
 #
 #  idx_on_patient_id_school_id_academic_year_652216fa07    (patient_id,school_id,academic_year) UNIQUE
 #  idx_on_school_id_academic_year_patient_id_c647e75f26    (school_id,academic_year,patient_id) UNIQUE
-#  index_patient_locations_on_location_id                  (location_id)
 #  index_patient_locations_on_school_id                    (school_id)
 #  index_patient_locations_on_school_id_and_academic_year  (school_id,academic_year)
 #
@@ -27,8 +26,6 @@
 #
 
 class PatientLocation < ApplicationRecord
-  self.ignored_columns = %i[location_id]
-
   audited associated_with: :patient
   has_associated_audits
 

db/migrate/20260428064915_remove_location_from_patient_locations.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class RemoveLocationFromPatientLocations < ActiveRecord::Migration[8.1]
+  def change
+    remove_column :patient_locations, :location_id, :integer
+  end
+end

db/schema.rb

@@ -650,11 +650,9 @@
     t.integer "academic_year", null: false
     t.datetime "created_at", null: false
     t.daterange "date_range", default: -::Float::INFINITY...::Float::INFINITY, null: false
-    t.bigint "location_id"
     t.bigint "patient_id", null: false
     t.bigint "school_id", null: false
     t.datetime "updated_at", null: false
-    t.index ["location_id"], name: "index_patient_locations_on_location_id"
     t.index ["patient_id", "school_id", "academic_year"], name: "idx_on_patient_id_school_id_academic_year_652216fa07", unique: true
     t.index ["school_id", "academic_year", "patient_id"], name: "idx_on_school_id_academic_year_patient_id_c647e75f26", unique: true
     t.index ["school_id", "academic_year"], name: "index_patient_locations_on_school_id_and_academic_year"

spec/factories/patient_locations.rb

@@ -16,7 +16,6 @@
 #
 #  idx_on_patient_id_school_id_academic_year_652216fa07    (patient_id,school_id,academic_year) UNIQUE
 #  idx_on_school_id_academic_year_patient_id_c647e75f26    (school_id,academic_year,patient_id) UNIQUE
-#  index_patient_locations_on_location_id                  (location_id)
 #  index_patient_locations_on_school_id                    (school_id)
 #  index_patient_locations_on_school_id_and_academic_year  (school_id,academic_year)
 #

spec/models/patient_location_spec.rb

@@ -16,7 +16,6 @@
 #
 #  idx_on_patient_id_school_id_academic_year_652216fa07    (patient_id,school_id,academic_year) UNIQUE
 #  idx_on_school_id_academic_year_patient_id_c647e75f26    (school_id,academic_year,patient_id) UNIQUE
-#  index_patient_locations_on_location_id                  (location_id)
 #  index_patient_locations_on_school_id                    (school_id)
 #  index_patient_locations_on_school_id_and_academic_year  (school_id,academic_year)
 #