sumocoders / FrameworkMultiUserBundle

Controller/PasswordResetController.php

<?php

namespace SumoCoders\FrameworkMultiUserBundle\Controller;

use SumoCoders\FrameworkMultiUserBundle\Command\ResetPasswordHandler;
use SumoCoders\FrameworkMultiUserBundle\DataTransferObject\ChangePasswordDataTransferObject;
use SumoCoders\FrameworkMultiUserBundle\Exception\InvalidPasswordConfirmationException;
use SumoCoders\FrameworkMultiUserBundle\Form\ChangePasswordType;
use SumoCoders\FrameworkMultiUserBundle\Security\PasswordResetToken;
use SumoCoders\FrameworkMultiUserBundle\User\BaseUserRepositoryCollection;
use Symfony\Bundle\FrameworkBundle\Routing\Router;
use Symfony\Bundle\FrameworkBundle\Templating\EngineInterface;
use Symfony\Bundle\FrameworkBundle\Translation\Translator;
use Symfony\Component\Form\FormFactoryInterface;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Session\Flash\FlashBagInterface;

class PasswordResetController
{
    /** @var BaseUserRepositoryCollection */
    private $userRepositoryCollection;

    /** @var ResetPasswordHandler */
    private $resetPasswordHandler;

    /** @var Router */
    private $router;

    /** @var FormFactoryInterface */
    private $formFactory;

    /** @var EngineInterface */
    private $templating;

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

    /** @var FlashBagInterface */
    private $flashBag;

    public function __construct(

This method seems to be duplicated in your project.

        BaseUserRepositoryCollection $userRepositoryCollection,
        ResetPasswordHandler $resetPasswordHandler,
        Router $router,
        FormFactoryInterface $formFactory,
        EngineInterface $templating,
        Translator $translator,
        FlashBagInterface $flashBag
    ) {
        $this->userRepositoryCollection = $userRepositoryCollection;
        $this->resetPasswordHandler = $resetPasswordHandler;
        $this->router = $router;
        $this->formFactory = $formFactory;
        $this->templating = $templating;
        $this->translator = $translator;
        $this->flashBag = $flashBag;
    }

    /**
     * @param Request $request
     * @param PasswordResetToken $token
     *
     * @throws InvalidPasswordConfirmationException
     *
     * @return Response
     */
    public function resetAction(Request $request, PasswordResetToken $token): Response
    {
        $user = $this->userRepositoryCollection->findUserByToken($token);

        if ($user === null) {
            return new RedirectResponse(
                $this->router->generate('multi_user_login')
            );
        }

        $changePasswordTransferObject = ChangePasswordDataTransferObject::forUser($user);
        $form = $this->formFactory->create(ChangePasswordType::class, $changePasswordTransferObject);

        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {

This code seems to be duplicated across your project.

            $this->resetPasswordHandler->handle($form->getData());

            $this->flashBag->add(
                'success',
                $this->translator->trans(
                    'sumocoders.multiuserbundle.flash.password_reset_success'
                )
            );

            return new RedirectResponse(
                $this->router->generate('multi_user_login')
            );
        }

        return $this->templating->renderResponse(
            'SumoCodersFrameworkMultiUserBundle:PasswordReset:reset.html.twig',
            [
                'form' => $form->createView(),
            ]
        );
    }
}