Skip Menu |

This queue is for tickets about the HTML-FormFu CPAN distribution.

Report information
The Basics
Id: 76034
Status: resolved
Priority: 0/
Queue: HTML-FormFu

People
Owner: Nobody in particular
Requestors: felix.ostmann [...] thewar.de
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.09007
Fixed in: 2.02



Subject: When a Inflator croak, userinput goes away
I am not sure, but i think this is a bug. When a Inflator croaks, the userinput is gone away and the field is empty. My patch fixed that. All tests pass. I am not sure, if this behavior is also desired for transformers, never used one before.
Subject: HTML-FormFu-0.09007.errorvalue.patch
Nur in HTML-FormFu-0.09007.patched/: blib. diff -ur HTML-FormFu-0.09007/lib/HTML/FormFu/Inflator.pm HTML-FormFu-0.09007.patched//lib/HTML/FormFu/Inflator.pm --- HTML-FormFu-0.09007/lib/HTML/FormFu/Inflator.pm 2011-05-10 12:29:36.000000000 +0200 +++ HTML-FormFu-0.09007.patched//lib/HTML/FormFu/Inflator.pm 2012-03-25 23:12:13.640499881 +0200 @@ -20,7 +20,7 @@ if ($@) { push @errors, $self->return_error($@); - push @return, undef; + push @return, $value; } else { push @return, $return; @@ -33,6 +33,7 @@ if ($@) { push @errors, $self->return_error($@); + $return = $values; } }
On Sun Mar 25 17:26:56 2012, Sadrak wrote: Show quoted text
> I am not sure, but i think this is a bug. When a Inflator croaks, the > userinput is gone away and the field is empty. > > My patch fixed that. All tests pass. I am not sure, if this behavior is > also desired for transformers, never used one before.
This has been applied in Github.