perf: check horizon.path instead of entire horizon config (#92)

Ayoub-Mabrouk · web-flow · commit 5e65f8cdedc0 · 2025-12-17T15:56:30.000+01:00
Only check the specific config value needed instead of loading
the entire horizon config array.
diff --git a/src/Notification.php b/src/Notification.php
@@ -38,7 +38,7 @@ public function toMail($notifiable): MailMessage
             ->line("Job class: {$this->event->job->resolveName()}")
             ->line("Job body: {$this->event->job->getRawBody()}")
             ->line("Exception: {$this->event->exception->getTraceAsString()}")
-            ->when(config('horizon') !== null, fn (MailMessage $mailMessage) => $mailMessage->action('View Error', url(config('horizon.path').'/failed/'.$this->event->job->getJobId())));
+            ->when(config('horizon.path') !== null, fn (MailMessage $mailMessage) => $mailMessage->action('View Error', url(config('horizon.path').'/failed/'.$this->event->job->getJobId())));
     }
 
     public function toSlack(): SlackMessage

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public function toMail($notifiable): MailMessage`
`38`	`38`	`->line("Job class: {$this->event->job->resolveName()}")`
`39`	`39`	`->line("Job body: {$this->event->job->getRawBody()}")`
`40`	`40`	`->line("Exception: {$this->event->exception->getTraceAsString()}")`
`41`		`- ->when(config('horizon') !== null, fn (MailMessage $mailMessage) => $mailMessage->action('View Error', url(config('horizon.path').'/failed/'.$this->event->job->getJobId())));`
	`41`	`+ ->when(config('horizon.path') !== null, fn (MailMessage $mailMessage) => $mailMessage->action('View Error', url(config('horizon.path').'/failed/'.$this->event->job->getJobId())));`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`public function toSlack(): SlackMessage`