input and output might not be the same (closes #825)

mojolicious · Aug 7, 2015 · 523ed7a · 523ed7a
1 parent 6a51c3e
commit 523ed7a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/lib/Mojolicious/Validator/Validation.pm b/lib/Mojolicious/Validator/Validation.pm
@@ -23,10 +23,10 @@ sub check {
 
   return $self unless $self->is_valid;
 
-  my $cb    = $self->validator->checks->{$check};
-  my $name  = $self->topic;
-  my $input = $self->input->{$name};
-  for my $value (ref $input eq 'ARRAY' ? @$input : $input) {
+  my $cb     = $self->validator->checks->{$check};
+  my $name   = $self->topic;
+  my $values = $self->output->{$name};
+  for my $value (ref $values eq 'ARRAY' ? @$values : $values) {
     next unless my $result = $self->$cb($name, $value, @_);
     return $self->error($name => [$check, $result, @_]);
   }

diff --git a/t/mojolicious/validation_lite_app.t b/t/mojolicious/validation_lite_app.t
@@ -160,6 +160,7 @@ ok !$validation->has_data, 'no data';
 $validation->input({foo => ['', 'bar', ''], bar => ['', 'baz', '']});
 ok $validation->has_data, 'has data';
 ok $validation->optional('bar')->is_valid, 'valid';
+ok $validation->in('baz')->is_valid,       'still valid';
 is_deeply $validation->output, {bar => 'baz'}, 'right result';
 ok !$validation->has_error, 'no error';
 ok !$validation->required('foo')->is_valid, 'not valid';