From ba0efd72333fa0679cc2ce55b3aeb90dfb1b6290 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 bda5a34..a1e53d6 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_street1, :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 ab6ab60..078eefd 100644 --- a/app/models/appearance_release.rb +++ b/app/models/appearance_release.rb @@ -107,19 +107,6 @@ class AppearanceRelease < ApplicationRecord 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 2244ca7..69e521e 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 d0867db..c3bb2de 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -341,7 +341,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 78e706a..dde1aac 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 person_address_city: City (ES) 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 cafb534..7b2ecf2 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 db35af2..7e8148c 100644 --- a/spec/factories/appearance_releases.rb +++ b/spec/factories/appearance_releases.rb @@ -58,7 +58,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 ffc86dd..dee4dea 100644 --- a/spec/models/contract_template_preview_spec.rb +++ b/spec/models/contract_template_preview_spec.rb @@ -58,7 +58,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',