From e63f2a46af44d33257f21c7e1b60219da7ae17e3 Mon Sep 17 00:00:00 2001
From: Nat Dean-Lewis <nat.dean-lewis@softwire.com>
Date: Mon, 7 Sep 2026 12:34:01 +0100
Subject: [PATCH] CLDC-3362: full rake task for clearing confidential address
 data

---
 clear_confidential_address_data.rb            | 42 -----------
 .../clear_confidential_address_data.rake      | 75 +++++++++++++++++++
 2 files changed, 75 insertions(+), 42 deletions(-)
 delete mode 100644 clear_confidential_address_data.rb
 create mode 100644 lib/tasks/clear_confidential_address_data.rake

diff --git a/clear_confidential_address_data.rb b/clear_confidential_address_data.rb
deleted file mode 100644
index c9700f7a6..000000000
--- a/clear_confidential_address_data.rb
+++ /dev/null
@@ -1,42 +0,0 @@
-ADDRESS_FIELDS_TO_REMOVE_FOR_CONFIDENTIAL_SCHEME = %w[
-      uprn
-      uprn_known
-      uprn_confirmed
-      uprn_selection
-      address_line1
-      address_line2
-      town_or_city
-      county
-      postcode_full
-      postcode_known
-      address_line1_input
-      postcode_full_input
-      address_line1_as_entered
-      address_line2_as_entered
-      town_or_city_as_entered
-      county_as_entered
-      postcode_full_as_entered
-      la_as_entered
-
-      is_la_inferred
-      la
-      address_search_value_check
-    ].freeze
-
-
-scope = LettingsLog
-          .joins(:scheme)
-          .where(schemes: { sensitive: true })
-          .where(
-            ADDRESS_FIELDS_TO_REMOVE_FOR_CONFIDENTIAL_SCHEME.map { |field| "#{LettingsLog.table_name}.#{field} IS NOT NULL" }.join(" OR ")
-          )
-
-scope.find_each do |log|
-  log.reset_address_fields!
-  log.uprn_selection = nil
-  log.postcode_known = nil
-
-  log.la = log.la
-
-  log.save!
-end
diff --git a/lib/tasks/clear_confidential_address_data.rake b/lib/tasks/clear_confidential_address_data.rake
new file mode 100644
index 000000000..eff626728
--- /dev/null
+++ b/lib/tasks/clear_confidential_address_data.rake
@@ -0,0 +1,75 @@
+desc "Clears already-collected address/UPRN data for 2026/27 lettings logs in confidential (sensitive) schemes"
+task clear_confidential_address_data: :environment do
+  year = 2026
+
+  fields_present_scope = %w[
+    uprn
+    uprn_known
+    uprn_confirmed
+    uprn_selection
+    address_line1
+    address_line2
+    town_or_city
+    county
+    postcode_full
+    postcode_known
+    address_line1_input
+    postcode_full_input
+    address_line1_as_entered
+    address_line2_as_entered
+    town_or_city_as_entered
+    county_as_entered
+    postcode_full_as_entered
+    la_as_entered
+    address_search_value_check
+    la
+  ]
+
+  scope = LettingsLog
+            .filter_by_year(year)
+            .joins(:scheme)
+            .where(schemes: { sensitive: "Yes" })
+            .where(fields_present_scope.map { |field| "#{LettingsLog.table_name}.#{field} IS NOT NULL" }.join(" OR "))
+
+  puts "Found #{scope.count} confidential-scheme lettings logs in #{year}/#{year + 1} with address data to clear"
+  scope.find_each do |log|
+    puts "log_id=#{log.id} scheme_id=#{log.scheme_id} location_id=#{log.location_id} status=#{log.status} la=#{log.la} postcode_full=#{log.postcode_full}"
+  end
+
+  updated = 0
+
+  scope.find_each do |log|
+    original_status = log.status
+
+    log.uprn = nil
+    log.uprn_known = nil
+    log.uprn_confirmed = nil
+    log.uprn_selection = nil
+    log.address_line1 = nil
+    log.address_line2 = nil
+    log.town_or_city = nil
+    log.county = nil
+    log.postcode_full = nil
+    log.postcode_known = nil
+    log.address_line1_input = nil
+    log.postcode_full_input = nil
+    log.address_line1_as_entered = nil
+    log.address_line2_as_entered = nil
+    log.town_or_city_as_entered = nil
+    log.county_as_entered = nil
+    log.postcode_full_as_entered = nil
+    log.la_as_entered = nil
+    log.address_search_value_check = nil
+    log.la = nil
+    log.is_la_inferred = log.la.present?
+
+    if log.save(validate: false)
+      updated += 1
+      puts "log_id=#{log.id} status changed: #{original_status} -> #{log.status}" if log.status != original_status
+    else
+      Rails.logger.error "CLDC-4462: failed to clear address data for log #{log.id}: #{log.errors.full_messages.join(', ')}"
+    end
+  end
+
+  puts "#{updated} logs updated"
+end