Skip Menu |
 

This queue is for tickets about the DBIx-Class CPAN distribution.

Report information
The Basics
Id: 122571
Status: rejected
Priority: 0/
Queue: DBIx-Class

People
Owner: Nobody in particular
Requestors: KES [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.082840
Fixed in: (no value)



Subject: select: can not indicate which columns should be selected from the storage.
Download (untitled) / with headers
text/plain 350b
I come with a question: https://stackoverflow.com/questions/45258128/why-select-does-not-replace-existing-columns But finally I have discovered that `prefetch` does not work as documented Here is details: https://stackoverflow.com/a/45264653/4632019 In short: When I use `prefetch` it selects all columns from table despite on `select` attribute.
Download (untitled) / with headers
text/plain 849b
On Sun Jul 23 07:46:58 2017, KES wrote: Show quoted text
> I come with a question: > https://stackoverflow.com/questions/45258128/why-select-does-not- > replace-existing-columns > > But finally I have discovered that `prefetch` does not work as > documented
This is merely a lack of documentation clarity - "It can be thought of as a rough superset of the "join" attribute." does say *rough* superset. If you want to select custom columns, you want to use join+select yourself. While I would appreciate a doc patch, I'm marking this rejected because (1) the bug title is a lie (2) a slightly unclear piece of documentation is not a 'critical' bug (3) the last time you claimed a document was unclear it was because it had "too many words" so you hadn't read it. Please feel free to supply a patch that rephrases it if you can think of a better phrasing, though.
Subject: Re: [rt.cpan.org #122571] select: can not indicate which columns should be selected from the storage.
Date: Tue, 25 Jul 2017 18:46:05 +0300
To: "bug-DBIx-Class [...] rt.cpan.org" <bug-dbix-class [...] rt.cpan.org>, "KES [...] cpan.org" <kes [...] cpan.org>
From: KES <kes-kes [...] yandex.ru>
Download (untitled) / with headers
text/plain 521b
Show quoted text
>you want to use join+select yourself
I can not: because I should patch parent class which use `prefetch` If I patch I broke all other classes which inherit that one. Also at the point when I want to specify columns to select I do not know what parent/preceding code did. I just got ResultSet and want to specify the columns Yeah, I should rewrite each `prefetch` in project by `join`+`select` to be sure that at some point it did not return not requested columns. If DOC will be batched it will be great. Thank you.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.