RE: [cgiapp] ANNOUNCE BETA: CGI::Application::Template 0.01

[Mark Stosberg <suppressed>]

> > - I was surprised you didn't use the new arrayref method of declaring
> >   run modes.
>
> I did, and then I switched back for two reasons:  First, I didn't know
> if the user would be using CGI-App 3.0.  Second, I wanted to make the
> output module more easy to adapt, if desired.  Perhaps I'll change it
> back.

You do know that the user is using at least v3.0, because you require it
the Makefile.PL:
	'CGI::Application' => 3,

> > - It would be nice to combine these common three rows of output code
> >   into a single function:
> >
> >    open(OUTPUT, ">WidgetBrowser.pm") || die($!);
> >    print OUTPUT $cat->output_app_module();
> >    close(OUTPUT);
>
> I thought about that, too.  In the end, I chose not to write the file
> because I didn't want to prevent someone from using this module in a
> different way which didn't involve creating a file.

I like that thinking, but I don't see a conflict with having two output
methods.

I like the handling of the run-mode to template mapping, which I hadn't
noticed the first time through. I'm also not in the habit of using
dump_html() for debugging. I'll have to try that out.

  Mark

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

---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/suppressed/
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed