Obblm\Core\Controller\AccountController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	editAccount()	0	22	4

<?php

namespace Obblm\Core\Controller;

use Obblm\Core\Form\Coach\EditUserForm;
use Obblm\Core\Security\Roles;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\Security\Core\Encoder\UserPasswordEncoderInterface;

/**
 * @Route("/account", name="obblm_account")
 */
class AccountController extends AbstractController
{
    /**
     * @Route("/", name="")
     */
    public function editAccount(Request $request, UserPasswordEncoderInterface $passwordEncoder):Response
    {
        $coach = $this->getUser();
        $this->denyAccessUnlessGranted(Roles::COACH, $coach);

        $form = $this->createForm(EditUserForm::class, $this->getUser());

        $form->handleRequest($request);
        if ($form->isSubmitted() && $form->isValid()) {
            $data = $form->getData();
            if ($data->getPlainPassword()) {
                $password = $passwordEncoder->encodePassword($coach, $coach->getPlainPassword());

It seems like $coach can also be of type null; however, parameter $user of Symfony\Component\Security\Core\Encoder\UserPasswordEncoderInterface::encodePassword() does only seem to accept Symfony\Component\Security\Core\User\UserInterface, maybe add an additional type check?

                $coach->setPassword($password)
                    ->setHash(hash('sha256', $coach->getEmail()));
            }
            $entityManager = $this->getDoctrine()->getManager();
            $entityManager->persist($coach);
            $entityManager->flush();
        }

        return $this->render('@ObblmCore/account/edit.html.twig', [
            'form' => $form->createView(),
        ]);
    }
}