From 50aac2c5fb1cadf6ddb1e642cc38d93d84d0e332 Mon Sep 17 00:00:00 2001
From: Arthur Campbell <arfa.camble@gmail.com>
Date: Fri, 14 Jun 2024 13:54:25 +0100
Subject: [PATCH] update the mortgage used question displayed answer options

---
 .../form/sales/questions/mortgageused.rb      | 21 +++++++++++--------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/app/models/form/sales/questions/mortgageused.rb b/app/models/form/sales/questions/mortgageused.rb
index ea855083b..8dbe3a650 100644
--- a/app/models/form/sales/questions/mortgageused.rb
+++ b/app/models/form/sales/questions/mortgageused.rb
@@ -10,21 +10,24 @@ class Form::Sales::Questions::Mortgageused < ::Form::Question
     @question_number = QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP.fetch(form.start_date.year, QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP.max_by { |k, _v| k }.last)[ownershipsch]
   end
 
+  def displayed_answer_options(log, _user = nil)
+    if log.outright_sale? && log.saledate && !form.start_year_after_2024?
+      answer_options_without_dont_know
+    elsif log.stairowned == 100 || log.outright_sale?
+      ANSWER_OPTIONS
+    else
+      answer_options_without_dont_know
+    end
+  end
+
   ANSWER_OPTIONS = {
     "1" => { "value" => "Yes" },
     "2" => { "value" => "No" },
     "3" => { "value" => "Don’t know" },
   }.freeze
 
-  def displayed_answer_options(log, _user = nil)
-    if log.stairowned == 100 || @ownershipsch == 3
-      ANSWER_OPTIONS
-    else
-      {
-        "1" => { "value" => "Yes" },
-        "2" => { "value" => "No" },
-      }
-    end
+  def answer_options_without_dont_know
+    ANSWER_OPTIONS.reject { |key, _v| key == "3" }
   end
 
   QUESTION_NUMBER_FROM_YEAR_AND_OWNERSHIP = {