Merge pull request #3576 from projectblacklight/3574-csv-stream

Stream csv response

Author: taylor-steve

app/controllers/spotlight/bulk_updates_controller.rb

@@ -24,7 +24,7 @@ def download_template
       headers['Content-Disposition'] = "attachment; filename=\"#{current_exhibit.slug}-bulk-update-template.csv\""
       headers.delete('Content-Length')
 
-      self.response_body = csv_template
+      stream_csv_template
     end
 
     def update
@@ -43,16 +43,31 @@ def monitor
 
     private
 
+    def stream_csv_template
+      csv_template.each do |row|
+        response.stream.write row
+      end
+    ensure
+      response.stream.close
+    end
+
     def csv_template
-      boolean = ActiveModel::Type::Boolean.new
-      Spotlight::BulkUpdatesCsvTemplateService.new(exhibit: current_exhibit).template(
-        view_context:,
-        title: boolean.cast(reference_field_params[:item_title]),
-        tags: boolean.cast(updatable_field_params[:tags]),
-        visibility: boolean.cast(updatable_field_params[:visibility])
+      @csv_template ||= csv_template_service.template(
+        view_context: view_context,
+        title: active_model_boolean.cast(reference_field_params[:item_title]),
+        visibility: active_model_boolean.cast(updatable_field_params[:visibility]),
+        tags: active_model_boolean.cast(updatable_field_params[:tags])
       )
     end
 
+    def csv_template_service
+      @csv_template_service ||= Spotlight::BulkUpdatesCsvTemplateService.new(exhibit: current_exhibit)
+    end
+
+    def active_model_boolean
+      @active_model_boolean ||= ActiveModel::Type::Boolean.new
+    end
+
     def reference_field_params
       params.require(:reference_fields).permit(:item_title)
     end

blacklight-spotlight.gemspec

@@ -44,7 +44,7 @@ these collections.)
   s.add_dependency 'openseadragon', '>= 0.8.0'
   s.add_dependency 'ostruct', '!= 0.3.0', '!= 0.3.1', '!= 0.3.2'
   s.add_dependency 'paper_trail', '>= 11.0', '< 17'
-  s.add_dependency 'rails', '>= 7.1', '< 8.0.3' # pending https://github.com/projectblacklight/spotlight/issues/3574
+  s.add_dependency 'rails', '>= 7.1', '< 9'
   s.add_dependency 'redcarpet', '>= 2.0.1', '< 4'
   s.add_dependency 'riiif', '~> 2.0'
   s.add_dependency 'roar', '~> 1.1'