[Maypole] Maypole model inheritance

Simon Flack sf@flacks.net
Sun, 12 Dec 2004 20:39:14 +0000


Dave Howorth wrote:
> Simon Flack wrote:
> 
>> Yes, I like that. One issue is that the table2class() differs from 
>> Class::DBI::Loader. E.g.
> 
> 
> I prefer AutoLoader's rule, but I guess one issue might be backwards 
> compatibility.
> 
>> table: hello-world
>> CDBI::AutoLoader: HelloWorld
>> CDBI::Loader: Hello-world    # this is an invalid package name
> 
> 
> So there's no backwards-compatibility issue there then :)
> 
>> table: hello__world
>> CDBI::AutoLoader: HelloWorld
>> CDBI::Loader: Hello_World
> 
> 
> That will either work, if the user hasn't actually written any code for 
> the table class, or it will cause a compile error that's easy to fix in 
> the more usual case that there is code for an untaint or a relationship 
> or something.
> 
> The most worrying case I can think of is if somebody has one table 
> called hello__world and another called hello_world. But then I think 
> they deserve everything that happens to them :) And something would 
> probably break.
> 
> The other issue will be Class::DBI::Loader::Relationship.

That's a tricky one. I think our best bet is proposing a patch to 
Class::DBI::Loader.

--simonflk