From 2b8ba97252fa6db92458941cb12c05629200dada Mon Sep 17 00:00:00 2001 From: natdeanlewissoftwire Date: Mon, 30 Oct 2023 14:59:53 +0000 Subject: [PATCH] feat: don't add old_id to new location as there is a uniqueness constraint on this attribute --- app/services/merge/merge_organisations_service.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/services/merge/merge_organisations_service.rb b/app/services/merge/merge_organisations_service.rb index c7aa8b97c..d8996f225 100644 --- a/app/services/merge/merge_organisations_service.rb +++ b/app/services/merge/merge_organisations_service.rb @@ -70,7 +70,7 @@ private new_scheme = Scheme.create!(scheme.attributes.except("id", "owning_organisation_id").merge(owning_organisation: @absorbing_organisation)) scheme.locations.each do |location| - new_scheme.locations << Location.new(location.attributes.except("id", "scheme_id")) unless location.deactivated? + new_scheme.locations << Location.new(location.attributes.except("id", "scheme_id", "old_id")) unless location.deactivated? end @merged_schemes[merging_organisation.name] << { name: new_scheme.service_name, code: new_scheme.id } SchemeDeactivationPeriod.create!(scheme:, deactivation_date: @merge_date)