[Dbix-class] Search with complex joins generating bad SQL

Byron Young Byron.Young at riverbed.com
Tue Apr 6 17:39:59 GMT 2010


> -----Original Message-----
> From: Peter Rabbitson [mailto:rabbit+dbic at rabbit.us]
> Sent: Monday, April 05, 2010 8:41 PM
> To: DBIx::Class user and developer list
> Subject: Re: [Dbix-class] Search with complex joins generating bad SQL
> 
> On Mon, Apr 05, 2010 at 10:55:08AM -0700, Byron Young wrote:
> >
> > Have you had a chance to look at this?  I tried to test with trunk to
> see if it had the issue, but I got lost in dependency hell.  The trunk
> version wants to upgrade Moose, which will cause us to have to upgrade
> Catalyst.  We just spent a pretty significant amount of time upgrading
> to the latest DBIC and testing all of our apps that depend on the
> various upgraded modules, and we're on a super tight schedule right
> now, so I didn't want to go down that road again with more upgrades at
> the moment.  I realize your time is limited too, but any help you can
> offer would be greatly appreciated.  Is it possible to patch our
> current version to fix the bug without upgrading?
> >
> 
> It's fixed by [1]. All you need is to switch keys() to values() in your
> local copy as seen in the commit. As far as dependency hell - you tried
> to work with a checkout, which assumes author mode, which in turn
> requires you to install *all* dependencies, so every regression test
> will run, You don't *have* to install them to use a dbic checkout
> however.
> 
> Sorry took so long. Cheers!
> 
> [1]
> http://dev.catalystframework.org/svnweb/bast/revision?rev=9085#diff__DB
> Ix-Class_0.08_trunk_lib_DBIx_Class_Storage_DBIHacks.pm
> 

You're a life-saver!  Thank you for the quick fix!

As for your comments about dependencies - how do I install trunk without having to install all dependencies for all the tests?  Is there some documentation somewhere I can read?

Thanks again!
Byron



More information about the DBIx-Class mailing list