Browse Source

More tweaks

pull/2252/head
Kat 2 years ago
parent
commit
778608a2e3
  1. 4
      app/models/form/sales/pages/address.rb
  2. 2
      app/models/form/sales/questions/address_line1.rb
  3. 2
      app/models/form/sales/questions/buyer_previous.rb
  4. 2
      app/models/form/sales/questions/county.rb
  5. 2
      app/models/form/sales/questions/deposit_amount.rb
  6. 2
      app/models/form/sales/questions/deposit_discount.rb
  7. 2
      app/models/form/sales/questions/discount.rb
  8. 2
      app/models/form/sales/questions/equity.rb
  9. 2
      app/models/form/sales/questions/extra_borrowing.rb
  10. 2
      app/models/form/sales/questions/fromprop.rb
  11. 2
      app/models/form/sales/questions/grant.rb
  12. 4
      app/models/form/sales/questions/has_leasehold_charges.rb
  13. 2
      app/models/form/sales/questions/leasehold_charges.rb
  14. 2
      app/models/form/sales/questions/living_before_purchase.rb
  15. 2
      app/models/form/sales/questions/monthly_rent.rb
  16. 2
      app/models/form/sales/questions/mortgage_amount.rb
  17. 2
      app/models/form/sales/questions/mortgage_lender.rb
  18. 2
      app/models/form/sales/questions/mortgage_length.rb
  19. 2
      app/models/form/sales/questions/mortgageused.rb
  20. 11
      app/models/form/sales/questions/nationality_all.rb
  21. 10
      app/models/form/sales/questions/nationality_all_group.rb
  22. 1
      app/models/form/sales/questions/person_age.rb
  23. 1
      app/models/form/sales/questions/person_age_known.rb
  24. 1
      app/models/form/sales/questions/person_gender_identity.rb
  25. 1
      app/models/form/sales/questions/person_relationship_to_buyer_1.rb
  26. 1
      app/models/form/sales/questions/person_working_situation.rb
  27. 2
      app/models/form/sales/questions/postcode_for_full_address.rb
  28. 2
      app/models/form/sales/questions/previous_bedrooms.rb
  29. 2
      app/models/form/sales/questions/previous_tenure.rb
  30. 2
      app/models/form/sales/questions/property_building_type.rb
  31. 2
      app/models/form/sales/questions/property_local_authority.rb
  32. 2
      app/models/form/sales/questions/property_local_authority_known.rb
  33. 2
      app/models/form/sales/questions/property_number_of_bedrooms.rb
  34. 2
      app/models/form/sales/questions/property_unit_type.rb
  35. 2
      app/models/form/sales/questions/purchase_price.rb
  36. 2
      app/models/form/sales/questions/town_or_city.rb
  37. 2
      app/models/form/sales/questions/uprn.rb
  38. 3
      app/models/form/sales/questions/uprn_known.rb
  39. 2
      app/models/form/sales/questions/value.rb
  40. 2
      spec/models/form/sales/pages/person_age_spec.rb
  41. 2
      spec/models/form/sales/pages/person_gender_identity_spec.rb
  42. 2
      spec/models/form/sales/pages/person_relationship_to_buyer1_spec.rb
  43. 2
      spec/models/form/sales/pages/person_working_situation_spec.rb
  44. 2
      spec/models/form/sales/questions/nationality_all_group_spec.rb
  45. 2
      spec/models/form/sales/questions/nationality_all_spec.rb
  46. 2
      spec/models/form/sales/questions/person_age_known_spec.rb
  47. 2
      spec/models/form/sales/questions/person_age_spec.rb
  48. 2
      spec/models/form/sales/questions/person_gender_identity_spec.rb
  49. 2
      spec/models/form/sales/questions/person_relationship_to_buyer1_spec.rb
  50. 2
      spec/models/form/sales/questions/person_working_situation_spec.rb
  51. 2
      spec/models/form/sales/questions/uprn_known_spec.rb

4
app/models/form/sales/pages/address.rb

@ -2,7 +2,7 @@ class Form::Sales::Pages::Address < ::Form::Page
def initialize(id, hsh, subsection) def initialize(id, hsh, subsection)
super super
@id = "address" @id = "address"
@header = "Q15 - What is the property's address?" @header = "Q#{QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]} - What is the property's address?"
@depends_on = [ @depends_on = [
{ "uprn_known" => nil }, { "uprn_known" => nil },
{ "uprn_known" => 0 }, { "uprn_known" => 0 },
@ -19,4 +19,6 @@ class Form::Sales::Pages::Address < ::Form::Page
Form::Sales::Questions::PostcodeForFullAddress.new(nil, nil, self), Form::Sales::Questions::PostcodeForFullAddress.new(nil, nil, self),
] ]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 16 }.freeze
end end

