From f2b6bf8351546da9be69fcd4208693a5cbb13741 Mon Sep 17 00:00:00 2001 From: bilal Date: Wed, 17 Jun 2020 09:44:34 +0200 Subject: [PATCH] use guardian_address_street1 for old guardian_address --- app/controllers/appearance_releases_controller.rb | 2 +- app/models/appearance_release.rb | 13 ------------- app/views/appearance_releases/_form.html.erb | 2 +- config/locales/en.yml | 1 - config/locales/es.yml | 1 - ...ardian_address_column_in_appearance_releases.rb} | 5 ++--- ...me_appearance_release_guardian_address_column.rb | 5 ----- .../public/appearance_releases_controller_spec.rb | 1 - spec/factories/appearance_releases.rb | 2 +- spec/models/contract_template_preview_spec.rb | 1 - 10 files changed, 5 insertions(+), 28 deletions(-) rename db/migrate/{20200616124401_add_guardian_address_components_to_appearance_releases.rb => 20200616124214_destructure_guardian_address_column_in_appearance_releases.rb} (68%) delete mode 100644 db/migrate/20200616124214_rename_appearance_release_guardian_address_column.rb diff --git a/app/controllers/appearance_releases_controller.rb b/app/controllers/appearance_releases_controller.rb index 2035e00..f22aa39 100644 --- a/app/controllers/appearance_releases_controller.rb +++ b/app/controllers/appearance_releases_controller.rb @@ -78,7 +78,7 @@ class AppearanceReleasesController < ApplicationController end def appearance_release_params - params.require(:appearance_release).permit(:contract, :guardian_address_old, :guardian_first_name, :guardian_last_name, :guardian_phone, :guardian_photo, :guardian_email, :minor, + params.require(:appearance_release).permit(:contract, :guardian_address_street1, :guardian_first_name, :guardian_last_name, :guardian_phone, :guardian_photo, :guardian_email, :minor, :person_address, :person_first_name, :person_last_name, :person_phone, :person_email, :person_photo, :applicable_medium_id, :applicable_medium_text, :territory_id, :territory_text, diff --git a/app/models/appearance_release.rb b/app/models/appearance_release.rb index a3441c3..5fd1bc5 100644 --- a/app/models/appearance_release.rb +++ b/app/models/appearance_release.rb @@ -85,19 +85,6 @@ class AppearanceRelease < ApplicationRecord searchable_on %i[person_first_name person_last_name person_address person_email person_phone] - def guardian_address - return self.guardian_address_old if self.guardian_address_old.present? - - Address.new( - self.guardian_address_street1, - self.guardian_address_street2, - self.guardian_address_city, - self.guardian_address_state, - self.guardian_address_zip, - self.guardian_address_country - ).to_s(format: :full) - end - # All releases must respond to the following messages def name person_name diff --git a/app/views/appearance_releases/_form.html.erb b/app/views/appearance_releases/_form.html.erb index e8a0457..9541d53 100644 --- a/app/views/appearance_releases/_form.html.erb +++ b/app/views/appearance_releases/_form.html.erb @@ -23,7 +23,7 @@ <%= form.phone_field :guardian_phone, wrapper_class: "col-sm-6" %>
- <%= form.text_field :guardian_address_old, wrapper_class: "col-sm-6" %> + <%= form.text_field :guardian_address_street1, wrapper_class: "col-sm-6" %>
<% end %> diff --git a/config/locales/en.yml b/config/locales/en.yml index d716704..2b876c5 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -336,7 +336,6 @@ en: person_last_name: Last name person_name: Name person_phone: Phone number - guardian_address_old: Guardian address guardian_address_city: Guardian city guardian_address_country: Guardian country guardian_address_state: Guardian state diff --git a/config/locales/es.yml b/config/locales/es.yml index d1467a8..84601e9 100644 --- a/config/locales/es.yml +++ b/config/locales/es.yml @@ -116,7 +116,6 @@ es: label: appearance_release: guardian_address: Dirección del tutor legal - guardian_address_old: Dirección del tutor legal guardian_name: Nómbre del tutor legal guardian_phone: Número de teléfono del tutor legal minor: El firmante es un menor diff --git a/db/migrate/20200616124401_add_guardian_address_components_to_appearance_releases.rb b/db/migrate/20200616124214_destructure_guardian_address_column_in_appearance_releases.rb similarity index 68% rename from db/migrate/20200616124401_add_guardian_address_components_to_appearance_releases.rb rename to db/migrate/20200616124214_destructure_guardian_address_column_in_appearance_releases.rb index 4e9a814..9cb94da 100644 --- a/db/migrate/20200616124401_add_guardian_address_components_to_appearance_releases.rb +++ b/db/migrate/20200616124214_destructure_guardian_address_column_in_appearance_releases.rb @@ -1,11 +1,10 @@ -class AddGuardianAddressComponentsToAppearanceReleases < ActiveRecord::Migration[6.0] +class DestructureGuardianAddressColumnInAppearanceReleases < ActiveRecord::Migration[6.0] def change - add_column :appearance_releases, :guardian_address_street1, :string + rename_column :appearance_releases, :guardian_address, :guardian_address_street1 add_column :appearance_releases, :guardian_address_street2, :string add_column :appearance_releases, :guardian_address_city, :string add_column :appearance_releases, :guardian_address_state, :string add_column :appearance_releases, :guardian_address_zip, :string add_column :appearance_releases, :guardian_address_country, :string - end end diff --git a/db/migrate/20200616124214_rename_appearance_release_guardian_address_column.rb b/db/migrate/20200616124214_rename_appearance_release_guardian_address_column.rb deleted file mode 100644 index 4d14700..0000000 --- a/db/migrate/20200616124214_rename_appearance_release_guardian_address_column.rb +++ /dev/null @@ -1,5 +0,0 @@ -class RenameAppearanceReleaseGuardianAddressColumn < ActiveRecord::Migration[6.0] - def change - rename_column :appearance_releases, :guardian_address, :guardian_address_old - end -end diff --git a/spec/controllers/public/appearance_releases_controller_spec.rb b/spec/controllers/public/appearance_releases_controller_spec.rb index a0f9a7e..b41237c 100644 --- a/spec/controllers/public/appearance_releases_controller_spec.rb +++ b/spec/controllers/public/appearance_releases_controller_spec.rb @@ -121,7 +121,6 @@ describe Public::AppearanceReleasesController do def minor_appearance_release_params(with_guardian_photo = true) minor_type = with_guardian_photo ? :minor_with_guardian_photo : :minor attributes_for(:appearance_release, minor_type) - .except(:guardian_address_old) .merge(signature_param, guardian_address_params) end diff --git a/spec/factories/appearance_releases.rb b/spec/factories/appearance_releases.rb index ffaffb8..683a935 100644 --- a/spec/factories/appearance_releases.rb +++ b/spec/factories/appearance_releases.rb @@ -53,7 +53,7 @@ FactoryBot.define do # guardian_address_state "State" # guardian_address_zip "ZIP" # guardian_address_country "Country" - guardian_address_old "St1 St2 City State ZIP Country" + guardian_address_street1 "St1 St2 City State ZIP Country" guardian_photo do path = Rails.root.join("spec", "fixtures", "files", "pratt.jpg") Rack::Test::UploadedFile.new(path, "image/jpeg") diff --git a/spec/models/contract_template_preview_spec.rb b/spec/models/contract_template_preview_spec.rb index 472bcd2..d026070 100644 --- a/spec/models/contract_template_preview_spec.rb +++ b/spec/models/contract_template_preview_spec.rb @@ -54,7 +54,6 @@ describe ContractTemplatePreview do 'person_phone' => '00 111 222 333 4444', 'updated_at' => nil, 'minor' => true, - 'guardian_address_old' => nil, 'guardian_address_street1' => 'Street 3', 'guardian_address_street2' => 'Street 4', 'guardian_address_city' => 'City-2',