Re: [Nms-cgi-devel] formmail patch

[Scott H. <js...@ve...>]

On Tue, Sep 20, 2005 at 08:48:55PM +0100, Jonathan Stowe wrote:
> On Tue, 2005-09-20 at 19:09, Scott Hepler wrote:
> > Greetings,
> > 
> > I don't know if formmail is still being updated (I've read of a preference
> > for TFMail updates in the archives of this list), but I'm including a patch
> > to pass the form values along to the 'missing_fields_redirect' page in a
> > query string.  It also sends the contents of @missing_fields.  This lets
> > the error page re-display the form, or give better feedback.
> 
> Thanks for this!  I'll take a look at putting this into the next
> release.  If you have some spare time, it might be nice to have a
> configuration to switch this behaviour on and off as it might confuse
> users upgrading.  An example of how to use this in Javascript or PHP
> might be useful as well :-)

Great!

I'll of course be glad if it gets in, but please feel free to fix as
necessary if you decide to have it.

I'm attaching a bigger patch with the other changes you suggested (switch,
example).  I'm not real thrilled with that huge variable name for the
switch, but couldn't really decide between "short" and "exact".....

-s.

-- 
Scott Hepler
System Administration and Security
Web Development
802.457.3796