X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FResultSetColumn.pm;h=e11f86849ab9365ce98cabfb788eeed4ed6beb15;hb=3963bf4c24f4bd61ef0b7e9ad9edae9aac6ab677;hp=57fa22797d05de5c420f55d08d130ef219ab9849;hpb=472d7df3682bdb73a76af18c6c78e4596cb8aa73;p=dbsrgits%2FDBIx-Class-Historic.git diff --git a/lib/DBIx/Class/ResultSetColumn.pm b/lib/DBIx/Class/ResultSetColumn.pm index 57fa227..e11f868 100644 --- a/lib/DBIx/Class/ResultSetColumn.pm +++ b/lib/DBIx/Class/ResultSetColumn.pm @@ -66,10 +66,14 @@ sub new { ( $rs->result_source->schema->storage->_parse_order_by ($orig_attrs->{order_by} ) ) ) { my $alias = $rs->current_source_alias; + # nuke the prefetch before collapsing to sql + my $subq_rs = $rs->search; + $subq_rs->{attrs}{join} = $subq_rs->_merge_attr( $subq_rs->{attrs}{join}, delete $subq_rs->{attrs}{prefetch} ); + $new_parent_rs = $rs->result_source->resultset->search ( {}, { alias => $alias, from => [{ - $alias => $rs->as_query, + $alias => $subq_rs->as_query, -alias => $alias, -source_handle => $rs->result_source->handle, }] @@ -79,16 +83,11 @@ sub new { $new_parent_rs ||= $rs->search_rs; my $new_attrs = $new_parent_rs->{attrs} ||= {}; - # FIXME - this should go away when the chaining branch is merged - # since what we do is actually chain to the original resultset, we need to throw - # away all selectors (otherwise they'll chain) - delete $new_attrs->{$_} for (qw/columns +columns select +select as +as cols include_columns/); - # prefetch causes additional columns to be fetched, but we can not just make a new # rs via the _resolved_attrs trick - we need to retain the separation between # +select/+as and select/as. At the same time we want to preserve any joins that the # prefetch would otherwise generate. - $new_attrs->{join} = $rs->_merge_attr( delete $new_attrs->{join}, delete $new_attrs->{prefetch} ); + $new_attrs->{join} = $rs->_merge_attr( $new_attrs->{join}, delete $new_attrs->{prefetch} ); # {collapse} would mean a has_many join was injected, which in turn means # we need to group *IF WE CAN* (only if the column in question is unique)