On Mon, May 07, 2007 at 12:09:49PM -0400, Juan Camacho wrote: > On 5/7/07, Matt S Trout <[EMAIL PROTECTED]> wrote: > > >in MyApp.pm - you were confusing the 'Controller::Foo' key that specifies > >which component MyApp-level config is for and the 'Controller::FormBuilder' > >key that specifies C::C::FB-specific config options within a controller. > >The names are sufficiently similar I'm not blaming you though - Juan, maybe > >a doc tweak or a clearer name is in order? > > > > Yes, I'll clarify the documentation and maybe include something about > a Controller base class as a best practice suggestion. Having it in > the synopsis section doesn't seem to provide enough descriptive info > it seems ;)
I think making the key 'FormBuilder' instead of 'Controller::FormBuilder' would probably also help people a lot; I understand your desire to keep the C::FB config params namespaced to avoid clashes but it -is- confusing, I have to think about it when I'm helping people using FB :) -- Matt S Trout Need help with your Catalyst or DBIx::Class project? Technical Director Want a managed development or deployment platform? Shadowcat Systems Ltd. Contact mst (at) shadowcatsystems.co.uk for a quote http://www.shadowcatsystems.co.uk/ _______________________________________________ List: Catalyst@lists.rawmode.org Listinfo: http://lists.rawmode.org/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/catalyst@lists.rawmode.org/ Dev site: http://dev.catalyst.perl.org/