wip #5475

diff --git a/app/Http/Controllers/Admin/Operations/FluidbookPublication/StatsOperation.php b/app/Http/Controllers/Admin/Operations/FluidbookPublication/StatsOperation.php
index 9e02f520e84f36f01299bfada433855bba3b3fef..156c82336adf97905018d77aa419d5095d1d37dc 100644 (file)
--- a/app/Http/Controllers/Admin/Operations/FluidbookPublication/StatsOperation.php
+++ b/app/Http/Controllers/Admin/Operations/FluidbookPublication/StatsOperation.php
@@ -41,7 +41,7 @@ trait StatsOperation
         // API testing tool (intended for superadmins only)
         Route::get($segment . '/stats/API/{id?}', $controller . '@statsAPI');
 
-        //
+        // Route to export datas to Excel
         Route::post($segment . '/stats/{fluidbook_id}_{hash}/{date?}/{period_override?}/generate', $controller . '@generateExcel')
             ->name('generateexcel');
     }
@@ -92,8 +92,8 @@ trait StatsOperation
     public function generateExcel($fluidbook_id, $hash, $date = null, $period_override = null) {
         $url = route('stats', compact('fluidbook_id', 'hash', 'date', 'period_override'));
 
-        $name = "excel";
-        $safename = Str::slug($name);
+        $dateForFilename = str_replace('-','',str_replace(',','_', $date));
+        $name = "stats_".$fluidbook_id."_".$dateForFilename;
 
         $locale = backpack_user()->locale;
         $acceptLang = [
@@ -109,7 +109,7 @@ trait StatsOperation
         $tables = $html->find('table');
         $tmpfile = Files::tempnam() . '.xlsx';
         $this->Excel_($tables, $names, null, $tmpfile);
-        return response()->download($tmpfile, $safename . '.xlsx')->deleteFileAfterSend(true);
+        return response()->download($tmpfile, $name . '.xlsx')->deleteFileAfterSend(true);
         die();
     }