Spatie\FailedJobMonitor\Notification

Complexity

Total Complexity

5

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	toSlack()	0	16	1
A	via()	0	4	1
A	setEvent()	0	6	1
A	toMail()	0	13	1
A	buildException()	0	15	1

<?php

namespace Spatie\FailedJobMonitor;

use Carbon\Carbon;
use Illuminate\Queue\Events\JobFailed;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Messages\SlackMessage;
use Illuminate\Notifications\Messages\SlackAttachment;
use Illuminate\Notifications\Notification as NotificationBase;

class Notification extends NotificationBase
{
    protected $event;

    public function via($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return config('laravel-failed-job-monitor.channels');
    }

    public function setEvent(JobFailed $event)
    {
        $this->event = $event;

        return $this;
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return (new MailMessage)
            ->subject(trans('laravel-failed-job-monitor::mail.subject'))

It seems like trans('laravel-failed-job-monitor::mail.subject') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::subject() does only seem to accept string, maybe add an additional type check?

            ->greeting(trans('laravel-failed-job-monitor::mail.greeting'))

It seems like trans('laravel-failed-job-monitor::mail.greeting') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::greeting() does only seem to accept string, maybe add an additional type check?

            ->line(trans('laravel-failed-job-monitor::mail.intro'))

It seems like trans('laravel-failed-job-monitor::mail.intro') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::line() does only seem to accept object<Illuminate\Notifications\Action>|string, maybe add an additional type check?

            ->line(trans('laravel-failed-job-monitor::mail.job_info', ['job' => $this->event->job->resolveName()]))

It seems like trans('laravel-failed-job-monitor::mail.job_info', array('job' => $this->event->job->resolveName())) targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::line() does only seem to accept object<Illuminate\Notifications\Action>|string, maybe add an additional type check?

            ->line(trans('laravel-failed-job-monitor::mail.attachment'))

It seems like trans('laravel-failed-job-monitor::mail.attachment') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::line() does only seem to accept object<Illuminate\Notifications\Action>|string, maybe add an additional type check?

            ->attachData($this->buildException($this->event->exception),
                'failed_job_'.Carbon::now()->format('Y-m-d h:i:s').'.txt')
            ->attachData($this->event->job->getRawBody(),
                'payload_'.Carbon::now()->format('Y-m-d h:i:s').'.txt');
    }

    /**
     * Get the Slack representation of the notification.
     *
     * @param  mixed $notifiable
     * @return \Illuminate\Notifications\Messages\SlackMessage
     */
    public function toSlack($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return (new SlackMessage)
            ->error()
            ->content(trans('laravel-failed-job-monitor::slack.intro'))

It seems like trans('laravel-failed-job-monitor::slack.intro') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SlackMessage::content() does only seem to accept string, maybe add an additional type check?

            ->attachment(function (SlackAttachment $attachment) {
                $attachment->title(trans('laravel-failed-job-monitor::slack.job_info'))

It seems like trans('laravel-failed-job-monitor::slack.job_info') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SlackAttachment::title() does only seem to accept string, maybe add an additional type check?

                    ->content($this->event->job->resolveName());
            })->attachment(function (SlackAttachment $attachment) {
                $attachment->title('failed_job_'.Carbon::now()->format('Y-m-d h:i:s').'.txt')
                    ->content($this->buildException($this->event->exception));
            })->attachment(function (SlackAttachment $attachment) {
                $attachment->title('payload_'.Carbon::now()->format('Y-m-d h:i:s').'.txt')
                    ->content($this->event->job->getRawBody());
            });
    }

    protected function buildException(\Exception $exception)
    {
        $response = sprintf(
            '%s: %s (%s:%s)',
            get_class($exception),
            $exception->getMessage(),
            $exception->getFile(),
            $exception->getLine()
        );

        $response .= PHP_EOL;
        $response .= $exception->getTraceAsString().PHP_EOL;

        return $response;
    }
}