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

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

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

Сообщение samdark »

Это и есть то, что я сказал, только другими словами.
skynin
Сообщения: 400
Зарегистрирован: 2017.12.12, 10:09

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

Сообщение skynin »

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

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

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

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

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

Сообщение zelenin »

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

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

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

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

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

Сообщение samdark »

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