Re: Columns substitution?

[prev] [thread] [next] [Date index for 2005/05/27]

On Wed, May 25, 2005 at 04:14:09PM -0700, Lonely Rolling Star wrote:
> I've added this to transform_sql:
> $sql =~ s/__COLUMNS\((.*?)\)__/join ", ", map "$_", $self->columns($1)/eg;
> Which makes __COLUMNS(Foo)__ expand to the list of columns defined 
> earlier by __PACKAGE__->columns(Foo => ...)
> Thoughts? Did I miss an easy way to do this?

Looks good.

You should write it up on the wiki so that other people can use it now, but
if you can dress it up with tests etc then I'll don't see any reason
not to add it to the next release.

Thanks,

Tony