Symfony. Как сделать валидацию с зависимостью от значений св-в сущности?

Question

[troublegum]

Всем доброго времени суток!
Столкнулся с такой проблемой, что надо сделать валидацию св-в сущности Post с определенной бизнес логикой.

Что есть:
- валидация через аннотации
- сущность Post (публикация) с признаком публикации и датой публикации
- дата публикации обязательна и должна быть больше или равно текущий дате, если выставлен признак публикации
- если признак публикации снят, то поле дата публикации никак не валидируем

Вопрос! Можно ли как то сделать такую валидацию без написания собственного валидатора поля "дата публикации"?

class Post
{
	/**
     * @Assert\NotNull
     */
	public bool $published;
	
    /**
     * Как написать правила валидации для этого св-ва?
     * @Assert\DateTime
     */
	public \DateTime $publishedAt;
}

Comments

[BoShurik]

Используются symfony/form?

Answer 1

[Максим]

На все такие бизнес требования невозможно написать валидаторы, можно исхитриться, но тогда уйдёт бизнес правило, которое еще и протестировать сложно.

Поэтому, что я вам посоветую. Поместить эту бизнес логику внутрь сущности в специальный метод. Например published(): void. Пример из кода.

Comments

[tukreb]

А чего ваш код так похож на https://github.com/ElisDN/demo-project-manager ? :) Ещё и автором в некоторые Классы себя пометили :) без указания первоисточника вдохновения.

[troublegum]

Максим спасибо за ответ!

Но проблема немного шире. Есть форма редактирования сущности Post с десятком полей, среди которых есть чекбокс опубликовать и поле "дата публикации" и мы должны показать пользователю ошибку в момент попытки сохранить данные. Я не совсем понимаю как показать ошибку типа вы выбрали чекбокс опубликовать, но не заполнили поле дата публикации... т.е. если я правильно понял вашу мысль, то код должен быть каким-то вот таким:

<?php

class PostController
{
	public function save($request)
	{
		$post = new Post();
        $form = $this->createForm(Post::class, $post);
		if ($form->isSubmitted() && $form->isValid()) {
			try {
				$post = $form->getData();
				if ($post->published) {				
					$post->publish();
				}
				// далее код сохранения данных в БД
			} catch (DomainLogin $e) {
				// тут ловим исключение если есть ошибка в бизнес-логике
				// тут как-то пытаемся добавить ошибку в форму
			}
		}
		
		return $this->render('post/form.html.twig', [
            'form' => $form->createView(),
        ]);
	}
}

я верно понял вашу мысль? )

[Максим]

tukreb, это и есть Елисеева практически код) И Елисеев про него знает не переживай)))

[Максим]

troublegum, да)) так и есть. Только делать проверку с published не нужно.

[troublegum]

Максим, а как же понять, что пост требует публикации или наоборот? Ведь пост может быть сохранен как опубликованный и как не опубликованный. Метод publish отвечает за публикацию и вызывать его когда пост не опубликован не корректно.

[Максим]

troublegum,
Если у вас нет промеждуточного слоя в контроллере, то так.

class PostController
{
  public function save($request)
  {
    $post = new Post();
        $form = $this->createForm(Post::class, $post);
    if ($form->isSubmitted() && $form->isValid()) {
      try {
        $post = $form->getData();			
        $post->publish();
        $post->flush();
      } catch (DomainLogin $e) {
        // тут ловим исключение если есть ошибка в бизнес-логике
        // тут как-то пытаемся добавить ошибку в форму
      }
    }
    
    return $this->render('post/form.html.twig', [
            'form' => $form->createView(),
        ]);
  }
}

public function isCanBePublish(): bool
    {
        return $this->status === self::STATUS_PUBLISHED
    }

    public function publish(DatitimeImmutable $date): void
    {
         if ($this->isCanBePublish()) { 
              trow new DomainExeption('Нельзя опубликовать');
        }
        $this->status = self::STATUS_PUBLISHED
        $this->publish_date = date
    }