On Thu, 2003-06-05 at 21:32, Steve Comrie wrote:
> I can't find see an immediate problem with your code, but chances are I'm
> overlooking something. A simple solution might be to pass the
> 'die_on_bad_params' => 0, argument to your HTML::Template->new() statement.

Put that in and now I can't force the error.  :-/

> A question about your choice of creating forms though. Why are you writing
> so much perl code to output html code, when it appears that your desired
> html is simple enough to be created in html. You're adding an extra level of
> abstraction that doesn't immediately (to me) seem necessary.
> 
> It looks like the only dynamic aspect to your template is the $switches
> array which could easily be inserted into the template via a TMPL_LOOP. You
> could cut your perl code down from 29 lines to about 5. And you wouldn't
> really be adding many more lines to the template, just a couple more
> characters per line then what's already there.

Yer right.  I was getting re-acquainted with CGI.pm when I wrote that
and was remembering it's calls better than the raw HTML for the form
entries.  If I have time I'll go back and rewrite it.

--[Lance]

-- 
  Carolina Spirit Quest:  http://www.carolinaspiritquest.org/
  Celebrate The Circle: http://www.angelfire.com/nc/celebratethecircle/
  My LiveJournal: http://www.livejournal.com/users/labrown/


---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/[EMAIL PROTECTED]/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to