Fix styling

src/Constraints/HoursSchedulerCheckerConstraint.php

@@ -2,16 +2,14 @@
 
 namespace Binarcode\LaravelMailator\Constraints;
 
-use Binarcode\LaravelMailator\Models\MailatorLog;
 use Binarcode\LaravelMailator\Models\MailatorSchedule;
-use Carbon\Carbon;
 use Illuminate\Support\Collection;
 
 class HoursSchedulerCheckerConstraint implements SendScheduleConstraint
 {
     public function canSend(MailatorSchedule $schedule, Collection $logs): bool
     {
-        if (!$schedule->hasPrecision()) {
+        if (! $schedule->hasPrecision()) {
             return true;
         }
 

src/Models/Concerns/ConstraintsResolver.php

@@ -37,8 +37,8 @@ public function configurationsPasses(): bool
             WeeklyConstraint::class,
             HoursSchedulerCheckerConstraint::class,
         ])
-            ->map(fn($class) => app($class))
-            ->every(fn(SendScheduleConstraint $event) => $event->canSend($this, $this->logs));
+            ->map(fn ($class) => app($class))
+            ->every(fn (SendScheduleConstraint $event) => $event->canSend($this, $this->logs));
     }
 
     public function whenPasses(): bool
@@ -49,18 +49,20 @@ public function whenPasses(): bool
     public function eventsPasses(): bool
     {
         return collect($this->constraints)
-                ->map(fn(string $event) => unserialize($event))
-                ->filter(fn($event) => is_subclass_of($event, SendScheduleConstraint::class))
-                ->filter(fn(SendScheduleConstraint $event) => $event->canSend($this,
-                    $this->logs))->count() === collect($this->constraints)->count();
+                ->map(fn (string $event) => unserialize($event))
+                ->filter(fn ($event) => is_subclass_of($event, SendScheduleConstraint::class))
+                ->filter(fn (SendScheduleConstraint $event) => $event->canSend(
+                    $this,
+                    $this->logs
+                ))->count() === collect($this->constraints)->count();
     }
 
     public function constraintsDescriptions(): array
     {
         try {
             return collect($this->constraints)
-                ->map(fn(string $event) => unserialize($event))
-                ->filter(fn($event) => is_subclass_of($event, Descriptionable::class))
+                ->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());
                 }, []);
@@ -75,10 +77,10 @@ public function constraintsNotSatisfiedDescriptions(): array
     {
         try {
             return collect($this->constraints)
-                ->map(fn(string $event) => unserialize($event))
-                ->filter(fn($event) => is_subclass_of($event, Descriptionable::class))
-                ->filter(fn($event) => is_subclass_of($event, SendScheduleConstraint::class))
-                ->filter(fn($event) => !$event->canSend($this, $this->logs))
+                ->map(fn (string $event) => unserialize($event))
+                ->filter(fn ($event) => is_subclass_of($event, Descriptionable::class))
+                ->filter(fn ($event) => is_subclass_of($event, SendScheduleConstraint::class))
+                ->filter(fn ($event) => ! $event->canSend($this, $this->logs))
                 ->reduce(function ($base, Descriptionable $descriable) {
                     return array_merge($base, $descriable::conditions());
                 }, []);

src/Models/MailatorSchedule.php

@@ -113,8 +113,8 @@ public function mailable(Mailable $mailable): self
     {
         if ($mailable instanceof Constraintable) {
             collect($mailable->constraints())
-                ->filter(fn($constraint) => $constraint instanceof SendScheduleConstraint)
-                ->each(fn(SendScheduleConstraint $constraint) => $this->constraint($constraint));
+                ->filter(fn ($constraint) => $constraint instanceof SendScheduleConstraint)
+                ->each(fn (SendScheduleConstraint $constraint) => $this->constraint($constraint));
         }
 
         $this->mailable_class = serialize($mailable);
@@ -337,7 +337,7 @@ public function recipients(...$recipients): self
     {
         $this->recipients = array_merge(collect($recipients)
             ->flatten()
-            ->filter(fn($email) => $this->ensureValidEmail($email))
+            ->filter(fn ($email) => $this->ensureValidEmail($email))
             ->unique()
             ->toArray(), $this->recipients ?? []);
 
@@ -363,15 +363,15 @@ public function shouldSend(): bool
         try {
             $this->load('logs');
 
-            if (!$this->configurationsPasses()) {
+            if (! $this->configurationsPasses()) {
                 return false;
             }
 
-            if (!$this->whenPasses()) {
+            if (! $this->whenPasses()) {
                 return false;
             }
 
-            if (!$this->eventsPasses()) {
+            if (! $this->eventsPasses()) {
                 if ($this->isStopable()) {
                     $this->markComplete();
                 }
@@ -391,7 +391,7 @@ public function shouldSend(): bool
 
     public function executeWhenPasses(bool $now = false): void
     {
-        if (!$this->save()) {
+        if (! $this->save()) {
             return;
         }
 
@@ -402,7 +402,7 @@ public function executeWhenPasses(bool $now = false): void
 
     public function execute(bool $now = false): void
     {
-        if (!$this->save()) {
+        if (! $this->save()) {
             return;
         }
 
@@ -432,7 +432,7 @@ public static function run(): void
 
     public function hasCustomAction(): bool
     {
-        return !is_null($this->action);
+        return ! is_null($this->action);
     }
 
     public function getMailable(): ?Mailable
@@ -487,13 +487,13 @@ public function markAsFailed(string $failureReason): self
     public function getRecipients(): array
     {
         return collect($this->recipients)
-            ->filter(fn($email) => $this->ensureValidEmail($email))
+            ->filter(fn ($email) => $this->ensureValidEmail($email))
             ->toArray();
     }
 
     protected function ensureValidEmail(string $email): bool
     {
-        return !Validator::make(
+        return ! Validator::make(
             compact('email'),
             ['email' => 'required|email']
         )->fails();
@@ -557,7 +557,7 @@ public function markComplete(): self
 
     public function isCompleted(): bool
     {
-        return !is_null($this->completed_at);
+        return ! is_null($this->completed_at);
     }
 
     public function failedLastTimes(int $times): bool
@@ -579,12 +579,12 @@ public function timestampTarget(): ?CarbonInterface
 
     public function isRepetitive(): bool
     {
-        return !$this->isOnce();
+        return ! $this->isOnce();
     }
 
     public function wasSentOnce(): bool
     {
-        return !is_null($this->last_sent_at);
+        return ! is_null($this->last_sent_at);
     }
 
     public function getConstraints(): ConstraintsCollection
@@ -594,7 +594,7 @@ public function getConstraints(): ConstraintsCollection
 
     public function save(array $options = [])
     {
-        if (!$this->isUnique()) {
+        if (! $this->isUnique()) {
             return parent::save($options);
         }
 

tests/Feature/WithWeekendsConstraintTest.php

@@ -2,7 +2,6 @@
 
 namespace Binarcode\LaravelMailator\Tests\Feature;
 
-use Binarcode\LaravelMailator\Constraints\AfterConstraint;
 use Binarcode\LaravelMailator\Models\MailatorSchedule;
 use Binarcode\LaravelMailator\Tests\Fixtures\InvoiceReminderMailable;
 use Binarcode\LaravelMailator\Tests\TestCase;