Павел
Ну короче какждый хочет так и делает, тут критикала ноль
The Ant
ну да, 2 интерфейса, 2 реализации и 1 аутентификатор. Нахрен бандл не понимаю
Павел
Vlad
Вот что мне не нравится в лексике, так это то что деграеться каждый раз ключи
Vlad
ну то есть в чем проблема сразу заинжектить их
The Ant
Vlad
Vlad
его профит не тащим вендор
The Ant
профит в том, что тебе не надо следить за этой зависимостью
Vlad
такой себе профит
The Ant
для тебя может это и не профит совсем конечно
Павел
Vlad
вроде не мобильное приложение пишем)
Павел
Я ее использую в 10 строчках модификации
The Ant
за тем что в нее могут напихать всякого )
The Ant
а ты и не заметишь
Павел
Vlad
Vlad
секурити чтоли?
The Ant
Юра
Проблема в самописе в том что никто кроме тебя не шарит в этом самописе
Юра
Новый проект, стоит jwt bundle, сразу понимаешь что куда
Vlad
Vlad
тулзы вроде есть
The Ant
или свое юзать )
Павел
The Ant
и исключить лишний гемор ))
Павел
Все просто
Vlad
и не юзать пхп
Vlad
мало ли
Vlad
что то подсунут
The Ant
вот да, пхп тоже ломали ))
Юра
Свое обычно полная хрень, в плане никто на свое не пишет доку, тесты, потом разраб ушел и проект превратился в кучу самописа без доки, считай проект помер
Юра
А если ты написал доку и тесты, то можно и поделиться с миром, но в мире уже есть джвт бандл
The Ant
что то подсунут
пропустил наверное бугурт, когда с подписью попова эксплойт в пхп засунули )
Юра
The Ant
🤣🤣
Юра
В нем будут только твои эксплоиты )
Павел
Всем привет.
Сейчас вроде симфа предлагает такие методы репо в мейкере.
public function save(User $entity, bool $flush = false): void
{
$this->getEntityManager()->persist($entity);
if ($flush) {
$this->getEntityManager()->flush();
}
}
Мне кажется, или этот bool какой то треш?
Shokha
Shokha
вы можете сразу сохранить после персиста или оставить это на потом
Юра
Вообще треш. Как по мне репозиторий не должен флашить
Null
А в умных книжках говорят, что буль флаги в методах - говно
Павел
вы можете сразу сохранить после персиста или оставить это на потом
То что он делает, я понимаю. Это архитектурный треш, я про это.
1.bool - в аргементах всегда говорит о разбитие на 2 метода
2. Контракт репозитория, имеет какой то flush (представим что мы не знаем что это UoF а видим только контракт), что это вообще такое. Save и какой то другой save который не save, но в тоже время save
Юра
Это слишком важная фция чтобы она где-то там внутрях вызвалась сама собой
Юра
А еще круче когда внутри еще и где-то clear вызывают
Юра
А ты сидишь три часа и дебажишь
Павел
Ну либо отдельный сервис
Павел
Я бы еще понял, если бы внутри был тупо всегда персист и флаш, но в таком виде - не понимаю.
Юра
Это все попахивает актив рекордом
Юра
Следующий шаг не инджектить вообще EM
Павел
Ну это я так)
Юра
Это все похоже на User::save()
Юра
Разве нет?
Юра
Мы берем центральрую точку входу на БД и оаспихиваем её по всем местам
Юра
Ок теперь вместо отдельной транзакции, будет куча флашей
Юра
Каждвй со своей транзакцией
Павел
Запутался что обсуждаем
Юра
И ты слабо это контролируешь потому что кто-то с флагом вызвает кто-то без флага и т.д.
Юра
Павел
Юра
Короче репозиторий теперь становится чем-то что само решает флашить емк или нет, мне это не нравится ) мое мнение
Павел
Флаш плохо
Это холивар) Например если говорим об агрегатах, там вообще 1 сейв - 1 транзакция дожна быть)
Опять таки, если не UoW в системе, то без save в репозитории не обойтись. Нет коллектора изменений. И отсуствтие UoW не равно актив рекорд. Дата маппер то остался.
Плюс контракт флашера спорный, что это вообще такое)))
Юра
Точнее решает не само но теперь оно МОЖЕТ флашить, и это затрудняет дебаг и все такое
Павел
Юра
Вызывай флаш и оно сохранит все либо ничего