[Fix] Hardcoded MailAttachment model should be replaced by config (#44)

* Use the MailAttachment model from the mails config

* Fix styling

* Remove non existing path check

* Fix undefined property

---------

Co-authored-by: Baspa <10845460+Baspa@users.noreply.github.com>

Author: Baspa

phpstan.neon.dist

@@ -6,7 +6,6 @@ parameters:
     paths:
         - src
         - config
-        - database
     tmpDir: build/phpstan
     checkOctaneCompatibility: true
     checkModelProperties: true

src/Controllers/MailDownloadController.php

@@ -3,8 +3,8 @@
 namespace Vormkracht10\FilamentMails\Controllers;
 
 use Illuminate\Routing\Controller;
+use Illuminate\Support\Facades\Config;
 use Illuminate\Support\Facades\Storage;
-use Vormkracht10\Mails\Models\MailAttachment;
 
 class MailDownloadController extends Controller
 {
@@ -17,8 +17,9 @@ public function __invoke(...$arguments)
             $tenant = null;
         }
 
-        /** @var MailAttachment $attachment */
-        $attachment = MailAttachment::find($attachment);
+        $attachmentModel = Config::get('mails.models.attachment');
+        /** @var \Vormkracht10\Mails\Models\MailAttachment $attachment */
+        $attachment = $attachmentModel::find($attachment);
 
         $file = Storage::disk($attachment->disk)->path($attachment->storagePath);
 

src/Resources/SuppressionResource.php

@@ -125,7 +125,7 @@ public static function table(Table $table): Table
                 Tables\Actions\Action::make('unsuppress')
                     ->label(__('Unsuppress'))
                     ->action(function (MailEvent $record) {
-                        event(new MailUnsuppressed(key($record->to), $record->mail->mailer == 'smtp' && filled($record->mail->transport) ? $record->mail->transport : $record->mail->mailer, $record->mail->stream_id ?? null));
+                        event(new MailUnsuppressed(key($record->mail->to), $record->mail->mailer == 'smtp' && filled($record->mail->transport) ? $record->mail->transport : $record->mail->mailer, $record->mail->stream_id ?? null));
                     })
                     ->visible(fn ($record) => Provider::tryFrom($record->mail->mailer == 'smtp' && filled($record->mail->transport) ? $record->mail->transport : $record->mail->mailer)),