Gemfile.lock

@@ -433,12 +433,9 @@ GEM
       actionpack (>= 7.0)
       railties (>= 7.0)
     rexml (3.4.4)
-    roo (3.0.0)
-      base64 (~> 0.2)
-      csv (~> 3)
-      logger (~> 1)
+    roo (2.10.1)
       nokogiri (~> 1)
-      rubyzip (>= 3.0.0, < 4.0.0)
+      rubyzip (>= 1.3.0, < 3.0.0)
     rotp (6.3.0)
     rspec-core (3.13.0)
       rspec-support (~> 3.13.0)
@@ -502,7 +499,7 @@ GEM
       faraday-multipart (>= 1)
     ruby-progressbar (1.13.0)
     ruby2_keywords (0.0.5)
-    rubyzip (3.6.0)
+    rubyzip (2.3.2)
     securerandom (0.4.1)
     selenium-webdriver (4.43.0)
       base64 (~> 0.2)

app/models/derived_variables/lettings_log_variables.rb

@@ -176,11 +176,15 @@ module DerivedVariables::LettingsLogVariables
 
     if !form.start_year_2026_or_later? && is_supported_housing?
       reset_address_fields!
-    elsif form.start_year_2026_or_later? && location_changed?
+    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?
+    end
+
     if scheme_has_confidential_information?
       reset_address_fields!
       self.uprn_selection = nil

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

@@ -5,6 +5,7 @@ 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
 

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?
+  before_validation :reset_location_fields!, unless: :postcode_known_or_la_derived_from_scheme_location?
   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,6 +375,10 @@ 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?

app/services/exports/xml_export_service.rb

@@ -48,7 +48,7 @@ module Exports
       @logger.info("Creating #{archive} - #{initial_count} resources")
       return {} if initial_count.zero?
 
-      zip_file = Zip::File.open_buffer(StringIO.new, create: true)
+      zip_file = Zip::File.open_buffer(StringIO.new)
 
       part_number = 1
       last_processed_marker = nil

lib/tasks/clear_confidential_address_data.rake

@@ -41,7 +41,26 @@ task clear_confidential_address_data: :environment do
   scope.find_each do |log|
     original_status = log.status
 
-    fields_present_scope.each { |field| log[field] = nil }
+    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)

spec/lib/tasks/clear_confidential_address_data_spec.rb

@@ -16,31 +16,6 @@ RSpec.describe "clear_confidential_address_data" do
     let(:non_confidential_scheme) { create(:scheme, sensitive: "No", owning_organisation:) }
     let(:location) { create(:location, scheme:) }
 
-    let(:cleared_fields) do
-      %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
-      ]
-    end
-
     def create_log_with_address_data(scheme:, location:, startdate:)
       log = create(
         :lettings_log,
@@ -57,24 +32,9 @@ RSpec.describe "clear_confidential_address_data" do
       # address/UPRN data collected before the confidential address feature existed.
       log.update_columns(
         uprn: "123456789012",
-        uprn_known: 1,
-        uprn_confirmed: 1,
-        uprn_selection: "123456789012",
         address_line1: "1 Secret Street",
-        address_line2: "Flat 2",
         town_or_city: "Secretville",
-        county: "Secretshire",
-        postcode_full: "SW1A 1AA",
         postcode_known: 1,
-        address_line1_input: "1 Secret Street",
-        postcode_full_input: "SW1A1AA",
-        address_line1_as_entered: "1 Secret Street",
-        address_line2_as_entered: "Flat 2",
-        town_or_city_as_entered: "Secretville",
-        county_as_entered: "Secretshire",
-        postcode_full_as_entered: "SW1A 1AA",
-        la_as_entered: "E09000003",
-        address_search_value_check: 1,
         la: "E09000003",
       )
       log
@@ -93,15 +53,6 @@ RSpec.describe "clear_confidential_address_data" do
         expect(log.postcode_known).to be_nil
       end
 
-      it "clears every field the task targets" do
-        task.invoke
-        log.reload
-
-        cleared_fields.each do |field|
-          expect(log[field]).to be_nil, "expected #{field} to be nil but was #{log[field].inspect}"
-        end
-      end
-
       it "keeps the log's status unchanged when the location's LA can be inferred" do
         task.invoke
         expect(log.reload.status).to eq("completed")
@@ -115,20 +66,6 @@ RSpec.describe "clear_confidential_address_data" do
         expect(log[:la]).to be_nil
         expect(log.is_la_inferred).to be true
       end
-
-      it "does not affect the scheme or location associations" do
-        task.invoke
-        log.reload
-
-        expect(log.scheme_id).to eq(scheme.id)
-        expect(log.location_id).to eq(location.id)
-      end
-
-      it "is idempotent" do
-        task.invoke
-        task.reenable
-        expect { task.invoke }.not_to(change { log.reload.updated_at })
-      end
     end
 
     context "when the scheme's location has no resolvable local authority" do
@@ -151,12 +88,7 @@ RSpec.describe "clear_confidential_address_data" do
 
       it "does not clear the address fields" do
         task.invoke
-        log.reload
-
-        expect(log.address_line1).to eq("1 Secret Street")
-        expect(log.town_or_city).to eq("Secretville")
-        expect(log.uprn).to eq("123456789012")
-        expect(log.postcode_known).to eq(1)
+        expect(log.reload.address_line1).to eq("1 Secret Street")
       end
     end
 
@@ -165,12 +97,7 @@ RSpec.describe "clear_confidential_address_data" do
 
       it "does not clear the address fields" do
         task.invoke
-        log.reload
-
-        expect(log.address_line1).to eq("1 Secret Street")
-        expect(log.town_or_city).to eq("Secretville")
-        expect(log.uprn).to eq("123456789012")
-        expect(log.postcode_known).to eq(1)
+        expect(log.reload.address_line1).to eq("1 Secret Street")
       end
     end
 
@@ -192,5 +119,23 @@ RSpec.describe "clear_confidential_address_data" do
         expect { task.invoke }.not_to(change { log.reload.updated_at })
       end
     end
+
+    it "does not affect the scheme or location associations" do
+      log = create_log_with_address_data(scheme:, location:, startdate: Time.zone.local(2026, 5, 1))
+
+      task.invoke
+      log.reload
+
+      expect(log.scheme_id).to eq(scheme.id)
+      expect(log.location_id).to eq(location.id)
+    end
+
+    it "is idempotent" do
+      log = create_log_with_address_data(scheme:, location:, startdate: Time.zone.local(2026, 5, 1))
+
+      task.invoke
+      task.reenable
+      expect { task.invoke }.not_to(change { log.reload.updated_at })
+    end
   end
 end

spec/services/storage/archive_service_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe Storage::ArchiveService do
     file
   end
   let(:archive_content) do
-    zip_file = Zip::File.open_buffer(StringIO.new, create: true)
+    zip_file = Zip::File.open_buffer(StringIO.new)
     zip_file.mkdir(compressed_folder)
     zip_file.add(compressed_filepath, compressed_file)
     zip_file.write_buffer
@@ -51,7 +51,7 @@ RSpec.describe Storage::ArchiveService do
 
     it "raises an error if the file exists but is too large" do
       archive = archive_service.instance_variable_get(:@archive)
-      allow(archive).to receive(:get_entry).and_return(Zip::Entry.new(nil, "", size: 100_000_000))
+      allow(archive).to receive(:get_entry).and_return(Zip::Entry.new(nil, "", nil, nil, nil, nil, nil, 100_000_000, nil))
 
       expect { archive_service.get_file_io(compressed_filepath) }
         .to raise_error(RuntimeError, "File too large to be extracted")