2
app/models/form/sales/questions/address_line1.rb

@ -19,5 +19,5 @@ class Form::Sales::Questions::AddressLine1 < ::Form::Question
].select(&:present?).join("\n") ].select(&:present?).join("\n")
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 19 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 16 }.freeze
end end

2
app/models/form/sales/questions/buyer_previous.rb

@ -26,5 +26,5 @@ class Form::Sales::Questions::BuyerPrevious < ::Form::Question
form.start_year_after_2024? form.start_year_after_2024?
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 84, 2024 => 86 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 84 }.freeze
end end

2
app/models/form/sales/questions/county.rb

@ -11,5 +11,5 @@ class Form::Sales::Questions::County < ::Form::Question
@hide_question_number_on_page = true @hide_question_number_on_page = true
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 19 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 16 }.freeze
end end

2
app/models/form/sales/questions/deposit_amount.rb

@ -22,7 +22,7 @@ class Form::Sales::Questions::DepositAmount < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 95, 2 => 108, 3 => 116 }, 2023 => { 1 => 95, 2 => 108, 3 => 116 },
2024 => { 1 => 97, 2 => 110, 3 => 117 }, 2024 => { 1 => 96, 2 => 109, 3 => 116 },
}.freeze }.freeze
def hint_text def hint_text

2
app/models/form/sales/questions/deposit_discount.rb

@ -14,5 +14,5 @@ class Form::Sales::Questions::DepositDiscount < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 96, 2024 => 98 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 96, 2024 => 97 }.freeze
end end

2
app/models/form/sales/questions/discount.rb

@ -16,5 +16,5 @@ class Form::Sales::Questions::Discount < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 102, 2024 => 104 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 102, 2024 => 103 }.freeze
end end

2
app/models/form/sales/questions/equity.rb

@ -14,5 +14,5 @@ class Form::Sales::Questions::Equity < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 89, 2024 => 91 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 89, 2024 => 90 }.freeze
end end

2
app/models/form/sales/questions/extra_borrowing.rb

@ -20,6 +20,6 @@ class Form::Sales::Questions::ExtraBorrowing < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 94, 2 => 107, 3 => 115 }, 2023 => { 1 => 94, 2 => 107, 3 => 115 },
2024 => { 1 => 96, 2 => 109, 3 => 116 }, 2024 => { 1 => 95, 2 => 108, 3 => 115 },
}.freeze }.freeze
end end

2
app/models/form/sales/questions/fromprop.rb

@ -30,5 +30,5 @@ class Form::Sales::Questions::Fromprop < ::Form::Question
} }
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 86, 2024 => 88 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 86, 2024 => 87 }.freeze
end end

2
app/models/form/sales/questions/grant.rb

@ -14,5 +14,5 @@ class Form::Sales::Questions::Grant < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 101, 2024 => 103 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 101, 2024 => 102 }.freeze
end end

4
app/models/form/sales/questions/has_leasehold_charges.rb

@ -29,9 +29,9 @@ class Form::Sales::Questions::HasLeaseholdCharges < ::Form::Question
def question_number def question_number
case @ownershipsch case @ownershipsch
when 1 when 1
98 99
when 2 when 2
109 110
when 3 when 3
117 117
end end

2
app/models/form/sales/questions/leasehold_charges.rb

@ -15,6 +15,6 @@ class Form::Sales::Questions::LeaseholdCharges < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 98, 2 => 109, 3 => 117 }, 2023 => { 1 => 98, 2 => 109, 3 => 117 },
2024 => { 1 => 100, 2 => 111, 3 => 118 }, 2024 => { 1 => 99, 2 => 110, 3 => 117 },
}.freeze }.freeze
end end

2
app/models/form/sales/questions/living_before_purchase.rb

@ -28,6 +28,6 @@ class Form::Sales::Questions::LivingBeforePurchase < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 75, 2 => 99 }, 2023 => { 1 => 75, 2 => 99 },
2024 => { 1 => 77, 2 => 101 }, 2024 => { 1 => 77, 2 => 100 },
}.freeze }.freeze
end end

