Browse Source

CLDC-4462: Remove unnecessary routing change (#3382)

* feat: wip clear confidential address data

* CLDC-3362: full rake task for clearing confidential address data

* CLDC-3362: add rake spec

* feat: route la question when not inferred for supported housing

* CLDC-4462: respond to PR comments

* CLDC-4462: test all fields

* feat: remove routing change as not needed

* feat: remove routing change as not needed
CLDC-4462-clear-confidential-data-follow-up-actions^2
Nat Dean-Lewis 3 days ago committed by GitHub
parent
commit
be4f02be36
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 10
      app/models/derived_variables/lettings_log_variables.rb
  2. 1
      app/models/form/lettings/pages/property_local_authority.rb
  3. 6
      app/models/lettings_log.rb

10
app/models/derived_variables/lettings_log_variables.rb

@ -176,13 +176,9 @@ module DerivedVariables::LettingsLogVariables
if !form.start_year_2026_or_later? && is_supported_housing?
reset_address_fields!
elsif form.start_year_2026_or_later?
if location_changed?
reset_address_fields!
self.la = nil
end
self.is_la_inferred = la.present? if is_supported_housing? && location && self[:la].blank?
elsif form.start_year_2026_or_later? && location_changed?
reset_address_fields!
self.la = nil
end
if scheme_has_confidential_information?

1
app/models/form/lettings/pages/property_local_authority.rb

@ -5,7 +5,6 @@ class Form::Lettings::Pages::PropertyLocalAuthority < ::Form::Page
@depends_on = [
{ "is_la_inferred" => false, "is_general_needs?" => true, "form.start_year_2025_or_later?" => false, "address_search_given?" => true },
{ "is_la_inferred" => false, "is_general_needs?" => true, "form.start_year_2025_or_later?" => true },
{ "is_la_inferred" => false, "is_supported_housing?" => true, "form.start_year_2026_or_later?" => true },
]
end

6
app/models/lettings_log.rb

@ -31,7 +31,7 @@ class LettingsLog < Log
before_validation :process_postcode_changes!, if: :postcode_full_changed?
before_validation :process_previous_postcode_changes!, if: :ppostcode_full_changed?
before_validation :reset_invalidated_dependent_fields!
before_validation :reset_location_fields!, unless: :postcode_known_or_la_derived_from_scheme_location?
before_validation :reset_location_fields!, unless: :postcode_known?
before_validation :reset_previous_location_fields!, unless: :previous_postcode_known?
before_validation :set_derived_fields!
before_validation :process_uprn_change!, if: :should_process_uprn_change?
@ -375,10 +375,6 @@ class LettingsLog < Log
postcode_known == 1
end
def postcode_known_or_la_derived_from_scheme_location?
postcode_known? || (form&.start_year_2026_or_later? && is_supported_housing? && location.present?)
end
def previous_postcode_known?
# 0: Yes
ppcodenk&.zero?

Loading…
Cancel
Save