В
Это способ агрегировать эвенты и затем все разом отправить в диспетчер
Size: a a a
2020 November 06
СП
Диспетчер будет в конструкторе. А этот сервис - не диспетчер
Если он будет диспетчер - мы сможем его засовывать куда угодно, где есть PSR интерфейс :)
СП
То есть мы можем взять либу какую-то, которая работает с psr-интерфейсом диспетчера и скормить ей наш deffered дисппетчер. А потом события зарелизить.
В
Если он будет диспетчер - мы сможем его засовывать куда угодно, где есть PSR интерфейс :)
В этом как раз и была проблема в PR, который сделал @samdark. Сложно получается ими управлять.
AM
На мой взгляд, в 90% случаев сервис подойдет. А там, где по каким-то причинам класс имеет зависимости и не создается через фабрики/контейнеры, придется трейт юзать.
Да не подойдёт. Где такие события нужны - это внутренние события агрегата.
СП
AM
А они не должны торчать наружу.
AB
Мнекажется что трейт, что сервис реализуется, во-первых, просто, во-вторых, в конкретном проекте отдельно. Тянуть это в библиотеку сомнительно
AM
AM
потому что он по определению не может работать с событиями, которые могут мутировать состояние
AM
а мы не можем запретить такие юзать
AM
ну и реализация, если хочется, очень простая
AM
я бы не заморачивался им (уже позаморачивались)
В
предлагаешь и трейт не добавлять в пакет?
AM
В
@predvoditelev, ну давай позакрываем тогда
СП
Давай)
СП
Закрыл тикет и PR