On Tue, Jan 11, 2011 at 2:27 PM, Gabor HALASZ <halas...@freemail.hu> wrote:

> 2011.01.11. 16:39 keltezéssel, Ronald J Kimball írta:
>
>> On Tue, Jan 11, 2011 at 8:21 AM, Gabor HALASZ <halas...@freemail.hu
>> <mailto:halas...@freemail.hu>> wrote:
>>
>>
>>    I think, simply on wrong place and checks wrong variable, the
>>    correct is:
>>
>>    for my $name ( keys %$from_stash ) {
>>
>>        croak "config value must not be a reference" if ref $name;
>>
>>
>> Checking whether $name is a reference wouldn't make sense.  It's a hash
>> key, so we already know it's a string.
>>
>>  Sure, sorry. Of course, I think:
>
> croak "config value must not be a reference" if ref $form_stash->{$name};
>
>
Aha, yes, that makes sense.  It will use $from_stash->{$name} as a hash key
in $form->stash.  As you said, it's checking the wrong variable.

Ronald
_______________________________________________
HTML-FormFu mailing list
HTML-FormFu@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu

Reply via email to