[Catalyst] Catalyst::Example::InstantCRUD RFC

[Will Hawes]

Zbigniew Lukasiak wrote:
> Hi,
> 
> In the attachments you'll find my Catalyst CRUD example ported to
> DBIx::Class.   Below I repeat the same arguments I presented for
> Catalyst::Example::PagingAndSort - so if you read my previous mail
> this might be redundant.
> 
> The main goal of this work is a script generating a working Catalyst
> example with minimal setup by the user.  The additional goal is to
> make the example extensible and customisable.
> 
> The generation is just one step:
> 
> instantcrud.pl -name=My::App -dsn='dbi:Pg:dbname=CE' -user=zby -password='pass'
> 
> I believe this is the simplest way to have a working non trivial
> Catalyst application.
> 
> To meet the goal of extensibility the generated controllers and models
> are empty subclasses of my library controller or model.
> This way the user can upgrade my libraries without braking the
> generated applications, but he can also take the responsibility of
> parts of the generated application and use the standard OO technique
> of overriding the methods from the super class.
> 
> I am waiting for comments.
> 

I like the idea.

I've only had a quick look at the code, but one thing that strikes me 
immediately is that you have subs named "Table_name" and "Table_Class". 
I wouldn't mix case like that. The usual perl naming convention is to 
use all lowercase and it's a good idea to stick to it, particularly as 
the code is intended to help new users.