"Если свойство помечено как nullable и не заполнено, валидация пропускает его."

[cyber-jet]

Page link: https://docs.1c-bitrix.ru/pages/framework/validation.html

Каким образом сделать опциональное поле, в документации об этом ни чего нет? Помеченные null свойства всё равно проходят(с ошибкой) проверку.

final class UpdateUserDto
{
    public function __construct(
        #[Length(min: 2)]
        public ?string $name = null,
        #[Email]
        public ?string $email = null,
    ) {}

    public static function createFromRequest(\Bitrix\Main\HttpRequest $request): self
    {
        $json = $request->getJsonList();

        return new self(
            name: $json['name'] ?? null,
            email: $json['email'] ?? null
        );
    }
}

[gromdron]

К сожалению в указанной вами схеме никак не сделать.

Дело в том как именно определяется nullable поле в системе: при определении битрикс опирается на свойство рефлексии isInitialized, которое в случае явной передачи null-параметра возвращается как true что трактуется заполненным.

При прохождении вашего UpdateUserDto::createFromRequest вы явно его заполняете null'ом, поэтому вы непременно зайдете в Length-валидатор и получите ошибку.
Когда в своей разработке я наткнулся на эту же ошибку, я полез изучать данное поведение и мне пришлось сделать свое AtLeastOne-правило и передавать Empty и Length.

Об этом поведении я написал в PR #62, но оно к сожалению в релиз не вошло.
Подробнее тут.

[cyber-jet]

Дело в том как именно определяется nullable поле в системе: при определении битрикс опирается на свойство рефлексии isInitialized, которое в случае явной передачи null-параметра возвращается как true что трактуется заполненным.

Пробовал по-разному, для проверки комментил //email: $json['email'] ?? null, пробовал так email: $json['email'], т.е. будет ошибка даже если не передать, в итоге написал свой валидатор отдельной функцией , выглядит конечно не так красиво как атрибуты, свой атрибут пока не разобрался как писать, что-то типа #[OptionalEmail] или #[Optional] в комбинации с #[Email] решило бы задачу.

        return new self(
            name: self::validateName($json['name']),
            email: self::validateEmail($json['email'])
        );

Дело в том что по логике фронт может отправить любое из этих полей только если оно изменилось, даже если это будет пустая строка, чтобы стереть это поле из базы. Т.е. три возможных варианта которые должны проходить проверку - null, 'mail@example.ru' или ''.

[neckita39]

Действительно, сейчас поле не может быть опционально отвалидировано.
Эта проблема решена в main 26.100.0 с введением групп валидации. Документация будет дополнена.

		$item = new class {
			#[ElementsType(Type::Integer, groups: ['create'])]
			private array $var = ['not', 'integers'];
		};

		$result = $this->getService()->validate($item, 'create');
		$this->assertFalse($result->isSuccess());

		$result = $this->getService()->validate($item, 'delete');
		$this->assertTrue($result->isSuccess());

[gromdron]

Эта проблема решена в main 26.100.0 с введением групп валидации. Документация будет дополнена.

@neckita39, @irpsv понимаю что тикет закрыт, но мне кажется создание "групп" валидаторов является архитектурной ошибкой.

На мой взгляд есть пара простых ходов, которые решают эту проблему:

1. Изменение подхода к валидации.
   После проверки на isInitialized в валидаторе, сделать проверку получение типа и проверку на allowNullable. Если тип позволяет быть nullable и имеет значение null, то пропускать валидацию.

2. Следует расширить список базовых правил, добавив IsNull и AtLeastOneOf

   Если бы в продукте существовали аналоги Symfony'вских "AtLeastOneOf" и "IsNull" правил, то указанный код легко мог бы быть переписан с использованием:

   final class UpdateUserDto
   {
       public function __construct(
           #[AtLeastOneOf([
               new Length(min: 2),
               new IsNull(),
           ])]
           public ?string $name = null,
           #[AtLeastOneOf([
               new Email(),
               new IsNull(),
           ])]
           public ?string $email = null,
       ) {}
   
       public static function createFromRequest(\Bitrix\Main\HttpRequest $request): self
       {
           $json = $request->getJsonList();
   
           return new self(
               name: $json['name'] ?? null,
               email: $json['email'] ?? null
           );
       }
   }