Browse Source

Count lettings logs after writting export files (#1934)

* Count lettings logs after writting export files

* lint
pull/1937/head
kosiakkatrina 1 year ago committed by GitHub
parent
commit
101fa7d33e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 7
      app/services/exports/lettings_log_export_service.rb

7
app/services/exports/lettings_log_export_service.rb

@ -77,14 +77,13 @@ module Exports
logs_count = retrieve_lettings_logs(start_time, recent_export, full_update).filter_by_year(collection).count
@logger.info("Creating #{archive} - #{logs_count} logs")
manifest_xml = build_manifest_xml(logs_count)
return {} if logs_count.zero?
zip_file = Zip::File.open_buffer(StringIO.new)
zip_file.add("manifest.xml", manifest_xml)
part_number = 1
last_processed_marker = nil
total_logs = 0
loop do
lettings_logs_slice = if last_processed_marker.present?
@ -105,9 +104,13 @@ module Exports
zip_file.add("#{archive}_#{part_number_str}.xml", data_xml)
part_number += 1
last_processed_marker = lettings_logs_slice.last.created_at
total_logs += lettings_logs_slice.count
@logger.info("Added #{archive}_#{part_number_str}.xml")
end
manifest_xml = build_manifest_xml(logs_count)
zip_file.add("manifest.xml", manifest_xml)
# Required by S3 to avoid Aws::S3::Errors::BadDigest
zip_io = zip_file.write_buffer
zip_io.rewind

Loading…
Cancel
Save