Хорошая ли практика привязываться к ивентам Doctrine?

Question

[Игорь Катков]

Добрый день!
Созрел вопрос. К примеру, есть задача создавать логи (или отправлять имейлы) в момент создания сущности User. Мы принимаем решение реализовать это через ивенты. Есть два варианта:
1) Слушать Doctrine:prePersist и вызвать соответсвующий сервис.
2) Создать собственный ивент:

$this->userRepository->save($user);
$this->eventDispatcher->dispatch(SomeEvent::EVENT, new UserEvent($user));

Какой вариант более предпочтительный? Что в этом случае будет BestPractice? Хорошо ли в целом слушать ивенты Doctrine или искать другие возможности?

Answer 1

[Andrey Kulikovsky]

Для таких целей лучше использовать события. А лучше domain-event.
Пример тут: https://github.com/fesor/domain-events

Comments

[yuras666]

Можешь поподробнее расписать про реализацию domain-event в symfony. Скажем имеет ли право Entity сам кидать событие UserRegistered()? Может ли Entity подписаться на доктриновские события postPersist, а затем кинуть UserRegistered()? Из примера по ссылке я не совсем понял где и когда вызывать releaseEvents() и последующий dispatch($events). Я ведь не могу вызывать это по событию kernel.response для всех запросов.
Либо же события UserRegistered вообще лучше кидать из класса userRepository после метода save. В таком случаи не проще ли использовать стандартный symfony - вский event dispatcher.

[Andrey Kulikovsky]

Доктрина реализует Unit of work, поэтому мы можем удобно выгрузить все измененные сущности, получить из них события и выбросить событие например через симфонийский "@event_dispatcher". Я обычно внедряю EventListener слушающий "postFlush"

services:
    infrastructure.doctrine_domain_events_handler:
        class: Infrastructure\BlaBlaNamespace\DomainEvent\DoctrineDomainEventsHandler
        arguments: ["@event_dispatcher"]
        tags:
            - { name: doctrine.event_listener, event: postFlush }

Упрощенный код:

class DoctrineDomainEventsHandler
{

    /**
     * @var EventDispatcherInterface
     */
    private $dispatcher;

   //....

    public function postFlush(PostFlushEventArgs $args)
    {
        $em = $args->getEntityManager();

        $identityMap = $em->getUnitOfWork()->getIdentityMap();

        foreach ($identityMap as $entities) {

            foreach ($entities as $entity) {
                if ($entity instanceof DomainEventsAwareInterface) {

                    $events = $entity->releaseEvents();
                    foreach ($events as $event) {
                        $this->dispatcher->dispatch($event->getName(), $event);
                    }

                }
            }
        }
    }
}

[Andrey Kulikovsky]

В итоге события пробрасываются в "@event_dispatcher", а дальше ловим их как обычные события, там все стандартно.

[yuras666]

В целом все логично, спасибо )

Answer 2

[shagguboy]

доктриновские ивенты сделаны как раз для того чтобы их слушали.