[Dbix-class] Re: SQL::Translator conversion from Class::Base
rabbit+dbic at rabbit.us
Wed Aug 29 03:43:08 GMT 2012
On Wed, Aug 29, 2012 at 01:15:50AM +0100, Dagfinn Ilmari Mannsåker wrote:
> ilmari at ilmari.org (Dagfinn Ilmari Mannsåker) writes:
> > The code passes the test suites of DBIx::Class (with
> > DBICTEST_SQLT_DEPLOY=1), ::DeploymentHandler and ::Migration, but I
> > would like some testing and review by actual users before merging it to
> > trunk.
> Thanks to frew and jnap for reviewing and testing this. Unless anyone
> objects I'll merge it in the next few days so we can get a dev release
> out for smoking.
A cursory test of non-DBIC based SQLT dependents turned up
Metabase::Backend::SQLite with a bunch of readonly accessors being called
as setters. Now, Metabase::Backend::SQLite fails one test on currently
released SQLT anyway (please bug xdg), but nevertheless this is might be
an indication that you were too eager on applying 'ro'.
More information about the DBIx-Class