App\MessageHandler\SendFeedbackHandler::__invoke()

Complexity

Conditions	1
Paths	1

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
    }