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/

Reply via email to