2
app/models/form/sales/questions/monthly_rent.rb

@ -13,5 +13,5 @@ class Form::Sales::Questions::MonthlyRent < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 97, 2024 => 99 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 97, 2024 => 98 }.freeze
end end

2
app/models/form/sales/questions/mortgage_amount.rb

@ -16,6 +16,6 @@ class Form::Sales::Questions::MortgageAmount < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 91, 2 => 104, 3 => 112 }, 2023 => { 1 => 91, 2 => 104, 3 => 112 },
2024 => { 1 => 93, 2 => 106, 3 => 114 }, 2024 => { 1 => 92, 2 => 105, 3 => 113 },
}.freeze }.freeze
end end

2
app/models/form/sales/questions/mortgage_lender.rb

@ -75,6 +75,6 @@ class Form::Sales::Questions::MortgageLender < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 92, 2 => 105, 3 => 113 }, 2023 => { 1 => 92, 2 => 105, 3 => 113 },
2024 => { 1 => 94, 2 => 107 }, 2024 => { 1 => 93, 2 => 106 },
}.freeze }.freeze
end end

2
app/models/form/sales/questions/mortgage_length.rb

@ -20,6 +20,6 @@ class Form::Sales::Questions::MortgageLength < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 93, 2 => 106, 3 => 114 }, 2023 => { 1 => 93, 2 => 106, 3 => 114 },
2024 => { 1 => 95, 2 => 108, 3 => 115 }, 2024 => { 1 => 94, 2 => 107, 3 => 114 },
}.freeze }.freeze
end end

2
app/models/form/sales/questions/mortgageused.rb

@ -29,6 +29,6 @@ class Form::Sales::Questions::Mortgageused < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 1 => 90, 2 => 103, 3 => 111 }, 2023 => { 1 => 90, 2 => 103, 3 => 111 },
2024 => { 1 => 92, 2 => 105, 3 => 113 }, 2024 => { 1 => 91, 2 => 104, 3 => 112 },
}.freeze }.freeze
end end

11
app/models/form/sales/questions/nationality_all.rb

@ -6,10 +6,19 @@ class Form::Sales::Questions::NationalityAll < ::Form::Question
@type = "select" @type = "select"
@answer_options = GlobalConstants::COUNTRIES_ANSWER_OPTIONS @answer_options = GlobalConstants::COUNTRIES_ANSWER_OPTIONS
@check_answers_card_number = buyer_index @check_answers_card_number = buyer_index
@question_number = buyer_index == 1 ? 24 : 32 @buyer_index = buyer_index
@question_number = question_number
end end
def answer_label(log, _current_user = nil) def answer_label(log, _current_user = nil)
answer_options[log.send(id).to_s]["name"] answer_options[log.send(id).to_s]["name"]
end end
def question_number
if form.start_date.year == 2023
@buyer_index == 1 ? 24 : 32
else
@buyer_index == 1 ? 26 : 34
end
end
end end

10
app/models/form/sales/questions/nationality_all_group.rb

