App\Manager\ActivationLinkManager

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createActivationLink()	0	10	1
A	__construct()	0	4	1
A	findOneByHash()	0	3	1
A	sendValidationMail()	0	7	1

<?php
namespace App\Manager;

use Doctrine\ORM\EntityManagerInterface;
use App\Entity\User\{User, ActivationLink};
use App\Utils\Mailer;

class ActivationLinkManager
{
    /** @var EntityManagerInterface **/
    protected $em;
    /** @var Mailer **/
    protected $mailer;

    /**
     * @param EntityManagerInterface $em
     * @param Mailer $mailer
     */
    public function __construct(EntityManagerInterface $em, Mailer $mailer)
    {
        $this->em = $em;
        $this->mailer = $mailer;
    }

    /**
     * @param \User $user

The type User was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function createActivationLink(User $user)
    {
        $activationLink =
            (new ActivationLink())
            ->setHash(md5(uniqid(null, true)))
        ;
        $user->setActivationLink($activationLink);
        $this->em->persist($activationLink);
        $this->em->persist($user);
        $this->em->flush();
    }

    /**
     * @param User $user
     * @return bool
     */
    public function sendValidationMail(User $user): bool
    {
        $nbRecipients = $this->mailer->sendTo($user->getEmail(), 'users.registration.mails.confirmation', [
            '%username%' => $user->getUsername(),
            '%activation_link%' => $user->getActivationLink()->getHash()
        ], 'Personal');
        return $nbRecipients > 0;
    }

    /**
     * @param string $hash
     * @return ActivationLink
     */
    public function findOneByHash(string $hash)
    {
        return $this->em->getRepository(ActivationLink::class)->findOneByHash($hash);

The method findOneByHash() does not exist on Doctrine\Common\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

    }
}