Re: [PATCH] give us more temporary column groups

[prev] [thread] [next] [Date index for 2004/12/12]

On Sun, Dec 12, 2004 at 05:54:28PM +0000, Aaron Trevena wrote:
> Therefore we would like to have any column group that starts with __TEMP 
> to be treated the same as TEMP. 

I'm happy with the idea, but not with the patch, as there is also code
like this is Class::DBI::Column that will need likewise changed.

It seems to me that the best approach here would be to add an
is_temporary() method to ColumnGrouper and fix the rest of the code to
use that.

Tests are always nice too :)

Thanks,

Tony