@ -7,10 +7,10 @@ class Form::Sales::Questions::NationalityAllGroup < ::Form::Question
@hint_text = buyer_index == 1 ? "Buyer 1 is the person in the household who does the most paid work. If it’s a joint purchase and the buyers do the same amount of paid work, buyer 1 is whoever is the oldest." : "" @hint_text = buyer_index == 1 ? "Buyer 1 is the person in the household who does the most paid work. If it’s a joint purchase and the buyers do the same amount of paid work, buyer 1 is whoever is the oldest." : ""
@answer_options = ANSWER_OPTIONS @answer_options = ANSWER_OPTIONS
@check_answers_card_number = buyer_index @check_answers_card_number = buyer_index
@question_number = buyer_index == 1 ? 24 : 32
@conditional_for = buyer_index == 1 ? { "nationality_all" => [12] } : { "nationality_all_buyer2" => [12] } @conditional_for = buyer_index == 1 ? { "nationality_all" => [12] } : { "nationality_all_buyer2" => [12] }
@hidden_in_check_answers = { "depends_on" => [{ id => 12 }] } @hidden_in_check_answers = { "depends_on" => [{ id => 12 }] }
@buyer_index = buyer_index @buyer_index = buyer_index
@question_number = question_number
end end
ANSWER_OPTIONS = { ANSWER_OPTIONS = {
@ -26,4 +26,12 @@ class Form::Sales::Questions::NationalityAllGroup < ::Form::Question
"If buyer 2 is a dual national of the United Kingdom and another country, enter United Kingdom. If they are a dual national of two other countries, the buyer should decide which country to enter." "If buyer 2 is a dual national of the United Kingdom and another country, enter United Kingdom. If they are a dual national of two other countries, the buyer should decide which country to enter."
end end
end end
def question_number
if form.start_date.year == 2023
@buyer_index == 1 ? 24 : 32
else
@buyer_index == 1 ? 26 : 34
end
end
end end

1
app/models/form/sales/questions/person_age.rb

@ -14,6 +14,7 @@ class Form::Sales::Questions::PersonAge < ::Form::Question
@max = 110 @max = 110
@step = 1 @step = 1
@person_index = person_index @person_index = person_index
@question_number = question_number
end end
def question_number def question_number

1
app/models/form/sales/questions/person_age_known.rb

@ -20,6 +20,7 @@ class Form::Sales::Questions::PersonAgeKnown < ::Form::Question
} }
@check_answers_card_number = person_index @check_answers_card_number = person_index
@person_index = person_index @person_index = person_index
@question_number = question_number
end end
ANSWER_OPTIONS = { ANSWER_OPTIONS = {

1
app/models/form/sales/questions/person_gender_identity.rb

@ -13,6 +13,7 @@ class Form::Sales::Questions::PersonGenderIdentity < ::Form::Question
"value" => "Prefers not to say", "value" => "Prefers not to say",
}] }]
@person_index = person_index @person_index = person_index
@question_number = question_number
end end
ANSWER_OPTIONS = { ANSWER_OPTIONS = {

1
app/models/form/sales/questions/person_relationship_to_buyer_1.rb

@ -13,6 +13,7 @@ class Form::Sales::Questions::PersonRelationshipToBuyer1 < ::Form::Question
"value" => "Prefers not to say", "value" => "Prefers not to say",
}] }]
@person_index = person_index @person_index = person_index
@question_number = question_number
end end
ANSWER_OPTIONS = { ANSWER_OPTIONS = {

1
app/models/form/sales/questions/person_working_situation.rb

@ -13,6 +13,7 @@ class Form::Sales::Questions::PersonWorkingSituation < ::Form::Question
"value" => "Prefers not to say", "value" => "Prefers not to say",
}] }]
@person_index = person_index @person_index = person_index
@question_number = question_number
end end
ANSWER_OPTIONS = { ANSWER_OPTIONS = {

2
app/models/form/sales/questions/postcode_for_full_address.rb

@ -23,5 +23,5 @@ class Form::Sales::Questions::PostcodeForFullAddress < ::Form::Question
@hide_question_number_on_page = true @hide_question_number_on_page = true
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 19 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 16 }.freeze
end end

2
app/models/form/sales/questions/previous_bedrooms.rb

@ -13,5 +13,5 @@ class Form::Sales::Questions::PreviousBedrooms < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 85, 2024 => 87 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 85, 2024 => 86 }.freeze
end end

2
app/models/form/sales/questions/previous_tenure.rb

@ -19,5 +19,5 @@ class Form::Sales::Questions::PreviousTenure < ::Form::Question
"10" => { "value" => "Don’t know" }, "10" => { "value" => "Don’t know" },
}.freeze }.freeze
QUESTION_NUMBER_FROM_YEAR = { 2023 => 87, 2024 => 89 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 87, 2024 => 88 }.freeze
end end

2
app/models/form/sales/questions/property_building_type.rb

@ -14,5 +14,5 @@ class Form::Sales::Questions::PropertyBuildingType < ::Form::Question
"2" => { "value" => "Converted from previous residential or non-residential property" }, "2" => { "value" => "Converted from previous residential or non-residential property" },
}.freeze }.freeze
QUESTION_NUMBER_FROM_YEAR = { 2023 => 13, 2024 => 17 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 13, 2024 => 20 }.freeze
end end

2
app/models/form/sales/questions/property_local_authority.rb

@ -17,5 +17,5 @@ class Form::Sales::Questions::PropertyLocalAuthority < ::Form::Question
log.saledate && log.saledate.year >= 2023 && log.is_la_inferred? log.saledate && log.saledate.year >= 2023 && log.is_la_inferred?
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 16, 2024 => 20 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 16, 2024 => 17 }.freeze
end end

