[Dbix-class] find redux

[Matt S Trout]

On Fri, Jan 25, 2008 at 05:10:07PM +0100, Zbigniew Lukasiak wrote:
> On Jan 25, 2008 2:30 PM, Matt S Trout <dbix-class at trout.me.uk> wrote:
> > On Fri, Jan 25, 2008 at 12:24:27PM +0100, Zbigniew Lukasiak wrote:
> > > Let me sidestep that a bit
> >
> > No, I won't let you.
> >
> > I spent an hour discussing with people how to structure this discussion so
> > we can actually work out how to proceed in a framework that's clear, precise
> > and reasonably easy to convert across to code.
> >
> > Please do me the courtesy of respecting the effort that went into this
> > and working within the structure provided.
> 
> OK, if that will help you I am happy to provide that help.
> 
> I am not sure what you meant in the end note - I'll assume here that
> it means that when you say 'all columns' you mean the columns in the
> query and in the internal resultset condition.  Correct me if I am
> wrong.  This question of including the 'implicite rs columns' is the
> most important thing in this whole analysis - because it is the least
> understood one.  My opinion is that those columns should be included.
> 
> And assuming that I differ with you in the analysis of
> 
> Case 1 - called with 'key' attr and all columns for that key
> 
> If all the key columns are in the query then the behaviour is correct,
> but if one of the columns for the key is in the condition instead of
> the query then a search by all columns will be issued.

Hrm. That's it seeing it as case 2 when it should be case 1.

> The Case 1 is the most important one to fix.

Fortunately, case 1 already works.

Now, please try again, reading my e-mail more carefully.

And this time reply to the original message as requested.

Failing to do that and also not actually talking about the things you're
replying to as they're written is why I started this new thread - you still
haven't actually managed to put your point across clearly enough for anybody
to know what you're talking about.

-- 
      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/