do not remove optional empty values

vti · Jul 22, 2017 · b7a1bf0 · b7a1bf0
1 parent ebf2d6d
commit b7a1bf0
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 28 deletions.
diff --git a/lib/Tu/Validator.pm b/lib/Tu/Validator.pm
@@ -161,10 +161,6 @@ sub _gather_validated_params {
     foreach my $name (keys %{$self->{fields}}) {
         next if exists $result->{errors}->{$name};
 
-        next
-          if !$self->{fields}->{$name}->{required}
-          && $self->_is_field_empty($result->{params}->{$name});
-
         $validated_params->{$name} = $result->{params}->{$name};
     }
 

diff --git a/t/validator.t b/t/validator.t
@@ -290,30 +290,6 @@ subtest 'no errors when field is optional' => sub {
     ok $result->is_success;
 };
 
-subtest 'removes empty optional field' => sub {
-    my $validator = _build_validator();
-
-    $validator->add_optional_field('foo');
-    $validator->add_rule('foo', 'regexp', qr/^\d+$/);
-
-    my $result = $validator->validate({foo => ''});
-
-    is_deeply $result->all_params, {foo => ''};
-    is_deeply $result->validated_params, {};
-};
-
-subtest 'removes optional multiple empty values' => sub {
-    my $validator = _build_validator();
-
-    $validator->add_optional_field('foo', multiple => 1);
-    $validator->add_rule('foo', 'regexp', qr/^\d+$/);
-
-    my $result = $validator->validate({foo => ['', '']});
-
-    is_deeply $result->all_params, {foo => ['', '']};
-    is_deeply $result->validated_params, {};
-};
-
 subtest 'set default message' => sub {
     my $validator = _build_validator();