#93 #33
#93 #33
Conversation
| options.required && IsNotEmpty({ | ||
| message: 'Обязательно для заполнения', | ||
| }), |
There was a problem hiding this comment.
Я правильно понимаю что текст "Обязательно для заполнения" больше ни в каком слу 10000 ае не будет выводиться?
There was a problem hiding this comment.
То есть мы теряем часть бизнес-логики при таком изменении? Чем она будет замещаться?
А если эта логика лишняя, то прошу пояснить почему именно.
There was a problem hiding this comment.
Будет стандартное сообщение из валидаторов class-validator
Но если нужно какое-то кастомное сообщение, то по логике нужно такое добавлять к каждому валидатору из class-validator
There was a problem hiding this comment.
А каким может быть решение, при котором сообщения об ошибках сохранятся в проектах, в которых люди стероиды обновят?
| @@ -54,17 +53,13 @@ export function RelationIdField(options: IRelationIdFieldOptions = {}) { | |||
| options.nullable = true; | |||
| } | |||
|
|
|||
| const arrayNotEmptyMessage = options.isFieldValidConstraintMessage || 'Не должно быть пустым'; | |||
There was a problem hiding this comment.
Я правильно понимаю что этот текст ошибки нигде не будет выводиться?
There was a problem hiding this comment.
То есть мы теряем часть бизнес-логики при таком изменении? Чем она будет замещаться?
А если эта логика лишняя, то прошу пояснить почему именно.
There was a problem hiding this comment.
Выше ответил + я не понимаю, почему свойство options.isFieldValidConstraintMessage отвечает именно за текст ошибки при пустом массиве и находится только в RelationIdField
There was a problem hiding this comment.
я не понимаю, почему свойство options.isFieldValidConstraintMessage отвечает именно за текст ошибки при пустом массиве и находится только в RelationIdField
Думаю стоит найти ответ на этот вопрос, в чатах или на мите.
There was a problem hiding this comment.
почему свойство options.isFieldValidConstraintMessage отвечает именно за текст ошибки при пустом массиве и находится только в RelationIdField
Получилось найти ответ на этот вопрос?
| @@ -21,7 +21,7 @@ export function IntegerField(options: IIntegerFieldOptions = {}) { | |||
| appType: 'integer', | |||
| jsType: 'number', | |||
| }), | |||
| options.nullable && ValidateIf((object, value) => options.isArray ? !isArrayEmpty(value) : !isEmpty(value)), | |||
| !options.required && ValidateIf((object, value) => options.isArray ? !isArrayEmpty(value) : !isEmpty(value)), | |||
There was a problem hiding this comment.
Почему везде такая логика убрана, а здесь оставлена?
|
Нужно проверить работу валидатора для RelationField |
Исправлена логика required/nullable в field декораторах