Better cleanup routine.

app/Http/Controllers/ExportController.php

@@ -48,6 +48,8 @@ class ExportController extends Controller
         $name   = 'Export job on ' . $date . '.zip';
         $quoted = sprintf('"%s"', addcslashes($name, '"\\'));
 
+        $job->change('export_downloaded');
+
 
         return response(file_get_contents($file), 200)
             ->header('Content-Description', 'File Transfer')

app/Repositories/ExportJob/ExportJobRepository.php

@@ -29,8 +29,7 @@ class ExportJobRepository implements ExportJobRepositoryInterface
     {
         $dayAgo = Carbon::create()->subDay();
         ExportJob::where('created_at', '<', $dayAgo->format('Y-m-d H:i:s'))
-                 ->where('status', 'never_started')
-            // TODO also delete others.
+                 ->whereIn('status', ['never_started', 'export_status_finished', 'export_downloaded'])
                  ->delete();
 
         return true;