Obblm\Core\Controller\AccountController::editAccount()

Complexity

Conditions	4
Paths	3

Size

Total Lines	22
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    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(),
        ]);