From a399d845c700c077ec19ef88e630415d37bd29b3 Mon Sep 17 00:00:00 2001
From: Manny Dinssa <44172848+Dinssa@users.noreply.github.com>
Date: Fri, 20 Sep 2024 16:51:32 +0100
Subject: [PATCH] Implement download file link

---
 .../bulk_upload_summary_component.html.erb    |  6 ++--
 app/helpers/bulk_upload_helper.rb             | 29 +++++++++++++++----
 app/services/bulk_upload/downloader.rb        |  4 +++
 app/views/bulk_upload_shared/uploads.html.erb |  2 +-
 config/routes.rb                              | 12 ++++++++
 5 files changed, 43 insertions(+), 10 deletions(-)

diff --git a/app/components/bulk_upload_summary_component.html.erb b/app/components/bulk_upload_summary_component.html.erb
index f89c3d9a7..9a418d099 100644
--- a/app/components/bulk_upload_summary_component.html.erb
+++ b/app/components/bulk_upload_summary_component.html.erb
@@ -13,9 +13,9 @@
           <p class="govuk-hint govuk-!-font-size-16 govuk-!-margin-bottom-1">Time of upload: <%= bulk_upload.created_at.to_formatted_s(:govuk_date_and_time) %></p>
       </div>
       <p class="govuk-body govuk-!-margin-bottom-3">
-        <%= download_file_link %>
-        <%= view_error_report_link %>
-        <%= view_logs_link %>
+        <%= download_file_link(controller, bulk_upload) %>
+        <%= view_error_report_link(bulk_upload) %>
+        <%= view_logs_link(bulk_upload) %>
       </p>
     </div>
     <footer class="govuk-grid-column-one-third app-log-summary__footer">
diff --git a/app/helpers/bulk_upload_helper.rb b/app/helpers/bulk_upload_helper.rb
index 07adaff07..a5bca94b2 100644
--- a/app/helpers/bulk_upload_helper.rb
+++ b/app/helpers/bulk_upload_helper.rb
@@ -2,11 +2,11 @@ module BulkUploadHelper
   def bulk_upload_title(controller)
     case controller.controller_name
     when "lettings_logs"
-      "Lettings Bulk Uploads"
+      "Lettings bulk uploads"
     when "sales_logs"
-      "Sales Bulk Uploads"
+      "Sales bulk uploads"
     else
-      "Bulk Uploads"
+      "Bulk uploads"
     end
   end
 
@@ -34,15 +34,32 @@ module BulkUploadHelper
     }.compact.join("").html_safe
   end
 
-  def download_file_link
-    link_to "Download file", "#", class: "govuk-link govuk-!-margin-right-2"
+  def download_file_link(controller, bulk_upload)
+    case controller.controller_name
+    when "lettings_logs"
+      download_lettings_file_link(bulk_upload)
+    when "sales_logs"
+      download_sales_file_link(bulk_upload)
+    else
+      raise "Download file link not found for bulk upload"
+    end
   end
 
   def view_error_report_link
     link_to "View error report", "#", class: "govuk-link"
+  def download_lettings_file_link(bulk_upload)
+    link_to "Download file", download_lettings_bulk_upload_path(bulk_upload), class: "govuk-link govuk-!-margin-right-2"
+  end
+
+  def download_sales_file_link(bulk_upload)
+    link_to "Download file", download_sales_bulk_upload_path(bulk_upload), class: "govuk-link govuk-!-margin-right-2"
+  end
+
+  def view_error_report_link(bulk_upload)
+    link_to "View error report", "/lettings-logs/bulk-upload-resume/#{bulk_upload.id}/start", class: "govuk-link"
   end
 
-  def view_logs_link
+  def view_logs_link(bulk_upload)
     link_to "View logs", "#", class: "govuk-link"
   end
 
diff --git a/app/services/bulk_upload/downloader.rb b/app/services/bulk_upload/downloader.rb
index a7dc9aad0..05111bf1e 100644
--- a/app/services/bulk_upload/downloader.rb
+++ b/app/services/bulk_upload/downloader.rb
@@ -15,6 +15,10 @@ class BulkUpload::Downloader
     file.unlink
   end
 
+  def file_path
+    file.path
+  end
+
 private
 
   def download
diff --git a/app/views/bulk_upload_shared/uploads.html.erb b/app/views/bulk_upload_shared/uploads.html.erb
index 2cf829d83..958964ff6 100644
--- a/app/views/bulk_upload_shared/uploads.html.erb
+++ b/app/views/bulk_upload_shared/uploads.html.erb
@@ -4,7 +4,7 @@
 <% content_for :title, title %>
 
 <h1 class="govuk-heading-l govuk-!-margin-bottom-7">
-  <%= "#{log_type_for_controller(controller).capitalize} bulk uploads" %>
+  <%= bulk_upload_title(controller) %>
 </h1>
 
 <div class="app-filter-layout" data-controller="filter-layout">
diff --git a/config/routes.rb b/config/routes.rb
index bbb32d58e..7551d5ed2 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -278,6 +278,12 @@ Rails.application.routes.draw do
         end
       end
 
+      resources :bulk_uploads, only: [] do
+        member do
+          get "download", to: "lettings_logs#download_bulk_upload", as: "download_lettings"
+        end
+      end
+
       get "update-logs", to: "lettings_logs#update_logs"
     end
 
@@ -350,6 +356,12 @@ Rails.application.routes.draw do
           patch "*page", to: "bulk_upload_sales_soft_validations_check#update"
         end
       end
+
+      resources :bulk_uploads, only: [] do
+        member do
+          get "download", to: "sales_logs#download_bulk_upload"
+        end
+      end
     end
 
     member do