gbere / simple-auth-bundle

src/Service/Mailer.php

<?php

declare(strict_types=1);

namespace Gbere\SimpleAuth\Service;

use Gbere\SimpleAuth\Bridge\Mime\TemplatedEmail;
use Gbere\SimpleAuth\Model\UserInterface;
use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
use Symfony\Component\Mailer\Exception\TransportExceptionInterface;
use Symfony\Component\Mailer\MailerInterface;
use Symfony\Component\Mime\Address;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Routing\RouterInterface;

class Mailer
{
    /** @var ParameterBagInterface */
    private $parameterBag;
    /** @var MailerInterface */
    private $mailer;
    /** @var RouterInterface */
    private $router;

    public function __construct(ParameterBagInterface $parameterBag, MailerInterface $mailer, RouterInterface $router)
    {
        $this->parameterBag = $parameterBag;
        $this->mailer = $mailer;
        $this->router = $router;
    }

    /**
     * @throws TransportExceptionInterface
     */
    public function sendConfirmRegistrationMessage(UserInterface $user): void
    {
        $this->mailer->send((new TemplatedEmail())
            ->from($this->getSenderEmail())
            ->to(new Address($user->getEmail(), $user->getName()))

It seems like $user->getEmail() 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 $user->getName() 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?

            ->subject('Confirm registration')
            ->context([
                'content' => 'Please click the next link to confirm registration',
                'action_url' => $this->generateUrl('simple_auth_confirm_registration', ['token' => $user->getConfirmationToken()]),
                'action_text' => 'Confirm registration',
            ])
        );
    }

    /**
     * @throws TransportExceptionInterface
     */
    public function sendWelcomeMessage(UserInterface $user): void
    {
        $this->mailer->send((new TemplatedEmail())
            ->from($this->getSenderEmail())
            ->to(new Address($user->getEmail(), $user->getName()))

It seems like $user->getEmail() 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 $user->getName() 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?

            ->subject('Welcome')
            ->context(['content' => 'Your registration was successfully completed'])
        );
    }

    /**
     * @throws TransportExceptionInterface
     */
    public function sendPasswordResetMessage(UserInterface $user): void
    {
        $this->mailer->send((new TemplatedEmail())
            ->from($this->getSenderEmail())
            ->to(new Address($user->getEmail(), $user->getName()))

It seems like $user->getEmail() 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 $user->getName() 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?

            ->subject('Password request')
            ->context([
                'content' => 'Please, click the next link to reset your password',
                'action_url' => $this->generateUrl('simple_auth_password_reset', ['token' => $user->getConfirmationToken()]),
                'action_text' => 'Reset password',
            ])
        );
    }

    /**
     * @throws TransportExceptionInterface
     */
    public function sendPasswordResetNotificationMessage(UserInterface $user): void
    {
        $this->mailer->send((new TemplatedEmail())
            ->from($this->getSenderEmail())
            ->to(new Address($user->getEmail(), $user->getName()))

It seems like $user->getEmail() 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 $user->getName() 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?

            ->subject('Password reset notification')
            ->context([
                'content' => 'Your password was successfully updated',
            ])
        );
    }

    private function getSenderEmail(): Address
    {
        return new Address(
            $this->parameterBag->get('simple_auth_sender_email'),
            $this->parameterBag->get('simple_auth_sender_name') ?? ''
        );
    }

    private function generateUrl(string $name, array $parameters = null): string
    {
        return $this->router->generate($name, $parameters, UrlGeneratorInterface::ABSOLUTE_URL);

It seems like $parameters can also be of type null; however, parameter $parameters of Symfony\Component\Routing\Generator\UrlGeneratorInterface::generate() does only seem to accept array, maybe add an additional type check?

    }
}