Ensure unserialize for description works.

Author: binaryk

config/mailator.php

@@ -31,6 +31,11 @@
          * Class that will mark the scheduled action as being completed so the action will do not be counted into the next iteration.
          */
         'garbage_resolver' => Binarcode\LaravelMailator\Actions\ResolveGarbageAction::class,
+
+        /**
+         * Mark action completed after this count of fails.
+         */
+        'mark_complete_after_fails_count' => env('MAILATOR_FAILED_COUNTS', 3),
     ],
 
     'log_model' => Binarcode\LaravelMailator\Models\MailatorLog::class,

src/Actions/ResolveGarbageAction.php

@@ -35,7 +35,7 @@ public function shouldMarkComplete(MailatorSchedule $schedule): bool
             return true;
         }
 
-        if ($schedule->failedLastTimes(3)) {
+        if ($schedule->failedLastTimes(config('mailator.scheduler.mark_complete_after_fails_count', 3))) {
             return true;
         }
 

src/Models/Concerns/ConstraintsResolver.php

@@ -14,6 +14,7 @@
 use Binarcode\LaravelMailator\Constraints\SendScheduleConstraint;
 use Binarcode\LaravelMailator\Constraints\WeeklyConstraint;
 use Binarcode\LaravelMailator\Models\MailatorSchedule;
+use Throwable;
 
 /**
  * Trait ConstraintsResolver
@@ -53,22 +54,34 @@ public function eventsPasses(): bool
 
     public function constraintsDescriptions(): array
     {
-        return collect($this->constraints)
-            ->map(fn (string $event) => unserialize($event))
-            ->filter(fn ($event) => is_subclass_of($event, Descriptionable::class))
-            ->reduce(function ($base, Descriptionable $descriable) {
-                return array_merge($base, $descriable::conditions());
-            }, []);
+        try {
+            return collect($this->constraints)
+                ->map(fn(string $event) => unserialize($event))
+                ->filter(fn($event) => is_subclass_of($event, Descriptionable::class))
+                ->reduce(function ($base, Descriptionable $descriable) {
+                    return array_merge($base, $descriable::conditions());
+                }, []);
+        } catch (Throwable $e) {
+            $this->markAsFailed($e->getMessage());
+
+            return [];
+        }
     }
 
     public function constraintsNotSatisfiedDescriptions(): array
     {
-        return collect($this->constraints)
-            ->map(fn (string $event) => unserialize($event))
-            ->filter(fn ($event) => is_subclass_of($event, Descriptionable::class))
-            ->filter(fn ($event) => ! $event->canSend($this, $this->logs))
-            ->reduce(function ($base, Descriptionable $descriable) {
-                return array_merge($base, $descriable::conditions());
-            }, []);
+        try {
+            return collect($this->constraints)
+                ->map(fn(string $event) => unserialize($event))
+                ->filter(fn($event) => is_subclass_of($event, Descriptionable::class))
+                ->filter(fn($event) => !$event->canSend($this, $this->logs))
+                ->reduce(function ($base, Descriptionable $descriable) {
+                    return array_merge($base, $descriable::conditions());
+                }, []);
+        } catch (Throwable $e) {
+            $this->markAsFailed($e->getMessage());
+
+            return [];
+        }
     }
 }