2
app/models/form/sales/questions/property_local_authority_known.rb

@ -22,5 +22,5 @@ class Form::Sales::Questions::PropertyLocalAuthorityKnown < ::Form::Question
"0" => { "value" => "No" }, "0" => { "value" => "No" },
}.freeze }.freeze
QUESTION_NUMBER_FROM_YEAR = { 2023 => 16, 2024 => 20 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 16, 2024 => 17 }.freeze
end end

2
app/models/form/sales/questions/property_number_of_bedrooms.rb

@ -13,5 +13,5 @@ class Form::Sales::Questions::PropertyNumberOfBedrooms < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 11, 2024 => 15 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 11, 2024 => 18 }.freeze
end end

2
app/models/form/sales/questions/property_unit_type.rb

@ -17,5 +17,5 @@ class Form::Sales::Questions::PropertyUnitType < ::Form::Question
"9" => { "value" => "Other" }, "9" => { "value" => "Other" },
}.freeze }.freeze
QUESTION_NUMBER_FROM_YEAR = { 2023 => 12, 2024 => 16 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 12, 2024 => 19 }.freeze
end end

2
app/models/form/sales/questions/purchase_price.rb

@ -16,7 +16,7 @@ class Form::Sales::Questions::PurchasePrice < ::Form::Question
QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = { QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {
2023 => { 2 => 100, 3 => 110 }, 2023 => { 2 => 100, 3 => 110 },
2024 => { 2 => 102, 3 => 112 }, 2024 => { 2 => 101, 3 => 111 },
}.freeze }.freeze
def hint_text def hint_text

2
app/models/form/sales/questions/town_or_city.rb

@ -11,5 +11,5 @@ class Form::Sales::Questions::TownOrCity < ::Form::Question
@hide_question_number_on_page = true @hide_question_number_on_page = true
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 19 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 15, 2024 => 16 }.freeze
end end

2
app/models/form/sales/questions/uprn.rb

@ -36,5 +36,5 @@ class Form::Sales::Questions::Uprn < ::Form::Question
"\n\n#{value.join("\n")}" "\n\n#{value.join("\n")}"
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 14, 2024 => 18 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 14, 2024 => 15 }.freeze
end end

3
app/models/form/sales/questions/uprn_known.rb

@ -22,6 +22,7 @@ class Form::Sales::Questions::UprnKnown < ::Form::Question
{ "uprn_known" => 1 }, { "uprn_known" => 1 },
], ],
} }
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
ANSWER_OPTIONS = { ANSWER_OPTIONS = {
@ -32,4 +33,6 @@ class Form::Sales::Questions::UprnKnown < ::Form::Question
def unanswered_error_message def unanswered_error_message
I18n.t("validations.property.uprn_known.invalid") I18n.t("validations.property.uprn_known.invalid")
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 14, 2024 => 15 }.freeze
end end

2
app/models/form/sales/questions/value.rb

@ -13,5 +13,5 @@ class Form::Sales::Questions::Value < ::Form::Question
@question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max] @question_number = QUESTION_NUMBER_FROM_YEAR[form.start_date.year] || QUESTION_NUMBER_FROM_YEAR[QUESTION_NUMBER_FROM_YEAR.keys.max]
end end
QUESTION_NUMBER_FROM_YEAR = { 2023 => 88, 2024 => 90 }.freeze QUESTION_NUMBER_FROM_YEAR = { 2023 => 88, 2024 => 89 }.freeze
end end

2
spec/models/form/sales/pages/person_age_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Pages::PersonAge, type: :model do
let(:page_id) { "person_1_age" } let(:page_id) { "person_1_age" }
let(:page_definition) { nil } let(:page_definition) { nil }
let(:subsection) { instance_double(Form::Subsection) } let(:subsection) { instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1))) }
let(:person_index) { 1 } let(:person_index) { 1 }
it "has correct subsection" do it "has correct subsection" do

2
spec/models/form/sales/pages/person_gender_identity_spec.rb

@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Pages::PersonGenderIdentity, type: :model do
subject(:page) { described_class.new(page_id, page_definition, subsection, person_index:) } subject(:page) { described_class.new(page_id, page_definition, subsection, person_index:) }
let(:page_definition) { nil } let(:page_definition) { nil }
let(:subsection) { instance_double(Form::Subsection) } let(:subsection) { instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1))) }
let(:person_index) { 1 } let(:person_index) { 1 }
let(:page_id) { "person_2_gender_identity" } let(:page_id) { "person_2_gender_identity" }

