2.1 Убить валидацию и фильтры в AR

Уже исправленные репорты или принятые предложения
Аватара пользователя
samdark
Администратор
Сообщения: 9172
Зарегистрирован: 2009.04.02, 13:46
Откуда: Воронеж
Контактная информация:

Re: 2.1 Убить валидацию и фильтры в AR

Сообщение samdark » 2018.02.05, 23:27

Это и есть то, что я сказал, только другими словами.

skynin
Сообщения: 186
Зарегистрирован: 2017.12.12, 10:09

Re: 2.1 Убить валидацию и фильтры в AR

Сообщение skynin » 2018.02.06, 14:47

Данные в AR попадают не только из формы.
Модификация AR происходит не только по действию с UI.

Поэтому валидацию нужно оставить.

А вот пихать или не пихать в валидацию AR бизнес-логику - так это программист пусть и решает. Я считаю что ей там не место. в AR должно быть поменьше бизнес-логики.

Хотите DDD - делайте отдельные модели. AR паттерн не для того. А для удобной работы с персистентными данными.
Соответственно и валидация данных должна в AR должна быть только на предмет целостности, консистентности, а не
Можно ли увеличить оклад сотруднику?
Никакого "оклада" в AR быть не должно. А только - значение поля salary не может меньше X и больше Y
Неврубающийся не может опознать врубающегося.

zelenin
Сообщения: 10596
Зарегистрирован: 2013.04.20, 11:30

Re: 2.1 Убить валидацию и фильтры в AR

Сообщение zelenin » 2018.02.06, 14:54

skynin писал(а):
2018.02.06, 14:47
Данные в AR попадают не только из формы.
Модификация AR происходит не только по действию с UI.

Поэтому валидацию нужно оставить.

А вот пихать или не пихать в валидацию AR бизнес-логику - так это программист пусть и решает. Я считаю что ей там не место. в AR должно быть поменьше бизнес-логики.

Хотите DDD - делайте отдельные модели. AR паттерн не для того. А для удобной работы с персистентными данными.
Соответственно и валидация данных должна в AR должна быть только на предмет целостности, консистентности, а не
Можно ли увеличить оклад сотруднику?
Никакого "оклада" в AR быть не должно. А только - значение поля salary не может меньше X и больше Y
в общем-то в этой ветке это и написано - отдельно бизнес-валидация, внутренняя валидация по схеме.

Аватара пользователя
samdark
Администратор
Сообщения: 9172
Зарегистрирован: 2009.04.02, 13:46
Откуда: Воронеж
Контактная информация:

Re: 2.1 Убить валидацию и фильтры в AR

Сообщение samdark » 2018.02.07, 00:23

Именно так. Выпиливать не вижу смысла.

Ответить