App\Form\EventSubscriber\FeedbackNotificationSubscriber

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	onContactFormSubmitted()	0	13	1
A	getSubscribedEvents()	0	4	1
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace App\Form\EventSubscriber;

use App\Entity\Contact;
use App\Event\ContactFormSubmittedEvent;
use App\Mailer\Sender\Adapter\SwiftMailerAdapter;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Contracts\Translation\TranslatorInterface;

final class FeedbackNotificationSubscriber implements EventSubscriberInterface
{
    /**
     * @var SwiftMailerAdapter
     */
    private $mailer;

    /**
     * @var TranslatorInterface
     */
    private $translator;

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

    public static function getSubscribedEvents(): array
    {
        return [
            ContactFormSubmittedEvent::class => 'onContactFormSubmitted',
        ];
    }

    public function onContactFormSubmitted(ContactFormSubmittedEvent $event): void
    {
        /** @var Contact $contact */
        $contact = $event->getContact();

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

        $this->mailer->send(
            $contact->getFromName(),

It seems like $contact->getFromName() can also be of type null; however, parameter $fromName of App\Mailer\Sender\Adapter\SwiftMailerAdapter::send() does only seem to accept string, maybe add an additional type check?

            $contact->getFromEmail(),

It seems like $contact->getFromEmail() can also be of type null; however, parameter $fromEmail of App\Mailer\Sender\Adapter\SwiftMailerAdapter::send() does only seem to accept string, maybe add an additional type check?

            $contact->getToEmail(),

It seems like $contact->getToEmail() can also be of type null; however, parameter $toEmail of App\Mailer\Sender\Adapter\SwiftMailerAdapter::send() does only seem to accept string, maybe add an additional type check?

            $subject,
            $contact->getMessage()

It seems like $contact->getMessage() can also be of type null; however, parameter $body of App\Mailer\Sender\Adapter\SwiftMailerAdapter::send() does only seem to accept string, maybe add an additional type check?

        );
    }
}