App\MessageHandler\SendFeedbackHandler

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	16	1
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace App\MessageHandler;

use App\Dto\FeedbackDto;
use App\Mailer\Mailer;
use App\Message\SendFeedback;
use Symfony\Component\Messenger\Handler\MessageHandlerInterface;
use Symfony\Component\Mime\Address;
use Symfony\Component\Mime\Email;
use Symfony\Contracts\Translation\TranslatorInterface;

final class SendFeedbackHandler implements MessageHandlerInterface
{
    private Mailer $mailer;
    private TranslatorInterface $translator;

    public function __construct(Mailer $mailer, TranslatorInterface $translator)
    {
        $this->mailer = $mailer;
        $this->translator = $translator;
    }

    public function __invoke(SendFeedback $sendFeedback): void
    {
        /** @var FeedbackDto $feedback */
        $feedback = $sendFeedback->getFeedback();

        $subject = $this->translator->trans('email.new_message');

        $email = (new Email())
            ->from(new Address($feedback->getFromEmail(), $feedback->getFromName()))

It seems like $feedback->getFromEmail() can also be of type null; however, parameter $address of Symfony\Component\Mime\Address::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $feedback->getFromName() can also be of type null; however, parameter $name of Symfony\Component\Mime\Address::__construct() does only seem to accept string, maybe add an additional type check?

            ->to($feedback->getToEmail())

It seems like $feedback->getToEmail() can also be of type null; however, parameter $addresses of Symfony\Component\Mime\Email::to() does only seem to accept Symfony\Component\Mime\Address|string, maybe add an additional type check?

            ->replyTo($feedback->getFromEmail())

It seems like $feedback->getFromEmail() can also be of type null; however, parameter $addresses of Symfony\Component\Mime\Email::replyTo() does only seem to accept Symfony\Component\Mime\Address|string, maybe add an additional type check?

            ->subject($subject)
            ->text($feedback->getMessage())
            ->html($feedback->getMessage());

        $this->mailer->send($email);
    }
}