6bdf6f03b4
The unique index doesn't work when the port is `NULL`. So use `-1` instead for when using the default ports (80/443), as if we would use the real ports, we could still have both 80 and 443 in the database at the same time.
36 lines
942 B
Ruby
36 lines
942 B
Ruby
# frozen_string_literal: true
|
|
|
|
class CleanupDuplicatePods < ActiveRecord::Migration[6.1]
|
|
class Pod < ApplicationRecord
|
|
end
|
|
|
|
def change
|
|
reversible do |change|
|
|
change.up do
|
|
remove_duplicates
|
|
|
|
Pod.where(port: nil).update_all(port: -1) # rubocop:disable Rails/SkipsModelValidations
|
|
end
|
|
|
|
change.down do
|
|
Pod.where(port: -1).update_all(port: nil) # rubocop:disable Rails/SkipsModelValidations
|
|
end
|
|
end
|
|
|
|
change_column_null :pods, :port, false
|
|
end
|
|
|
|
private
|
|
|
|
def remove_duplicates
|
|
Pod.where(port: nil).group(:host).having("count(*) > 1").pluck(:host).each do |host|
|
|
duplicate_ids = Pod.where(host: host).order(:id).ids
|
|
target_pod_id = duplicate_ids.shift
|
|
|
|
duplicate_ids.each do |pod_id|
|
|
Person.where(pod_id: pod_id).update_all(pod_id: target_pod_id) # rubocop:disable Rails/SkipsModelValidations
|
|
Pod.delete(pod_id)
|
|
end
|
|
end
|
|
end
|
|
end
|