Symfony 5 контроллер не будет проверять ошибки - PullRequest
1 голос
/ 26 апреля 2020

Я пытаюсь перенести некоторые части проекта, созданного с помощью Typescript + Express + Firebase, на Symfony 5 и MySQL. Но я не понимаю, почему ValidatorInterface не работает.

Когда я отправляю форму с неожиданными символами (например, пароль с 3 символами), он создает пользователя, презирающего проверочные ограничения , назначенные в сущности User. Единственная проверка ограничений, которая действительно работает, несмотря на то, что она не показывает никаких ошибок формы, это UniqueEntity .

Вот так выглядит мой контроллер:

    /**
     * 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
            ]);
        }
    }

И это моя сущность:

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;
    }
}

Итак, есть какие-нибудь подсказки, что здесь не так?

1 Ответ

1 голос
/ 26 апреля 2020

Я не уверен в своем ответе, потому что я никогда не обрабатываю запрос с ValidatorInterface. Я использую метод handlerequest, предоставленный контроллером.

Вы создали форму

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

Но вы забыли обработать запрос

 $form->handleRequest($request);

Таким образом, ваша форма не проверяет данные отправлено по запросу. Попробуйте добавить эту строку сразу после создания формы. Затем вы можете проверить, что пользовательский объект действителен с

if ($form->isValid()) {
 //no errors
}
//error exists
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...