[Dbix-class] '+select'/'+as' is broken when chaining resultsets

BUCHMULLER Norbert norbi.lists at nix.hu
Tue Oct 28 12:49:26 GMT 2008


On Tue, 28 Oct 2008 13:26:52 +0100 BUCHMULLER Norbert
<norbi.lists at nix.hu> wrote:

Sorry, I sent it too early by accident.

> > > Regarding 1.: I think that should be OK.
> > 
> > These tests go into t/76retrieve.t (new file)
> 
> Done (see the new patch).

When I started to create the patch, I found that this test should go
tot/88_result_set_column.t indeed, as it operates on resultset columns
like the other tests in that file. Just it has to be reformulated like the
other tests for '+select'/'+as' in that file. (Ignore the '->first' part
for now - that's unrelated to it.)


> The other approach that came to my mind is to change
> 
>  defined($psrs->get_column('...'))
> 
> to
> 
>  defined($psrs->get_column('...')->first)

Oops, here I meant:

 defined($psrs->get_column('...')->next)


PS: Sorry for the above mistake I made (ie. sending the mail too early).

norbi



More information about the DBIx-Class mailing list