SM
т.е. это случай когда ряд дебильных, но привычных в симфони среде решений (юзать апи платформу, иметь не валидные сущности, шо-то хуячить в терминейте), вместе приводят к пиздецу. хотя каждое из них кажется не таким уж и вредным.
Size: a a a
2020 May 12
M
был случай. юзаем мы апиплатформу. делаем пост. сущность меняется. валидируется. не валидна. отправляем 400. еще был листенер, кот. че-то флашил на терминейте после долгих сложных вычислений, ну а иногда не флашил, зависит от контектса. ну и потом баг, что вот 400 и все сохранилось. чтоб починить пришлось прилично порефакторить.
Так ексепшен должен вылетать и не должно быть дальше флаша никакого
SM
флаш в терминейте, а ексепшн пойман и обработан ексепшн лисенером.
M
Ну флаш в терминейте это очевидно что делать не надо, как мне кажется
SM
как и не валидные сущности
M
как и не валидные сущности
А конкретнее, нельзя флашить невалидные сущности )
SM
а откуда ты знаешь что там в ем? $em->flush() - там валидные или не валидные сущности?
M
Даже с валидной сущностью могла быть проблема, что вылетел какой то эксепшен, допустим, и сущность сохранять не нужно. Но флаш в терминейте ее все равно сохранит.
M
а откуда ты знаешь что там в ем? $em->flush() - там валидные или не валидные сущности?
If validator->valid(entity) em->flush()
АС
а откуда ты знаешь что там в ем? $em->flush() - там валидные или не валидные сущности?
Так это легко. Невалидная сущность просто не должна существовать по определению.
АС
В чем проблема-то?
АС
Как дети, блин ))
M
Так это легко. Невалидная сущность просто не должна существовать по определению.
Окей, сущность валидная, но пришел от юзера месседж, что новые данные сущности сохранять не надо. При обработке месседжа ты забыл вернуть сущности первоначальные данные, случайно зафлашил ее. А юзер ожидает, что данные не зафлашены. В итоге, для него она не валидна получается )
АС
набор слов
M
набор слов
Если их мозгом их не обрабатывать, то да
АС
не надо сохранять - не сохраняй
SM
Так это легко. Невалидная сущность просто не должна существовать по определению.
так я ж и говорю
АС
надо откатиться на предыдущий стейт - напиши метод отката
АС
Окей, сущность валидная, но пришел от юзера месседж, что новые данные сущности сохранять не надо. При обработке месседжа ты забыл вернуть сущности первоначальные данные, случайно зафлашил ее. А юзер ожидает, что данные не зафлашены. В итоге, для него она не валидна получается )
а это бессмысленный набор слов
M
надо откатиться на предыдущий стейт - напиши метод отката
Не надо сохранять невалидную сущность - не сохраняй