[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[cgiapp] Re: fill_password and CGI::Application::ValidateRM


On 2004-03-04, Gabor Szabo <suppressed> wrote:
> ... and keeping my self discussion on, here is an untested patch to CAV
> that could solve my problem. If there really is a problem.
>
> --- /opt/perl582/lib/site_perl/5.8.2/CGI/Application/ValidateRM.pm	Tue Dec  2 13:01:37 2003
> +++ CGI/Application/ValidateRM.pm	Thu Mar  4 10:44:28 2004
> @@ -21,6 +21,7 @@
>       my $self = shift;
>  	 my $return_rm = shift || die 'missing required return run mode';
>       my $profile_in = shift || die 'missing required profile';
> +     my $html_fill_params = shift;
>
>  	# If the profile is not a hash reference,
>  	# assume it's a CGI::App method
> @@ -53,7 +54,8 @@
>  		 my $fif = new HTML::FillInForm;
>  		 $err_page = $fif->fill(
>               scalarref => $return_pageref,
> -			 fobject => $self->query
> +			 fobject => $self->query,
> +			 %$html_fill_params,
>  		 );
>  	}
>  	return ($r,$err_page);

Gabor,

Thanks for the patch.  Seems like a good idea to me. Could you expand
your patch to include documentation and a test case? With that, I so no
reason not to release it.

	Mark

-- 
http://mark.stosberg.com/ 


---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/suppressed/
              http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed


Mail converted by mhonarc 2.6.15
This archive provided courtesy of JSW4.NET, Internet Hosting Services for Small Business.