[Catalyst] C::C::FormBuilder question

Juan Camacho jc5826 at gmail.com
Tue Jan 30 16:42:14 GMT 2007


On 1/30/07, Tobias Kremer <list at funkreich.de> wrote:
> > I was just looking at this last week, as I was using the code as a
> > basis for a new controller. I suspect that it's never been tested,
> > because as far as I can tell, it incorrectly reads the config from the
> > controller object, rather than the application object.
>
> Yeah, I stumbled across this the other day but was unsure if it was
> done on purpose (for a controller-specific configuration rather than a
> global config). Nevertheless I found it awkward and IMHO it should be changed.
>
> --Tobias

It was done somewhat on purpose since configuring it via the
controller was how I use it. I would be happy to accept a patch to
make it read from both places.  Just make sure existing tests continue
to pass and work off the version that's up on subversion at
http://dev.catalyst.perl.org/repos/Catalyst/trunk/Catalyst-Controller-FormBuilder/



More information about the Catalyst mailing list