Fix CSV import storage disk handling and fail loudly.

This forces import uploads and reads to use the local disk and throws explicit exceptions when files are missing or unreadable so queue failures are visible.

Made-with: Cursor

Author: swilla

src/Jobs/ImportCourseFromCsv.php

@@ -9,8 +9,9 @@
 use Illuminate\Foundation\Bus\Dispatchable;
 use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Queue\SerializesModels;
-use Illuminate\Support\Facades\File;
+use Illuminate\Support\Facades\Storage;
 use Maatwebsite\Excel\Facades\Excel;
+use RuntimeException;
 use Tapp\FilamentLms\Imports\CourseStepsImport;
 
 class ImportCourseFromCsv implements ShouldQueue
@@ -19,17 +20,25 @@ class ImportCourseFromCsv implements ShouldQueue
 
     public function __construct(
         protected string $courseName,
-        protected string $filePath
+        protected string $storedPath
     ) {}
 
     public function handle(): void
     {
-        if (! File::isReadable($this->filePath)) {
-            return;
+        $disk = Storage::disk('local');
+
+        if (! $disk->exists($this->storedPath)) {
+            throw new RuntimeException("Course import file does not exist on local disk: {$this->storedPath}");
+        }
+
+        $filePath = $disk->path($this->storedPath);
+
+        if (! is_readable($filePath)) {
+            throw new RuntimeException("Course import file is not readable: {$filePath}");
         }
 
-        Excel::import(new CourseStepsImport($this->courseName), $this->filePath);
+        Excel::import(new CourseStepsImport($this->courseName), $filePath);
 
-        File::delete($this->filePath);
+        $disk->delete($this->storedPath);
     }
 }

src/Resources/CourseResource/Pages/ListCourses.php

@@ -59,7 +59,8 @@ protected function getHeaderActions(): array
 
                     $storedPath = $file->storeAs(
                         'filament-lms/course-imports',
-                        Str::uuid().'.csv'
+                        Str::uuid().'.csv',
+                        'local'
                     );
 
                     if ($storedPath === false) {
@@ -72,7 +73,7 @@ protected function getHeaderActions(): array
                         return;
                     }
 
-                    ImportCourseFromCsv::dispatch($courseName, Storage::path($storedPath));
+                    ImportCourseFromCsv::dispatch($courseName, $storedPath);
 
                     Notification::make()
                         ->title('Import queued')