Bug #39871 for HTML-FormFu:

Tue Oct 07 16:50:21 2008 rod.taylor [...] gmail.com - Ticket created

Chekbox defaults seem to require both $field->value() and $field->default() to be set. LIKE THIS: my $fk = $form->get_field($formKey); $fk->default($measurementValue); $fk->value($measurementValue); The offending code appears to be in HTML::FormFu::Element::Checkbox at line 57. my $default = $self->default; my $original = $self->value; . . . 57| elsif ( defined $default && defined $original && $default eq $original ) { $render->{attributes}{checked} = 'checked'; } Here you can see if looks for both $default and $original to be defined even when the form has not been submitted. It should probably be something like (defined $default || defined $original).

Fri Oct 31 15:18:40 2008 cpan [...] fireartist.com - Correspondence added

Hi, Can you check the documented behaviour [1], and then describe what you want to happen, as I'm not sure what you're reporting as being wrong. [1] http://search.cpan.org/~cfranks/HTML-FormFu-0.03005/lib/HTML/FormFu/Element/_Field.pm#value Cheers, Carl

Fri Oct 31 15:18:42 2008 The RT System itself - Status changed from 'new' to 'open'

Fri Oct 31 15:18:43 2008 cpan [...] fireartist.com - Status changed from 'open' to 'stalled'

Mon Nov 03 15:33:16 2008 rod.taylor [...] gmail.com - Correspondence added

From:

rod.taylor [...] gmail.com

On Fri Oct 31 15:18:40 2008, CFRANKS wrote: Show quoted text

> Hi, > > Can you check the documented behaviour [1], and then describe what you > want to happen, as I'm not sure what you're reporting as being wrong. > > [1] > http://search.cpan.org/~cfranks/HTML-FormFu- > 0.03005/lib/HTML/FormFu/Element/_Field.pm#value > > Cheers, > Carl

Technically I suppose it works as documented. To get a checkbox checked I need to do this: $field->value($value); $field->default($value); Both $value and $default were undef since that is how FormFu initializes their state. For a single checkbox, I think this is overkill and the checkbox should be checked if $value has a value regardless of what the default might be.

Mon Nov 03 15:33:18 2008 The RT System itself - Status changed from 'stalled' to 'open'

Tue Jan 27 14:35:07 2009 cpan [...] fireartist.com - Correspondence added

CheckBox has been changed, so value() now defaults to 1. This means default(1) will now do what you expect, and you'll only need to set value() if you intend using a different value, such as 'on' or 'checked'.

Tue Jan 27 14:35:11 2009 cpan [...] fireartist.com - Status changed from 'open' to 'resolved'