Silverback\ApiComponentBundle\Mailer\Mailer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	newUsernameConfirmation()	0	26	1
A	pathToAppUrl()	0	10	2
A	__construct()	0	10	1
A	passwordResetEmail()	0	26	1

<?php

namespace Silverback\ApiComponentBundle\Mailer;

use Silverback\ApiComponentBundle\Entity\User\User;
use Symfony\Component\HttpFoundation\RequestStack;
use Twig\Environment;

class Mailer
{
    private $mailer;
    private $twig;
    private $requestStack;
    private $fromEmailAddress;

    public function __construct(
        \Swift_Mailer $mailer,
        Environment $twig,
        RequestStack $requestStack,
        string $fromEmailAddress
    ) {
        $this->mailer = $mailer;
        $this->twig = $twig;
        $this->requestStack = $requestStack;
        $this->fromEmailAddress = $fromEmailAddress;
    }

    public function passwordResetEmail(User $user, string $resetUrl): int
    {
        $resetUrl = $this->pathToAppUrl(
            $resetUrl,
            $user->getPasswordResetConfirmationToken(),

It seems like $user->getPasswordResetConfirmationToken() can also be of type null; however, parameter $token of Silverback\ApiComponentBundle\Mailer\Mailer::pathToAppUrl() does only seem to accept string, maybe add an additional type check?

            $user->getUsername()

It seems like $user->getUsername() can also be of type null; however, parameter $email of Silverback\ApiComponentBundle\Mailer\Mailer::pathToAppUrl() does only seem to accept string, maybe add an additional type check?

        );
        $message = (new \Swift_Message('Password Reset Request'))
            ->setFrom($this->fromEmailAddress)
            ->setTo($user->getUsername())
            ->setBody(
                $this->twig->render(
                    '@SilverbackApiComponent/emails/password_reset.html.twig',
                    ['user' => $user, 'reset_url' => $resetUrl]
                ),
                'text/html'
            )
            ->addPart(
                $this->twig->render(
                    '@SilverbackApiComponent/emails/password_reset.txt.twig',
                    ['user' => $user, 'reset_url' => $resetUrl]
                ),
                'text/plain'
            )
        ;
        return $this->mailer->send($message);
    }

    public function newUsernameConfirmation(User $user, string $confirmUrl): int
    {
        $confirmUrl = $this->pathToAppUrl(
            $confirmUrl,
            $user->getUsernameConfirmationToken(),

It seems like $user->getUsernameConfirmationToken() can also be of type null; however, parameter $token of Silverback\ApiComponentBundle\Mailer\Mailer::pathToAppUrl() does only seem to accept string, maybe add an additional type check?

            $user->getNewUsername()

It seems like $user->getNewUsername() can also be of type null; however, parameter $email of Silverback\ApiComponentBundle\Mailer\Mailer::pathToAppUrl() does only seem to accept string, maybe add an additional type check?

        );
        $message = (new \Swift_Message('Confirm change of username'))
            ->setFrom($this->fromEmailAddress)
            ->setTo($user->getNewUsername())
            ->setBody(
                $this->twig->render(
                    '@SilverbackApiComponent/emails/new_username_confirmation.html.twig',
                    ['user' => $user, 'confirm_url' => $confirmUrl]
                ),
                'text/html'
            )
            ->addPart(
                $this->twig->render(
                    '@SilverbackApiComponent/emails/new_username_confirmation.txt.twig',
                    ['user' => $user, 'confirm_url' => $confirmUrl]
                ),
                'text/plain'
            )
        ;
        return $this->mailer->send($message);
    }

    private function pathToAppUrl(string $path, string $token, string $email): string
    {
        $request = $this->requestStack->getCurrentRequest();
        $urlParts = $request ? parse_url($request->headers->get('referer')) : ['scheme' => 'https', 'host' => 'no-referrer'];
        $path = str_replace(['{{ token }}', '{{ email }}'], [$token, $email], $path);
        return sprintf(
            '%s://%s/%s',
            $urlParts['scheme'],
            $urlParts['host'],
            ltrim($path, '/')
        );
    }
}