file name download should include MD5 from the contents of getCode #48

CareSet · Dec 10, 2019 · 5187b10 · 5187b10
1 parent 192afb9
commit 5187b10
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/CareSet/Zermelo/Http/Controllers/TabularApiController.php b/src/CareSet/Zermelo/Http/Controllers/TabularApiController.php
@@ -47,6 +47,13 @@ public function download( ZermeloRequest $request )
         $reportGenerator = new ReportGenerator( $cache );
         $collection = $reportGenerator->getCollection();
 
+        // File name download should include MD5 from the contents of getCode #48
+        if ($report->getCode()) {
+            $filename = $report->GetReportName() . '-'.$report->getCode().'.csv';
+        } else {
+            $filename = $report->GetReportName() . '.csv';
+        }
+
         $response = new StreamedResponse( function() use ( $header, $collection ) {
             // Open output stream
             $handle = fopen('php://output', 'w');
@@ -65,7 +72,7 @@ public function download( ZermeloRequest $request )
         }, 200, [
             'Content-Description' => 'File Transfer',
             'Content-Type' => 'text/csv',
-            'Content-Disposition' => 'attachment; filename="'.$report->GetReportName().'.csv"',
+            'Content-Disposition' => 'attachment; filename="'.$filename.'"',
             'Content-Type' => 'application/octet-stream',
             'Expires' => '0',
             'Cache-Control' => 'must-revalidate',