fix: fix empty array error (#9)

* fix: fix empty array error

* Fix styling

Author: RichardGL11

src/Filament/Pages/ListBetterEmails.php

@@ -123,9 +123,9 @@ protected function getHeaderWidgets(): array
         ];
     }
 
-    private static function formatMailState(array $emails, bool $mailOnly = false): string
+    private static function formatMailState(?array $emails, bool $mailOnly = false): string
     {
-        return collect($emails)
+        return collect($emails ?? [])
             ->mapWithKeys(fn ($value, $key) => [$key => $value ?? $key])
             ->map(fn ($value, $key): string|int => $mailOnly ? $value : ($value == null ? $key : ($value !== $key ? sprintf('%s <%s>', $value, $key) : $key)))
             ->implode(', ');

src/Filament/Schemas/BetterEmailInfolist.php

@@ -271,9 +271,9 @@ public static function buildMailEventsTab(): Tab
             ]);
     }
 
-    private static function formatMailState(array $emails, bool $mailOnly = false): string
+    private static function formatMailState(?array $emails, bool $mailOnly = false): string
     {
-        return collect($emails)
+        return collect($emails ?? [])
             ->mapWithKeys(fn ($value, $key) => [$key => $value ?? $key])
             ->map(fn ($value, $key): string|int => $mailOnly ? $key : ($value == null ? $key : ($value !== $key ? sprintf('%s <%s>', $value, $key) : $key)))
             ->implode(', ');

src/Filament/Widgets/BetterEmailStatsWidget.php

@@ -50,25 +50,25 @@ protected function getStats(): array
 
         $widgets[] = Stat::make(__('Delivered'), number_format(($deliveredMails / $mailCount) * 100, 1).'%')
             ->label(__('Delivered'))
-            ->description($deliveredMails.' '.__('of').' '.$mailCount.' '.__('emails'))
+            ->description($deliveredMails.' '.(string) __('of').' '.$mailCount.' '.(string) __('emails'))
             ->color('success')
             ->url($generateUrl('delivered'));
 
         $widgets[] = Stat::make(__('Opened'), number_format(($openedMails / $mailCount) * 100, 1).'%')
             ->label(__('Opened'))
-            ->description($openedMails.' '.__('of').' '.$mailCount.' '.__('emails'))
+            ->description($openedMails.' '.(string) __('of').' '.$mailCount.' '.(string) __('emails'))
             ->color('info')
             ->url($generateUrl('opened'));
 
         $widgets[] = Stat::make(__('Clicked'), number_format(($clickedMails / $mailCount) * 100, 1).'%')
             ->label(__('Clicked'))
-            ->description($clickedMails.' '.__('of').' '.$mailCount.' '.__('emails'))
+            ->description($clickedMails.' '.(string) __('of').' '.$mailCount.' '.(string) __('emails'))
             ->color('clicked')
             ->url($generateUrl('clicked'));
 
         $widgets[] = Stat::make(__('Bounced'), number_format(($bouncedMails / $mailCount) * 100, 1).'%')
             ->label(__('Bounced'))
-            ->description($bouncedMails.' '.__('of').' '.$mailCount.' '.__('emails'))
+            ->description($bouncedMails.' '.(string) __('of').' '.$mailCount.' '.(string) __('emails'))
             ->color('danger')
             ->url($generateUrl('bounced'));