php - Symfony 5 controller won't validate errors

I am trying to migrate a some parts of a project made with Typescript + Express + Firebase to Symfony 5 and MySQL. But I don't get why the ValidatorInterface isn't working.

When I submit the form with unexpected characters (i.e.: a password with 3 charactesrs) it creates the user despites the validation constraints assigned in theUser entity. The only constraint validation that actually works despites it does not show any form errors, is the UniqueEntity.

This is how my controller looks like:

    /**
     * GET: Display form
     * POST: Creates a user
     * 
     * @Route("/users/crear", name="create_user", methods={"GET", "POST"})
     */
    public function create_user(Request $request, EntityManagerInterface $entityManager, ValidatorInterface $validator)
    {
        $user = new User();

        // Form
        if ($request->isMethod('GET')) {
            $form = $this->createForm(CrearUserType::class, $user);
            return $this->render('user/create.html.twig', [
                'form' => $form->createView(),
                'errors' => []
            ]);
        }

        // Form submit
        $data = $request->request->get('create_user');
        $errors = [];

        $user->setName($data['name']);
        $user->setEmail($data['email']);
        $user->setPassword($data['password']);
        $user->setCreatedAt(new \DateTime());
        $user->setUpdatedAt(new \DateTime());

        $form = $this->createForm(CrearUserType::class, $user);

        // Entity validation
        $errors = $validator->validate($user);        

        if (count($errors) > 0) { // This is always 0
            $user->setPassword('');
            return $this->render('user/create.html.twig', [
                'form' => $form->createView(),
                'user' => $user,
                'errors' => $errors
            ]);
        }

        try {
            $user->setPassword(password_hash($user->getPassword(), PASSWORD_BCRYPT));
            $entityManager->persist($user);
            $entityManager->flush();
            $this->addFlash(
                'success', 
                'User '.$user->getName().' has been saved.'
            );
            return $this->redirectToRoute('users');
        } catch(\Exception $exception) {
            $user->setPassword('');
            $this->addFlash(
                'error', 
                'Server error: ' . $exception->getMessage()
            );
            return $this->render('user/create.html.twig', [
                'form' => $form->createView(),
                'user' => $user,
                'errors' => $errors
            ]);
        }
    }
Copy code

And this is my entity:

namespace App\Entity;

use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Validator\Constraints as Assert;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;

/**
 * @ORM\Entity(repositoryClass="App\Repository\UserRepository")
 * @UniqueEntity("email")
 */
class User
{
    /**
     * @ORM\Id()
     * @ORM\GeneratedValue()
     * @ORM\Column(type="integer")
     */
    private $id;

    /**
     * @ORM\Column(type="string", length=255)
     * @Assert\NotBlank(message="Name is mandatory")
     */
    private $name;

    /**
     * @ORM\Column(type="string", length=255, unique=true)
     * @Assert\NotBlank(message="Email is mandatory")
     * @Assert\Email(
     *  message="Invalid email address"
     * )
     */
    private $email;
    /**
     * @ORM\Column(type="string", length=60)
     * @Assert\NotBlank(message="Password is mandatory")
     * @Assert\GreaterThanOrEqual(
     *  value=6,
     *  message="The password has to be at least 6 chars long"
     * )
     */
    private $password;

    /**
     * @ORM\Column(type="date")
     */
    private $createdAt;

    /**
     * @ORM\Column(type="date")
     */
    private $updatedAt;

    public function getId(): ?int
    {
        return $this->id;
    }

    public function getName(): ?string
    {
        return $this->name;
    }

    public function setName(string $name): self
    {
        $this->name = $name;

        return $this;
    }

    public function getEmail(): ?string
    {
        return $this->email;
    }

    public function setEmail(string $email): self
    {
        $this->email = $email;

        return $this;
    }

    public function getPassword(): ?string
    {
        return $this->password;
    }

    public function setPassword(?string $password): self
    {
        $this->password = $password;

        return $this;
    }

    public function getCreatedAt(): ?\DateTimeInterface
    {
        return $this->createdAt;
    }

    public function setCreatedAt(\DateTimeInterface $createdAt): self
    {
        $this->createdAt = $createdAt;

        return $this;
    }

    public function getUpdatedAt(): ?\DateTimeInterface
    {
        return $this->updatedAt;
    }

    public function setUpdatedAt(\DateTimeInterface $updatedAt): self
    {
        $this->updatedAt = $updatedAt;

        return $this;
    }
}
Copy code

So, any clues on what is wrong here?

Didn't find the answer?

Our community is visited by hundreds of web development professionals every day. Ask your question and get a quick answer for free.