2
spec/models/form/sales/pages/person_relationship_to_buyer1_spec.rb

@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Pages::PersonRelationshipToBuyer1, type: :model do
subject(:page) { described_class.new(page_id, page_definition, subsection, person_index:) } subject(:page) { described_class.new(page_id, page_definition, subsection, person_index:) }
let(:page_definition) { nil } let(:page_definition) { nil }
let(:subsection) { instance_double(Form::Subsection) } let(:subsection) { instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1))) }
let(:person_index) { 1 } let(:person_index) { 1 }
let(:page_id) { "person_1_relationship_to_buyer_1" } let(:page_id) { "person_1_relationship_to_buyer_1" }

2
spec/models/form/sales/pages/person_working_situation_spec.rb

@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Pages::PersonWorkingSituation, type: :model do
subject(:page) { described_class.new(page_id, page_definition, subsection, person_index:) } subject(:page) { described_class.new(page_id, page_definition, subsection, person_index:) }
let(:page_definition) { nil } let(:page_definition) { nil }
let(:subsection) { instance_double(Form::Subsection) } let(:subsection) { instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1))) }
let(:person_index) { 2 } let(:person_index) { 2 }
let(:page_id) { "person_2_working_situation" } let(:page_id) { "person_2_working_situation" }

2
spec/models/form/sales/questions/nationality_all_group_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::NationalityAllGroup, type: :model do
let(:buyer_index) { 1 } let(:buyer_index) { 1 }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
it "has correct page" do it "has correct page" do
expect(question.page).to be page expect(question.page).to be page

2
spec/models/form/sales/questions/nationality_all_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::NationalityAll, type: :model do
let(:buyer_index) { 1 } let(:buyer_index) { 1 }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
it "has correct page" do it "has correct page" do
expect(question.page).to be page expect(question.page).to be page

2
spec/models/form/sales/questions/person_age_known_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::PersonAgeKnown, type: :model do
let(:question_id) { "age3_known" } let(:question_id) { "age3_known" }
let(:question_definition) { nil } let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
let(:person_index) { 2 } let(:person_index) { 2 }
it "has correct page" do it "has correct page" do

2
spec/models/form/sales/questions/person_age_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::PersonAge, type: :model do
let(:question_id) { "age3" } let(:question_id) { "age3" }
let(:question_definition) { nil } let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
let(:person_index) { 2 } let(:person_index) { 2 }
it "has correct page" do it "has correct page" do

2
spec/models/form/sales/questions/person_gender_identity_spec.rb

@ -8,7 +8,7 @@ RSpec.describe Form::Sales::Questions::PersonGenderIdentity, type: :model do
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page) }
let(:person_index) { 2 } let(:person_index) { 2 }
let(:subsection) { instance_double(Form::Subsection) } let(:subsection) { instance_double(Form::Subsection) }
let(:form) { instance_double(Form) } let(:form) { instance_double(Form, start_date: Time.zone.local(2023, 4, 1)) }
before do before do
allow(page).to receive(:subsection).and_return(subsection) allow(page).to receive(:subsection).and_return(subsection)

2
spec/models/form/sales/questions/person_relationship_to_buyer1_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::PersonRelationshipToBuyer1, type: :model
let(:question_id) { "relat2" } let(:question_id) { "relat2" }
let(:question_definition) { nil } let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
let(:person_index) { 2 } let(:person_index) { 2 }
it "has correct page" do it "has correct page" do

2
spec/models/form/sales/questions/person_working_situation_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::PersonWorkingSituation, type: :model do
let(:question_id) { "ecstat2" } let(:question_id) { "ecstat2" }
let(:question_definition) { nil } let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
let(:person_index) { 2 } let(:person_index) { 2 }
it "has correct page" do it "has correct page" do

2
spec/models/form/sales/questions/uprn_known_spec.rb

@ -5,7 +5,7 @@ RSpec.describe Form::Sales::Questions::UprnKnown, type: :model do
let(:question_id) { nil } let(:question_id) { nil }
let(:question_definition) { nil } let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page, subsection: instance_double(Form::Subsection, form: instance_double(Form, start_date: Time.zone.local(2023, 4, 1)))) }
it "has correct page" do it "has correct page" do
expect(question.page).to eq(page) expect(question.page).to eq(page)

Loading…
Cancel
Save