From fe4649ca641f3d0115ccd4546abd49938e30098d Mon Sep 17 00:00:00 2001 From: bilal Date: Wed, 17 Jun 2020 09:31:38 +0200 Subject: [PATCH] use person_address_street1 for old person_address --- .../appearance_releases_controller.rb | 2 +- app/models/appearance_release.rb | 24 ++++--------------- app/models/sample_appearance_release.rb | 3 +-- app/views/appearance_releases/_form.html.erb | 2 +- config/locales/en.yml | 1 - config/locales/es.yml | 1 - ..._address_column_in_appearance_releases.rb} | 4 ++-- ...ppearance_release_person_address_column.rb | 5 ---- spec/models/contract_template_preview_spec.rb | 1 - .../appearance_release_log_sheet_spec.rb | 2 +- 10 files changed, 11 insertions(+), 34 deletions(-) rename db/migrate/{20200615140920_add_address_component_fields_to_appearance_releases.rb => 20200615133602_destructure_person_address_column_in_appearance_releases.rb} (68%) delete mode 100644 db/migrate/20200615133602_rename_appearance_release_person_address_column.rb diff --git a/app/controllers/appearance_releases_controller.rb b/app/controllers/appearance_releases_controller.rb index 012dbaf..610eea5 100644 --- a/app/controllers/appearance_releases_controller.rb +++ b/app/controllers/appearance_releases_controller.rb @@ -79,7 +79,7 @@ class AppearanceReleasesController < ApplicationController def appearance_release_params params.require(:appearance_release).permit(:contract, :guardian_address, :guardian_first_name, :guardian_last_name, :guardian_phone, :guardian_photo, :guardian_email, :minor, - :person_address_old, :person_first_name, :person_last_name, :person_phone, :person_email, :person_photo, + :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, :term_id, :term_text, :person_date_of_birth, diff --git a/app/models/appearance_release.rb b/app/models/appearance_release.rb index 387c0de..c1126ff 100644 --- a/app/models/appearance_release.rb +++ b/app/models/appearance_release.rb @@ -20,11 +20,11 @@ class AppearanceRelease < ApplicationRecord class_name: 'Address', mapping: [ %w[person_address_street1 street1], - %w[person_address_street2 street2], - %w[person_address_city city], - %w[person_address_state state], - %w[person_address_zip zip], - %w[person_address_country country] + %w[person_address_street2 street2], + %w[person_address_city city], + %w[person_address_state state], + %w[person_address_zip zip], + %w[person_address_country country] ] # These validations apply to all releases @@ -85,7 +85,6 @@ class AppearanceRelease < ApplicationRecord searchable_on %i[ person_first_name person_last_name - person_address_old person_address_street1 person_address_street2 person_address_city @@ -96,19 +95,6 @@ class AppearanceRelease < ApplicationRecord person_phone ] - def person_address - return self.person_address_old if self.person_address_old.present? - - Address.new( - self.person_address_street1, - self.person_address_street2, - self.person_address_city, - self.person_address_state, - self.person_address_zip, - self.person_address_country - ).to_s(format: :full) - end - # All releases must respond to the following messages def name person_name diff --git a/app/models/sample_appearance_release.rb b/app/models/sample_appearance_release.rb index 5107365..a1856eb 100644 --- a/app/models/sample_appearance_release.rb +++ b/app/models/sample_appearance_release.rb @@ -8,8 +8,7 @@ class SampleAppearanceRelease < AppearanceRelease def default_attrs { - person_address_old: "Street Address, City, State Zipcode", - person_address_street1: "St1", + person_address_street1: "Street Address, City, State Zipcode", person_address_street2: "St2", person_address_city: "City", person_address_state: "State", diff --git a/app/views/appearance_releases/_form.html.erb b/app/views/appearance_releases/_form.html.erb index c3b2260..5e67d9d 100644 --- a/app/views/appearance_releases/_form.html.erb +++ b/app/views/appearance_releases/_form.html.erb @@ -13,7 +13,7 @@
<%= form.email_field :person_email, wrapper_class: "col-sm-6" %> <%= form.date_field :person_date_of_birth, wrapper_class: "col-sm-6", placeholder: Date.current %> - <%= form.text_field :person_address_old, wrapper_class: "col-sm-6" %> + <%= form.text_field :person_address_street1, wrapper_class: "col-sm-6" %>
!appearance_release.minor?) %>" data-ujs-target="guardian-fields"> diff --git a/config/locales/en.yml b/config/locales/en.yml index c606755..fb64905 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -329,7 +329,6 @@ en: person_title: Title appearance_release: minor: Is the person a minor? - person_address_old: Address person_address: Address person_address_city: City person_address_country: Country diff --git a/config/locales/es.yml b/config/locales/es.yml index 44d005d..113dfdd 100644 --- a/config/locales/es.yml +++ b/config/locales/es.yml @@ -125,7 +125,6 @@ es: person_address_street2: Address (Line 2) (ES) minor: El firmante es un menor person_address: Dirección - person_address_old: Dirección person_email: Dirección de correo electrónico person_name: Nómbre person_phone: Número de teléfono diff --git a/db/migrate/20200615140920_add_address_component_fields_to_appearance_releases.rb b/db/migrate/20200615133602_destructure_person_address_column_in_appearance_releases.rb similarity index 68% rename from db/migrate/20200615140920_add_address_component_fields_to_appearance_releases.rb rename to db/migrate/20200615133602_destructure_person_address_column_in_appearance_releases.rb index 00abd0e..ba88cc4 100644 --- a/db/migrate/20200615140920_add_address_component_fields_to_appearance_releases.rb +++ b/db/migrate/20200615133602_destructure_person_address_column_in_appearance_releases.rb @@ -1,6 +1,6 @@ -class AddAddressComponentFieldsToAppearanceReleases < ActiveRecord::Migration[6.0] +class DestructurePersonAddressColumnInAppearanceReleases < ActiveRecord::Migration[6.0] def change - add_column :appearance_releases, :person_address_street1, :string + rename_column :appearance_releases, :person_address, :person_address_street1 add_column :appearance_releases, :person_address_street2, :string add_column :appearance_releases, :person_address_city, :string add_column :appearance_releases, :person_address_state, :string diff --git a/db/migrate/20200615133602_rename_appearance_release_person_address_column.rb b/db/migrate/20200615133602_rename_appearance_release_person_address_column.rb deleted file mode 100644 index da38869..0000000 --- a/db/migrate/20200615133602_rename_appearance_release_person_address_column.rb +++ /dev/null @@ -1,5 +0,0 @@ -class RenameAppearanceReleasePersonAddressColumn < ActiveRecord::Migration[6.0] - def change - rename_column :appearance_releases, :person_address, :person_address_old - end -end diff --git a/spec/models/contract_template_preview_spec.rb b/spec/models/contract_template_preview_spec.rb index 33efb63..c191e12 100644 --- a/spec/models/contract_template_preview_spec.rb +++ b/spec/models/contract_template_preview_spec.rb @@ -50,7 +50,6 @@ describe ContractTemplatePreview do 'id' => nil, 'person_first_name' => 'Dummy', 'person_last_name' => 'Person', - 'person_address_old' => nil, 'person_address_street1' => 'Street 1', 'person_address_street2' => 'Street 2', 'person_address_city' => 'City', diff --git a/spec/models/excel_reports/video_reports/nat_geo_legal_binder_logs/appearance_release_log_sheet_spec.rb b/spec/models/excel_reports/video_reports/nat_geo_legal_binder_logs/appearance_release_log_sheet_spec.rb index 9e397ee..9b1490e 100644 --- a/spec/models/excel_reports/video_reports/nat_geo_legal_binder_logs/appearance_release_log_sheet_spec.rb +++ b/spec/models/excel_reports/video_reports/nat_geo_legal_binder_logs/appearance_release_log_sheet_spec.rb @@ -29,7 +29,7 @@ module ExcelReports restriction: Restriction.last, person_first_name: "John", person_last_name: "Doe", - person_address_old: "123 Main Street, New York, NY 10000") + person_address_street1: "123 Main Street, New York, NY 10000") ) ) allow(sheet).to receive(:add_row)