[Html-widget] formfu - HTML::FormFu::Localize

[Carl Franks]

On 20/04/07, Daisuke Maki <daisuke at endeworks.jp> wrote:
> I'm trying to use a custom localize object so I can display errors from
> my custom validator, and added this to my yaml config:
>
>   ---
>   localize_class: 'My::I18N'
>
> This barfed on me at HTML/FormFu/Localize.pm line 95, where the class is
> being required. The errors stated was "class argument missing" in
> require_class().
>
> So I looked at the code, and I'm guessing this is the culprit:
>
> Index: lib/HTML/FormFu/Localize.pm
> ===================================================================
> --- lib/HTML/FormFu/Localize.pm (revision 189)
> +++ lib/HTML/FormFu/Localize.pm (working copy)
> @@ -151,9 +151,7 @@
>  #warn "+ loading ".$localize_data->{localize_object};
>
>          $localize_data->{localize_object} =
> -            $self->get_localize_object_from_class(
> -                $localize_data->{localize_object}
> -            );
> +            $self->get_localize_object_from_class( $self->localize_class );
>
>          $localize_data->{dies_on_missing_key} =
>              $self->get_localize_object_dies_on_missing_key(
>
> Is my correction above right? This is the first time I try out to change
> localize_class, so I wasn't too sure if this was indeed the intended
> behavior.

Sorry that this got overlooked.
I had hoped mario might respond, as I'm not familiar with the code in
Localize.pm
I've committed your patch, but the L10N behaviour really needs more
tests to pin down its behaviour.

Cheers,
Carl