[Dbix-class] Re: Bind order bug, patch/test for review

Matt S Trout dbix-class at trout.me.uk
Wed Oct 3 20:43:27 GMT 2007


On Sat, Sep 29, 2007 at 12:12:43PM -0700, Marc Mims wrote:
> > Can it be this simple?  Surely there was some reason for unshift instead
> > of push, but I didn't discover it.
> 
> No.  It's not that simple.  An updated test is attached.  The simple
> cases pass with the patch applied.  But the complex tests, based on the
> Cookbook "Arbitrary SQL" technique fail.
> http://search.cpan.org/~ash/DBIx-Class-0.08007/lib/DBIx/Class/Manual/Cookbook.pod#Arbitrary_SQL_through_a_custom_ResultSource
> 
> I'm not sure how to tackle this problem in a way that will satisfy all
> the tests...yet.

Well, TODO the tests that don't pass and chuck it into 08 trunk, at least we'll
know it's there then.

-- 
      Matt S Trout       Need help with your Catalyst or DBIx::Class project?
   Technical Director                    http://www.shadowcat.co.uk/catalyst/
 Shadowcat Systems Ltd.  Want a managed development or deployment platform?
http://chainsawblues.vox.com/            http://www.shadowcat.co.uk/servers/



More information about the DBIx-Class mailing list