Sebastian Willert wrote:
> Hi all,
>
> after some late-night discussion with castaway about
> DBIC::Schema->deploy() we noticed that
> SQL::Translator::Parser::DBIx::Class tries to query the connected
> database. This shouldn't be done IMO because this might change the info
> provided in the schema and might not be stable between different
> underlying RDBMs.
>
> This are the reasons why I've produced the patch below, the changes
> should be self-explainatory. The patch has been tested on the
> command-line, using the programmatic interface of SQLT and using
> DBIC::Schema->deploy() but could need some additional testing with more
> complex schema then I have readily available.
The changes are clearly harmful. A schema class simply acts as a prototype for
objects of that class, it is not guaranteed to contain all the requisite data.
--
Matt S Trout Offering custom development, consultancy and support
Technical Director contracts for Catalyst, DBIx::Class and BAST. Contact
Shadowcat Systems Ltd. mst (at) shadowcatsystems.co.uk for more information
+ Help us build a better perl ORM: http://dbix-class.shadowcatsystems.co.uk/ +
_______________________________________________
List: http://lists.rawmode.org/cgi-bin/mailman/listinfo/dbix-class
Wiki: http://dbix-class.shadowcatsystems.co.uk/
IRC: irc.perl.org#dbix-class
SVN: http://dev.catalyst.perl.org/repos/bast/trunk/DBIx-Class/
Searchable Archive: http://www.mail-archive.com/[email protected]/
