Commit | Line | Data |
c443438f |
1 | package #hide from PAUSE |
2 | DBIx::Class::Storage::DBIHacks; |
d28bb90d |
3 | |
4 | # |
5 | # This module contains code that should never have seen the light of day, |
6 | # does not belong in the Storage, or is otherwise unfit for public |
7eb76996 |
7 | # display. The arrival of SQLA2 should immediately oboslete 90% of this |
d28bb90d |
8 | # |
9 | |
10 | use strict; |
11 | use warnings; |
12 | |
13 | use base 'DBIx::Class::Storage'; |
14 | use mro 'c3'; |
15 | |
16 | use Carp::Clan qw/^DBIx::Class/; |
6298a324 |
17 | use List::Util 'first'; |
18 | use Scalar::Util 'blessed'; |
19 | use namespace::clean; |
d28bb90d |
20 | |
21 | # |
052e8431 |
22 | # This code will remove non-selecting/non-restricting joins from |
4b1b5ea3 |
23 | # {from} specs, aiding the RDBMS query optimizer |
052e8431 |
24 | # |
25 | sub _prune_unused_joins { |
4b1b5ea3 |
26 | my ($self) = shift; |
27 | |
28 | my ($from, $select, $where, $attrs) = @_; |
052e8431 |
29 | |
052e8431 |
30 | if (ref $from ne 'ARRAY' || ref $from->[0] ne 'HASH' || ref $from->[1] ne 'ARRAY') { |
31 | return $from; # only standard {from} specs are supported |
32 | } |
33 | |
4b1b5ea3 |
34 | my $aliastypes = $self->_resolve_aliastypes_from_select_args(@_); |
35 | |
36 | # a grouped set will not be affected by amount of rows. Thus any |
37 | # {multiplying} joins can go |
38 | delete $aliastypes->{multiplying} if $attrs->{group_by}; |
39 | |
052e8431 |
40 | |
41 | my @newfrom = $from->[0]; # FROM head is always present |
42 | |
43 | my %need_joins = (map { %{$_||{}} } (values %$aliastypes) ); |
44 | for my $j (@{$from}[1..$#$from]) { |
539ffe87 |
45 | push @newfrom, $j if ( |
4b1b5ea3 |
46 | (! $j->[0]{-alias}) # legacy crap |
539ffe87 |
47 | || |
48 | $need_joins{$j->[0]{-alias}} |
49 | ); |
052e8431 |
50 | } |
51 | |
52 | return \@newfrom; |
53 | } |
54 | |
052e8431 |
55 | # |
d28bb90d |
56 | # This is the code producing joined subqueries like: |
57 | # SELECT me.*, other.* FROM ( SELECT me.* FROM ... ) JOIN other ON ... |
58 | # |
59 | sub _adjust_select_args_for_complex_prefetch { |
60 | my ($self, $from, $select, $where, $attrs) = @_; |
61 | |
62 | $self->throw_exception ('Nothing to prefetch... how did we get here?!') |
63 | if not @{$attrs->{_prefetch_select}}; |
64 | |
65 | $self->throw_exception ('Complex prefetches are not supported on resultsets with a custom from attribute') |
66 | if (ref $from ne 'ARRAY' || ref $from->[0] ne 'HASH' || ref $from->[1] ne 'ARRAY'); |
67 | |
68 | |
69 | # generate inner/outer attribute lists, remove stuff that doesn't apply |
70 | my $outer_attrs = { %$attrs }; |
71 | delete $outer_attrs->{$_} for qw/where bind rows offset group_by having/; |
72 | |
73 | my $inner_attrs = { %$attrs }; |
74 | delete $inner_attrs->{$_} for qw/for collapse _prefetch_select _collapse_order_by select as/; |
75 | |
76 | |
77 | # bring over all non-collapse-induced order_by into the inner query (if any) |
78 | # the outer one will have to keep them all |
79 | delete $inner_attrs->{order_by}; |
80 | if (my $ord_cnt = @{$outer_attrs->{order_by}} - @{$outer_attrs->{_collapse_order_by}} ) { |
81 | $inner_attrs->{order_by} = [ |
82 | @{$outer_attrs->{order_by}}[ 0 .. $ord_cnt - 1] |
83 | ]; |
84 | } |
85 | |
d28bb90d |
86 | # generate the inner/outer select lists |
87 | # for inside we consider only stuff *not* brought in by the prefetch |
88 | # on the outside we substitute any function for its alias |
89 | my $outer_select = [ @$select ]; |
90 | my $inner_select = []; |
91 | for my $i (0 .. ( @$outer_select - @{$outer_attrs->{_prefetch_select}} - 1) ) { |
92 | my $sel = $outer_select->[$i]; |
93 | |
94 | if (ref $sel eq 'HASH' ) { |
95 | $sel->{-as} ||= $attrs->{as}[$i]; |
96 | $outer_select->[$i] = join ('.', $attrs->{alias}, ($sel->{-as} || "inner_column_$i") ); |
97 | } |
98 | |
99 | push @$inner_select, $sel; |
bb9bffea |
100 | |
101 | push @{$inner_attrs->{as}}, $attrs->{as}[$i]; |
d28bb90d |
102 | } |
103 | |
d28bb90d |
104 | # construct the inner $from for the subquery |
48580715 |
105 | # we need to prune first, because this will determine if we need a group_by below |
052e8431 |
106 | my $inner_from = $self->_prune_unused_joins ($from, $inner_select, $where, $inner_attrs); |
ad630f4b |
107 | |
539ffe87 |
108 | # if a multi-type join was needed in the subquery - add a group_by to simulate the |
109 | # collapse in the subq |
110 | $inner_attrs->{group_by} ||= $inner_select |
6298a324 |
111 | if first { ! $_->[0]{-is_single} } (@{$inner_from}[1 .. $#$inner_from]); |
d28bb90d |
112 | |
d28bb90d |
113 | # generate the subquery |
114 | my $subq = $self->_select_args_to_query ( |
052e8431 |
115 | $inner_from, |
d28bb90d |
116 | $inner_select, |
117 | $where, |
118 | $inner_attrs, |
119 | ); |
120 | |
121 | my $subq_joinspec = { |
122 | -alias => $attrs->{alias}, |
052e8431 |
123 | -source_handle => $inner_from->[0]{-source_handle}, |
d28bb90d |
124 | $attrs->{alias} => $subq, |
125 | }; |
126 | |
127 | # Generate the outer from - this is relatively easy (really just replace |
128 | # the join slot with the subquery), with a major caveat - we can not |
129 | # join anything that is non-selecting (not part of the prefetch), but at |
130 | # the same time is a multi-type relationship, as it will explode the result. |
131 | # |
132 | # There are two possibilities here |
133 | # - either the join is non-restricting, in which case we simply throw it away |
134 | # - it is part of the restrictions, in which case we need to collapse the outer |
135 | # result by tackling yet another group_by to the outside of the query |
136 | |
052e8431 |
137 | # normalize a copy of $from, so it will be easier to work with further |
138 | # down (i.e. promote the initial hashref to an AoH) |
139 | $from = [ @$from ]; |
140 | $from->[0] = [ $from->[0] ]; |
141 | |
d28bb90d |
142 | # so first generate the outer_from, up to the substitution point |
143 | my @outer_from; |
144 | while (my $j = shift @$from) { |
145 | if ($j->[0]{-alias} eq $attrs->{alias}) { # time to swap |
146 | push @outer_from, [ |
147 | $subq_joinspec, |
148 | @{$j}[1 .. $#$j], |
149 | ]; |
150 | last; # we'll take care of what's left in $from below |
151 | } |
152 | else { |
153 | push @outer_from, $j; |
154 | } |
155 | } |
156 | |
052e8431 |
157 | # scan the from spec against different attributes, and see which joins are needed |
158 | # in what role |
159 | my $outer_aliastypes = |
539ffe87 |
160 | $self->_resolve_aliastypes_from_select_args( $from, $outer_select, $where, $outer_attrs ); |
052e8431 |
161 | |
d28bb90d |
162 | # see what's left - throw away if not selecting/restricting |
163 | # also throw in a group_by if restricting to guard against |
164 | # cross-join explosions |
165 | # |
166 | while (my $j = shift @$from) { |
167 | my $alias = $j->[0]{-alias}; |
168 | |
1a736efb |
169 | if ($outer_aliastypes->{selecting}{$alias}) { |
d28bb90d |
170 | push @outer_from, $j; |
171 | } |
1a736efb |
172 | elsif ($outer_aliastypes->{restricting}{$alias}) { |
d28bb90d |
173 | push @outer_from, $j; |
539ffe87 |
174 | $outer_attrs->{group_by} ||= $outer_select unless $j->[0]{-is_single}; |
d28bb90d |
175 | } |
176 | } |
177 | |
178 | # demote the outer_from head |
179 | $outer_from[0] = $outer_from[0][0]; |
180 | |
181 | # This is totally horrific - the $where ends up in both the inner and outer query |
182 | # Unfortunately not much can be done until SQLA2 introspection arrives, and even |
183 | # then if where conditions apply to the *right* side of the prefetch, you may have |
184 | # to both filter the inner select (e.g. to apply a limit) and then have to re-filter |
185 | # the outer select to exclude joins you didin't want in the first place |
186 | # |
187 | # OTOH it can be seen as a plus: <ash> (notes that this query would make a DBA cry ;) |
188 | return (\@outer_from, $outer_select, $where, $outer_attrs); |
189 | } |
190 | |
1a736efb |
191 | # |
192 | # I KNOW THIS SUCKS! GET SQLA2 OUT THE DOOR SO THIS CAN DIE! |
193 | # |
ad630f4b |
194 | # Due to a lack of SQLA2 we fall back to crude scans of all the |
195 | # select/where/order/group attributes, in order to determine what |
196 | # aliases are neded to fulfill the query. This information is used |
197 | # throughout the code to prune unnecessary JOINs from the queries |
198 | # in an attempt to reduce the execution time. |
199 | # Although the method is pretty horrific, the worst thing that can |
1a736efb |
200 | # happen is for it to fail due to some scalar SQL, which in turn will |
201 | # result in a vocal exception. |
539ffe87 |
202 | sub _resolve_aliastypes_from_select_args { |
052e8431 |
203 | my ( $self, $from, $select, $where, $attrs ) = @_; |
546f1cd9 |
204 | |
ad630f4b |
205 | $self->throw_exception ('Unable to analyze custom {from}') |
206 | if ref $from ne 'ARRAY'; |
546f1cd9 |
207 | |
ad630f4b |
208 | # what we will return |
964a3c71 |
209 | my $aliases_by_type; |
546f1cd9 |
210 | |
ad630f4b |
211 | # see what aliases are there to work with |
212 | my $alias_list; |
539ffe87 |
213 | for (@$from) { |
214 | my $j = $_; |
ad630f4b |
215 | $j = $j->[0] if ref $j eq 'ARRAY'; |
539ffe87 |
216 | my $al = $j->{-alias} |
217 | or next; |
218 | |
219 | $alias_list->{$al} = $j; |
220 | $aliases_by_type->{multiplying}{$al} = 1 |
221 | unless $j->{-is_single}; |
546f1cd9 |
222 | } |
546f1cd9 |
223 | |
1a736efb |
224 | # get a column to source/alias map (including unqualified ones) |
225 | my $colinfo = $self->_resolve_column_info ($from); |
226 | |
ad630f4b |
227 | # set up a botched SQLA |
228 | my $sql_maker = $self->sql_maker; |
229 | my $sep = quotemeta ($self->_sql_maker_opts->{name_sep} || '.'); |
07f31d19 |
230 | |
1a736efb |
231 | my ($orig_lquote, $orig_rquote) = map { quotemeta $_ } (do { |
232 | if (ref $sql_maker->{quote_char} eq 'ARRAY') { |
233 | @{$sql_maker->{quote_char}} |
234 | } |
235 | else { |
236 | ($sql_maker->{quote_char} || '') x 2; |
237 | } |
ad630f4b |
238 | }); |
07f31d19 |
239 | |
1a736efb |
240 | local $sql_maker->{quote_char} = "\x00"; # so that we can regex away |
241 | |
242 | # generate sql chunks |
a7e643b1 |
243 | local $sql_maker->{having_bind}; # these are throw away results |
1a736efb |
244 | my $to_scan = { |
245 | restricting => [ |
246 | $sql_maker->_recurse_where ($where), |
a7e643b1 |
247 | $sql_maker->_parse_rs_attrs ({ |
1a736efb |
248 | map { $_ => $attrs->{$_} } (qw/group_by having/) |
249 | }), |
250 | ], |
251 | selecting => [ |
252 | $self->_parse_order_by ($attrs->{order_by}, $sql_maker), |
253 | $sql_maker->_recurse_fields ($select), |
254 | ], |
255 | }; |
256 | |
257 | # throw away empty chunks |
258 | $_ = [ map { $_ || () } @$_ ] for values %$to_scan; |
259 | |
260 | # first loop through all fully qualified columns and get the corresponding |
261 | # alias (should work even if they are in scalarrefs) |
ad630f4b |
262 | for my $alias (keys %$alias_list) { |
1a736efb |
263 | my $al_re = qr/ |
264 | \x00 $alias \x00 $sep |
265 | | |
266 | \b $alias $sep |
267 | /x; |
268 | |
269 | # add matching for possible quoted literal sql |
270 | $al_re = qr/ $al_re | $orig_lquote $alias $orig_rquote /x |
271 | if ($orig_lquote && $orig_rquote); |
07f31d19 |
272 | |
1a736efb |
273 | |
274 | for my $type (keys %$to_scan) { |
275 | for my $piece (@{$to_scan->{$type}}) { |
276 | $aliases_by_type->{$type}{$alias} = 1 if ($piece =~ $al_re); |
277 | } |
ad630f4b |
278 | } |
1a736efb |
279 | } |
280 | |
281 | # now loop through unqualified column names, and try to locate them within |
282 | # the chunks |
283 | for my $col (keys %$colinfo) { |
284 | next if $col =~ $sep; # if column is qualified it was caught by the above |
285 | |
286 | my $col_re = qr/ \x00 $col \x00 /x; |
287 | |
288 | $col_re = qr/ $col_re | $orig_lquote $col $orig_rquote /x |
289 | if ($orig_lquote && $orig_rquote); |
290 | |
291 | for my $type (keys %$to_scan) { |
292 | for my $piece (@{$to_scan->{$type}}) { |
293 | $aliases_by_type->{$type}{$colinfo->{$col}{-source_alias}} = 1 if ($piece =~ $col_re); |
294 | } |
07f31d19 |
295 | } |
296 | } |
297 | |
298 | # Add any non-left joins to the restriction list (such joins are indeed restrictions) |
ad630f4b |
299 | for my $j (values %$alias_list) { |
07f31d19 |
300 | my $alias = $j->{-alias} or next; |
1a736efb |
301 | $aliases_by_type->{restricting}{$alias} = 1 if ( |
07f31d19 |
302 | (not $j->{-join_type}) |
303 | or |
304 | ($j->{-join_type} !~ /^left (?: \s+ outer)? $/xi) |
305 | ); |
306 | } |
307 | |
308 | # mark all join parents as mentioned |
309 | # (e.g. join => { cds => 'tracks' } - tracks will need to bring cds too ) |
964a3c71 |
310 | for my $type (keys %$aliases_by_type) { |
311 | for my $alias (keys %{$aliases_by_type->{$type}}) { |
312 | $aliases_by_type->{$type}{$_} = 1 |
faeb2407 |
313 | for (map { values %$_ } @{ $alias_list->{$alias}{-join_path} || [] }); |
07f31d19 |
314 | } |
315 | } |
ad630f4b |
316 | |
964a3c71 |
317 | return $aliases_by_type; |
07f31d19 |
318 | } |
319 | |
d28bb90d |
320 | sub _resolve_ident_sources { |
321 | my ($self, $ident) = @_; |
322 | |
323 | my $alias2source = {}; |
324 | my $rs_alias; |
325 | |
326 | # the reason this is so contrived is that $ident may be a {from} |
327 | # structure, specifying multiple tables to join |
6298a324 |
328 | if ( blessed $ident && $ident->isa("DBIx::Class::ResultSource") ) { |
d28bb90d |
329 | # this is compat mode for insert/update/delete which do not deal with aliases |
330 | $alias2source->{me} = $ident; |
331 | $rs_alias = 'me'; |
332 | } |
333 | elsif (ref $ident eq 'ARRAY') { |
334 | |
335 | for (@$ident) { |
336 | my $tabinfo; |
337 | if (ref $_ eq 'HASH') { |
338 | $tabinfo = $_; |
339 | $rs_alias = $tabinfo->{-alias}; |
340 | } |
341 | if (ref $_ eq 'ARRAY' and ref $_->[0] eq 'HASH') { |
342 | $tabinfo = $_->[0]; |
343 | } |
344 | |
345 | $alias2source->{$tabinfo->{-alias}} = $tabinfo->{-source_handle}->resolve |
346 | if ($tabinfo->{-source_handle}); |
347 | } |
348 | } |
349 | |
350 | return ($alias2source, $rs_alias); |
351 | } |
352 | |
353 | # Takes $ident, \@column_names |
354 | # |
355 | # returns { $column_name => \%column_info, ... } |
356 | # also note: this adds -result_source => $rsrc to the column info |
357 | # |
09e14fdc |
358 | # If no columns_names are supplied returns info about *all* columns |
359 | # for all sources |
d28bb90d |
360 | sub _resolve_column_info { |
361 | my ($self, $ident, $colnames) = @_; |
362 | my ($alias2src, $root_alias) = $self->_resolve_ident_sources($ident); |
363 | |
364 | my $sep = $self->_sql_maker_opts->{name_sep} || '.'; |
09e14fdc |
365 | my $qsep = quotemeta $sep; |
d28bb90d |
366 | |
09e14fdc |
367 | my (%return, %seen_cols, @auto_colnames); |
d28bb90d |
368 | |
369 | # compile a global list of column names, to be able to properly |
370 | # disambiguate unqualified column names (if at all possible) |
371 | for my $alias (keys %$alias2src) { |
372 | my $rsrc = $alias2src->{$alias}; |
373 | for my $colname ($rsrc->columns) { |
374 | push @{$seen_cols{$colname}}, $alias; |
09e14fdc |
375 | push @auto_colnames, "$alias$sep$colname" unless $colnames; |
d28bb90d |
376 | } |
377 | } |
378 | |
09e14fdc |
379 | $colnames ||= [ |
380 | @auto_colnames, |
381 | grep { @{$seen_cols{$_}} == 1 } (keys %seen_cols), |
382 | ]; |
383 | |
d28bb90d |
384 | COLUMN: |
385 | foreach my $col (@$colnames) { |
09e14fdc |
386 | my ($alias, $colname) = $col =~ m/^ (?: ([^$qsep]+) $qsep)? (.+) $/x; |
d28bb90d |
387 | |
388 | unless ($alias) { |
389 | # see if the column was seen exactly once (so we know which rsrc it came from) |
390 | if ($seen_cols{$colname} and @{$seen_cols{$colname}} == 1) { |
391 | $alias = $seen_cols{$colname}[0]; |
392 | } |
393 | else { |
394 | next COLUMN; |
395 | } |
396 | } |
397 | |
398 | my $rsrc = $alias2src->{$alias}; |
399 | $return{$col} = $rsrc && { |
400 | %{$rsrc->column_info($colname)}, |
401 | -result_source => $rsrc, |
402 | -source_alias => $alias, |
403 | }; |
404 | } |
405 | |
406 | return \%return; |
407 | } |
408 | |
289ac713 |
409 | # The DBIC relationship chaining implementation is pretty simple - every |
410 | # new related_relationship is pushed onto the {from} stack, and the {select} |
411 | # window simply slides further in. This means that when we count somewhere |
412 | # in the middle, we got to make sure that everything in the join chain is an |
413 | # actual inner join, otherwise the count will come back with unpredictable |
414 | # results (a resultset may be generated with _some_ rows regardless of if |
415 | # the relation which the $rs currently selects has rows or not). E.g. |
416 | # $artist_rs->cds->count - normally generates: |
417 | # SELECT COUNT( * ) FROM artist me LEFT JOIN cd cds ON cds.artist = me.artistid |
418 | # which actually returns the number of artists * (number of cds || 1) |
419 | # |
420 | # So what we do here is crawl {from}, determine if the current alias is at |
421 | # the top of the stack, and if not - make sure the chain is inner-joined down |
422 | # to the root. |
423 | # |
424 | sub _straight_join_to_node { |
425 | my ($self, $from, $alias) = @_; |
426 | |
427 | # subqueries and other oddness are naturally not supported |
428 | return $from if ( |
429 | ref $from ne 'ARRAY' |
430 | || |
431 | @$from <= 1 |
432 | || |
433 | ref $from->[0] ne 'HASH' |
434 | || |
435 | ! $from->[0]{-alias} |
436 | || |
7eb76996 |
437 | $from->[0]{-alias} eq $alias # this last bit means $alias is the head of $from - nothing to do |
289ac713 |
438 | ); |
439 | |
440 | # find the current $alias in the $from structure |
441 | my $switch_branch; |
442 | JOINSCAN: |
443 | for my $j (@{$from}[1 .. $#$from]) { |
444 | if ($j->[0]{-alias} eq $alias) { |
445 | $switch_branch = $j->[0]{-join_path}; |
446 | last JOINSCAN; |
447 | } |
448 | } |
449 | |
7eb76996 |
450 | # something else went quite wrong |
289ac713 |
451 | return $from unless $switch_branch; |
452 | |
453 | # So it looks like we will have to switch some stuff around. |
454 | # local() is useless here as we will be leaving the scope |
455 | # anyway, and deep cloning is just too fucking expensive |
7eb76996 |
456 | # So replace the first hashref in the node arrayref manually |
289ac713 |
457 | my @new_from = ($from->[0]); |
faeb2407 |
458 | my $sw_idx = { map { (values %$_), 1 } @$switch_branch }; #there's one k/v per join-path |
289ac713 |
459 | |
460 | for my $j (@{$from}[1 .. $#$from]) { |
461 | my $jalias = $j->[0]{-alias}; |
462 | |
463 | if ($sw_idx->{$jalias}) { |
464 | my %attrs = %{$j->[0]}; |
465 | delete $attrs{-join_type}; |
466 | push @new_from, [ |
467 | \%attrs, |
468 | @{$j}[ 1 .. $#$j ], |
469 | ]; |
470 | } |
471 | else { |
472 | push @new_from, $j; |
473 | } |
474 | } |
475 | |
476 | return \@new_from; |
477 | } |
478 | |
bac6c4fb |
479 | # Most databases do not allow aliasing of tables in UPDATE/DELETE. Thus |
480 | # a condition containing 'me' or other table prefixes will not work |
481 | # at all. What this code tries to do (badly) is introspect the condition |
482 | # and remove all column qualifiers. If it bails out early (returns undef) |
483 | # the calling code should try another approach (e.g. a subquery) |
484 | sub _strip_cond_qualifiers { |
485 | my ($self, $where) = @_; |
486 | |
487 | my $cond = {}; |
488 | |
489 | # No-op. No condition, we're updating/deleting everything |
490 | return $cond unless $where; |
491 | |
492 | if (ref $where eq 'ARRAY') { |
493 | $cond = [ |
494 | map { |
495 | my %hash; |
496 | foreach my $key (keys %{$_}) { |
497 | $key =~ /([^.]+)$/; |
498 | $hash{$1} = $_->{$key}; |
499 | } |
500 | \%hash; |
501 | } @$where |
502 | ]; |
503 | } |
504 | elsif (ref $where eq 'HASH') { |
505 | if ( (keys %$where) == 1 && ( (keys %{$where})[0] eq '-and' )) { |
506 | $cond->{-and} = []; |
507 | my @cond = @{$where->{-and}}; |
508 | for (my $i = 0; $i < @cond; $i++) { |
509 | my $entry = $cond[$i]; |
510 | my $hash; |
037e8dca |
511 | my $ref = ref $entry; |
512 | if ($ref eq 'HASH' or $ref eq 'ARRAY') { |
bac6c4fb |
513 | $hash = $self->_strip_cond_qualifiers($entry); |
514 | } |
037e8dca |
515 | elsif (! $ref) { |
bac6c4fb |
516 | $entry =~ /([^.]+)$/; |
517 | $hash->{$1} = $cond[++$i]; |
518 | } |
037e8dca |
519 | else { |
520 | $self->throw_exception ("_strip_cond_qualifiers() is unable to handle a condition reftype $ref"); |
521 | } |
bac6c4fb |
522 | push @{$cond->{-and}}, $hash; |
523 | } |
524 | } |
525 | else { |
526 | foreach my $key (keys %$where) { |
527 | $key =~ /([^.]+)$/; |
528 | $cond->{$1} = $where->{$key}; |
529 | } |
530 | } |
531 | } |
532 | else { |
533 | return undef; |
534 | } |
535 | |
536 | return $cond; |
537 | } |
538 | |
c0748280 |
539 | sub _parse_order_by { |
1a736efb |
540 | my ($self, $order_by, $sql_maker) = @_; |
c0748280 |
541 | |
1a736efb |
542 | my $parser = sub { |
543 | my ($sql_maker, $order_by) = @_; |
c0748280 |
544 | |
1a736efb |
545 | return scalar $sql_maker->_order_by_chunks ($order_by) |
546 | unless wantarray; |
c0748280 |
547 | |
1a736efb |
548 | my @chunks; |
549 | for my $chunk (map { ref $_ ? @$_ : $_ } ($sql_maker->_order_by_chunks ($order_by) ) ) { |
550 | $chunk =~ s/\s+ (?: ASC|DESC ) \s* $//ix; |
551 | push @chunks, $chunk; |
552 | } |
553 | |
554 | return @chunks; |
555 | }; |
556 | |
557 | if ($sql_maker) { |
558 | return $parser->($sql_maker, $order_by); |
559 | } |
560 | else { |
561 | $sql_maker = $self->sql_maker; |
562 | local $sql_maker->{quote_char}; |
563 | return $parser->($sql_maker, $order_by); |
564 | } |
c0748280 |
565 | } |
bac6c4fb |
566 | |
d28bb90d |
567 | 1; |