Fix multi-value literal populate not working with simplified bind spec
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / SQLMaker.pm
CommitLineData
d5dedbd6 1package DBIx::Class::SQLMaker;
6f4ddea1 2
a697fa31 3use strict;
4use warnings;
5
d5dedbd6 6=head1 NAME
7
8DBIx::Class::SQLMaker - An SQL::Abstract-based SQL maker class
9
10=head1 DESCRIPTION
11
12This module is a subclass of L<SQL::Abstract> and includes a number of
13DBIC-specific workarounds, not yet suitable for inclusion into the
14L<SQL::Abstract> core. It also provides all (and more than) the functionality
15of L<SQL::Abstract::Limit>, see L<DBIx::Class::SQLMaker::LimitDialects> for
16more info.
17
18Currently the enhancements to L<SQL::Abstract> are:
19
20=over
21
22=item * Support for C<JOIN> statements (via extended C<table/from> support)
23
24=item * Support of functions in C<SELECT> lists
25
26=item * C<GROUP BY>/C<HAVING> support (via extensions to the order_by parameter)
27
28=item * Support of C<...FOR UPDATE> type of select statement modifiers
29
30=back
31
32=cut
6a247f33 33
34use base qw/
d5dedbd6 35 DBIx::Class::SQLMaker::LimitDialects
6a247f33 36 SQL::Abstract
70c28808 37 DBIx::Class
6a247f33 38/;
39use mro 'c3';
a697fa31 40
6298a324 41use Sub::Name 'subname';
70c28808 42use DBIx::Class::Carp;
e8fc51c7 43use namespace::clean;
b2b22cd6 44
6a247f33 45__PACKAGE__->mk_group_accessors (simple => qw/quote_char name_sep limit_dialect/);
46
3f5b99fe 47# for when I need a normalized l/r pair
48sub _quote_chars {
49 map
50 { defined $_ ? $_ : '' }
51 ( ref $_[0]->{quote_char} ? (@{$_[0]->{quote_char}}) : ( ($_[0]->{quote_char}) x 2 ) )
52 ;
53}
54
70c28808 55# FIXME when we bring in the storage weaklink, check its schema
56# weaklink and channel through $schema->throw_exception
57sub throw_exception { DBIx::Class::Exception->throw($_[1]) }
58
b2b22cd6 59BEGIN {
2ea6032a 60 # reinstall the belch()/puke() functions of SQL::Abstract with custom versions
70c28808 61 # that use DBIx::Class::Carp/DBIx::Class::Exception instead of plain Carp
b2b22cd6 62 no warnings qw/redefine/;
2ea6032a 63
64 *SQL::Abstract::belch = subname 'SQL::Abstract::belch' => sub (@) {
65 my($func) = (caller(1))[3];
66 carp "[$func] Warning: ", @_;
67 };
68
69 *SQL::Abstract::puke = subname 'SQL::Abstract::puke' => sub (@) {
70 my($func) = (caller(1))[3];
70c28808 71 __PACKAGE__->throw_exception("[$func] Fatal: " . join ('', @_));
2ea6032a 72 };
b2b22cd6 73}
6f4ddea1 74
e9657379 75# the "oh noes offset/top without limit" constant
fcb7fcbb 76# limited to 31 bits for sanity (and consistency,
77# since it may be handed to the like of sprintf %u)
78#
79# Also *some* builds of SQLite fail the test
80# some_column BETWEEN ? AND ?: 1, 4294967295
81# with the proper integer bind attrs
82#
6a247f33 83# Implemented as a method, since ::Storage::DBI also
84# refers to it (i.e. for the case of software_limit or
85# as the value to abuse with MSSQL ordered subqueries)
fcb7fcbb 86sub __max_int () { 0x7FFFFFFF };
e9657379 87
1b5ddf23 88# we ne longer need to check this - DBIC has ways of dealing with it
89# specifically ::Storage::DBI::_resolve_bindattrs()
90sub _assert_bindval_matches_bindtype () { 1 };
91
e39f188a 92# poor man's de-qualifier
93sub _quote {
94 $_[0]->next::method( ( $_[0]{_dequalify_idents} and ! ref $_[1] )
95 ? $_[1] =~ / ([^\.]+) $ /x
96 : $_[1]
97 );
98}
99
b1d821de 100sub _where_op_NEST {
70c28808 101 carp_unique ("-nest in search conditions is deprecated, you most probably wanted:\n"
b1d821de 102 .q|{..., -and => [ \%cond0, \@cond1, \'cond2', \[ 'cond3', [ col => bind ] ], etc. ], ... }|
70c28808 103 );
b1d821de 104
105 shift->next::method(@_);
106}
107
6a247f33 108# Handle limit-dialect selection
6f4ddea1 109sub select {
6a247f33 110 my ($self, $table, $fields, $where, $rs_attrs, $limit, $offset) = @_;
111
112
113 $fields = $self->_recurse_fields($fields);
114
115 if (defined $offset) {
70c28808 116 $self->throw_exception('A supplied offset must be a non-negative integer')
6a247f33 117 if ( $offset =~ /\D/ or $offset < 0 );
118 }
119 $offset ||= 0;
1cbd3034 120
6a247f33 121 if (defined $limit) {
70c28808 122 $self->throw_exception('A supplied limit must be a positive integer')
6a247f33 123 if ( $limit =~ /\D/ or $limit <= 0 );
124 }
125 elsif ($offset) {
126 $limit = $self->__max_int;
6f4ddea1 127 }
c2b7c5dc 128
a6b68a60 129
6a247f33 130 my ($sql, @bind);
131 if ($limit) {
132 # this is legacy code-flow from SQLA::Limit, it is not set in stone
133
134 ($sql, @bind) = $self->next::method ($table, $fields, $where);
135
67341081 136 my $limiter;
137
138 if( $limiter = $self->can ('emulate_limit') ) {
139 carp_unique(
140 'Support for the legacy emulate_limit() mechanism inherited from '
141 . 'SQL::Abstract::Limit has been deprecated, and will be removed when '
142 . 'DBIC transitions to Data::Query. If your code uses this type of '
143 . 'limit specification please file an RT and provide the source of '
144 . 'your emulate_limit() implementation, so an acceptable upgrade-path '
145 . 'can be devised'
146 );
147 }
148 else {
149 my $dialect = $self->limit_dialect
150 or $self->throw_exception( "Unable to generate SQL-limit - no limit dialect specified on $self" );
151
152 $limiter = $self->can ("_$dialect")
153 or $self->throw_exception(__PACKAGE__ . " does not implement the requested dialect '$dialect'");
154 }
6a247f33 155
f74d22e2 156 $sql = $self->$limiter (
157 $sql,
158 { %{$rs_attrs||{}}, _selector_sql => $fields },
159 $limit,
160 $offset
161 );
6a247f33 162 }
163 else {
164 ($sql, @bind) = $self->next::method ($table, $fields, $where, $rs_attrs);
165 }
166
49afd714 167 push @{$self->{where_bind}}, @bind;
583a0c65 168
169# this *must* be called, otherwise extra binds will remain in the sql-maker
49afd714 170 my @all_bind = $self->_assemble_binds;
583a0c65 171
e5372da4 172 $sql .= $self->_lock_select ($rs_attrs->{for})
173 if $rs_attrs->{for};
174
49afd714 175 return wantarray ? ($sql, @all_bind) : $sql;
583a0c65 176}
177
178sub _assemble_binds {
179 my $self = shift;
8b31f62e 180 return map { @{ (delete $self->{"${_}_bind"}) || [] } } (qw/pre_select select from where group having order limit/);
6f4ddea1 181}
182
e5372da4 183my $for_syntax = {
184 update => 'FOR UPDATE',
185 shared => 'FOR SHARE',
186};
187sub _lock_select {
188 my ($self, $type) = @_;
8249c09b 189
190 my $sql;
191 if (ref($type) eq 'SCALAR') {
192 $sql = "FOR $$type";
193 }
194 else {
195 $sql = $for_syntax->{$type} || $self->throw_exception( "Unknown SELECT .. FOR type '$type' requested" );
196 }
197
e5372da4 198 return " $sql";
199}
200
6a247f33 201# Handle default inserts
6f4ddea1 202sub insert {
6a247f33 203# optimized due to hotttnesss
204# my ($self, $table, $data, $options) = @_;
7a72e5a5 205
206 # SQLA will emit INSERT INTO $table ( ) VALUES ( )
207 # which is sadly understood only by MySQL. Change default behavior here,
208 # until SQLA2 comes with proper dialect support
6a247f33 209 if (! $_[2] or (ref $_[2] eq 'HASH' and !keys %{$_[2]} ) ) {
bf51641f 210 my @bind;
20595c02 211 my $sql = sprintf(
212 'INSERT INTO %s DEFAULT VALUES', $_[0]->_quote($_[1])
213 );
28d28903 214
bf51641f 215 if ( ($_[3]||{})->{returning} ) {
216 my $s;
217 ($s, @bind) = $_[0]->_insert_returning ($_[3]);
218 $sql .= $s;
28d28903 219 }
220
bf51641f 221 return ($sql, @bind);
7a72e5a5 222 }
223
6a247f33 224 next::method(@_);
6f4ddea1 225}
226
227sub _recurse_fields {
81446c4f 228 my ($self, $fields) = @_;
6f4ddea1 229 my $ref = ref $fields;
230 return $self->_quote($fields) unless $ref;
231 return $$fields if $ref eq 'SCALAR';
232
233 if ($ref eq 'ARRAY') {
81446c4f 234 return join(', ', map { $self->_recurse_fields($_) } @$fields);
83e09b5b 235 }
236 elsif ($ref eq 'HASH') {
81446c4f 237 my %hash = %$fields; # shallow copy
83e09b5b 238
50136dd9 239 my $as = delete $hash{-as}; # if supplied
240
81446c4f 241 my ($func, $args, @toomany) = %hash;
242
243 # there should be only one pair
244 if (@toomany) {
70c28808 245 $self->throw_exception( "Malformed select argument - too many keys in hash: " . join (',', keys %$fields ) );
81446c4f 246 }
50136dd9 247
248 if (lc ($func) eq 'distinct' && ref $args eq 'ARRAY' && @$args > 1) {
70c28808 249 $self->throw_exception (
50136dd9 250 'The select => { distinct => ... } syntax is not supported for multiple columns.'
251 .' Instead please use { group_by => [ qw/' . (join ' ', @$args) . '/ ] }'
252 .' or { select => [ qw/' . (join ' ', @$args) . '/ ], distinct => 1 }'
83e09b5b 253 );
6f4ddea1 254 }
83e09b5b 255
50136dd9 256 my $select = sprintf ('%s( %s )%s',
257 $self->_sqlcase($func),
258 $self->_recurse_fields($args),
259 $as
0491b597 260 ? sprintf (' %s %s', $self->_sqlcase('as'), $self->_quote ($as) )
50136dd9 261 : ''
262 );
263
83e09b5b 264 return $select;
6f4ddea1 265 }
266 # Is the second check absolutely necessary?
267 elsif ( $ref eq 'REF' and ref($$fields) eq 'ARRAY' ) {
4c2b30d6 268 push @{$self->{select_bind}}, @{$$fields}[1..$#$$fields];
269 return $$fields->[0];
6f4ddea1 270 }
271 else {
70c28808 272 $self->throw_exception( $ref . qq{ unexpected in _recurse_fields()} );
6f4ddea1 273 }
274}
275
a6b68a60 276
277# this used to be a part of _order_by but is broken out for clarity.
278# What we have been doing forever is hijacking the $order arg of
279# SQLA::select to pass in arbitrary pieces of data (first the group_by,
280# then pretty much the entire resultset attr-hash, as more and more
4a0eed52 281# things in the SQLA space need to have more info about the $rs they
a6b68a60 282# create SQL for. The alternative would be to keep expanding the
283# signature of _select with more and more positional parameters, which
284# is just gross. All hail SQLA2!
285sub _parse_rs_attrs {
1cbd3034 286 my ($self, $arg) = @_;
15827712 287
a6b68a60 288 my $sql = '';
1cbd3034 289
0542ec57 290 if ($arg->{group_by}) {
a4f0b0ab 291 # horrible horrible, waiting for refactor
0542ec57 292 local $self->{select_bind};
293 if (my $g = $self->_recurse_fields($arg->{group_by}) ) {
294 $sql .= $self->_sqlcase(' group by ') . $g;
295 push @{$self->{group_bind} ||= []}, @{$self->{select_bind}||[]};
296 }
a6b68a60 297 }
1cbd3034 298
a6b68a60 299 if (defined $arg->{having}) {
300 my ($frag, @bind) = $self->_recurse_where($arg->{having});
301 push(@{$self->{having_bind}}, @bind);
302 $sql .= $self->_sqlcase(' having ') . $frag;
303 }
15827712 304
a6b68a60 305 if (defined $arg->{order_by}) {
306 $sql .= $self->_order_by ($arg->{order_by});
307 }
15827712 308
a6b68a60 309 return $sql;
310}
311
312sub _order_by {
313 my ($self, $arg) = @_;
15827712 314
a6b68a60 315 # check that we are not called in legacy mode (order_by as 4th argument)
316 if (ref $arg eq 'HASH' and not grep { $_ =~ /^-(?:desc|asc)/i } keys %$arg ) {
317 return $self->_parse_rs_attrs ($arg);
fde3719a 318 }
1cbd3034 319 else {
6a247f33 320 my ($sql, @bind) = $self->next::method($arg);
a6b68a60 321 push @{$self->{order_bind}}, @bind;
1cbd3034 322 return $sql;
fd4cb60a 323 }
6f4ddea1 324}
325
cb3e87f5 326sub _split_order_chunk {
327 my ($self, $chunk) = @_;
328
329 # strip off sort modifiers, but always succeed, so $1 gets reset
330 $chunk =~ s/ (?: \s+ (ASC|DESC) )? \s* $//ix;
331
332 return (
333 $chunk,
334 ( $1 and uc($1) eq 'DESC' ) ? 1 : 0,
335 );
336}
337
6f4ddea1 338sub _table {
6a247f33 339# optimized due to hotttnesss
340# my ($self, $from) = @_;
341 if (my $ref = ref $_[1] ) {
342 if ($ref eq 'ARRAY') {
343 return $_[0]->_recurse_from(@{$_[1]});
344 }
345 elsif ($ref eq 'HASH') {
4c2b30d6 346 return $_[0]->_recurse_from($_[1]);
6a247f33 347 }
1bffc6b8 348 elsif ($ref eq 'REF' && ref ${$_[1]} eq 'ARRAY') {
349 my ($sql, @bind) = @{ ${$_[1]} };
350 push @{$_[0]->{from_bind}}, @bind;
351 return $sql
352 }
6f4ddea1 353 }
6a247f33 354 return $_[0]->next::method ($_[1]);
6f4ddea1 355}
356
b8391c87 357sub _generate_join_clause {
358 my ($self, $join_type) = @_;
359
726c8f65 360 $join_type = $self->{_default_jointype}
361 if ! defined $join_type;
362
b8391c87 363 return sprintf ('%s JOIN ',
726c8f65 364 $join_type ? $self->_sqlcase($join_type) : ''
b8391c87 365 );
366}
367
6f4ddea1 368sub _recurse_from {
726c8f65 369 my $self = shift;
726c8f65 370 return join (' ', $self->_gen_from_blocks(@_) );
371}
372
373sub _gen_from_blocks {
374 my ($self, $from, @joins) = @_;
375
376 my @fchunks = $self->_from_chunk_to_sql($from);
6f4ddea1 377
726c8f65 378 for (@joins) {
4c2b30d6 379 my ($to, $on) = @$_;
aa82ce29 380
6f4ddea1 381 # check whether a join type exists
6f4ddea1 382 my $to_jt = ref($to) eq 'ARRAY' ? $to->[0] : $to;
aa82ce29 383 my $join_type;
384 if (ref($to_jt) eq 'HASH' and defined($to_jt->{-join_type})) {
385 $join_type = $to_jt->{-join_type};
386 $join_type =~ s/^\s+ | \s+$//xg;
6f4ddea1 387 }
aa82ce29 388
726c8f65 389 my @j = $self->_generate_join_clause( $join_type );
6f4ddea1 390
391 if (ref $to eq 'ARRAY') {
726c8f65 392 push(@j, '(', $self->_recurse_from(@$to), ')');
393 }
394 else {
395 push(@j, $self->_from_chunk_to_sql($to));
6f4ddea1 396 }
726c8f65 397
a697fa31 398 my ($sql, @bind) = $self->_join_condition($on);
b4e9f590 399 push(@j, ' ON ', $sql);
a697fa31 400 push @{$self->{from_bind}}, @bind;
726c8f65 401
402 push @fchunks, join '', @j;
6f4ddea1 403 }
726c8f65 404
405 return @fchunks;
6f4ddea1 406}
407
4c2b30d6 408sub _from_chunk_to_sql {
409 my ($self, $fromspec) = @_;
410
e8885a53 411 return join (' ', do {
412 if (! ref $fromspec) {
413 $self->_quote($fromspec);
414 }
415 elsif (ref $fromspec eq 'SCALAR') {
4c2b30d6 416 $$fromspec;
e8885a53 417 }
418 elsif (ref $fromspec eq 'REF' and ref $$fromspec eq 'ARRAY') {
4c2b30d6 419 push @{$self->{from_bind}}, @{$$fromspec}[1..$#$$fromspec];
420 $$fromspec->[0];
e8885a53 421 }
422 elsif (ref $fromspec eq 'HASH') {
4c2b30d6 423 my ($as, $table, $toomuch) = ( map
424 { $_ => $fromspec->{$_} }
425 ( grep { $_ !~ /^\-/ } keys %$fromspec )
426 );
6f4ddea1 427
70c28808 428 $self->throw_exception( "Only one table/as pair expected in from-spec but an exra '$toomuch' key present" )
4c2b30d6 429 if defined $toomuch;
6f4ddea1 430
4c2b30d6 431 ($self->_from_chunk_to_sql($table), $self->_quote($as) );
e8885a53 432 }
433 else {
434 $self->throw_exception('Unsupported from refkind: ' . ref $fromspec );
435 }
436 });
6f4ddea1 437}
438
439sub _join_condition {
440 my ($self, $cond) = @_;
4c2b30d6 441
a697fa31 442 # Backcompat for the old days when a plain hashref
443 # { 't1.col1' => 't2.col2' } meant ON t1.col1 = t2.col2
444 # Once things settle we should start warning here so that
445 # folks unroll their hacks
446 if (
447 ref $cond eq 'HASH'
448 and
449 keys %$cond == 1
450 and
451 (keys %$cond)[0] =~ /\./
452 and
453 ! ref ( (values %$cond)[0] )
454 ) {
455 $cond = { keys %$cond => { -ident => values %$cond } }
6f4ddea1 456 }
a697fa31 457 elsif ( ref $cond eq 'ARRAY' ) {
458 # do our own ORing so that the hashref-shim above is invoked
9aae3566 459 my @parts;
460 my @binds;
461 foreach my $c (@$cond) {
462 my ($sql, @bind) = $self->_join_condition($c);
463 push @binds, @bind;
464 push @parts, $sql;
465 }
466 return join(' OR ', @parts), @binds;
6f4ddea1 467 }
a697fa31 468
469 return $self->_recurse_where($cond);
6f4ddea1 470}
471
66137dff 472# This is hideously ugly, but SQLA does not understand multicol IN expressions
473# FIXME TEMPORARY - DQ should have native syntax for this
474# moved here to raise API questions
475#
476# !!! EXPERIMENTAL API !!! WILL CHANGE !!!
477sub _where_op_multicolumn_in {
478 my ($self, $lhs, $rhs) = @_;
479
480 if (! ref $lhs or ref $lhs eq 'ARRAY') {
481 my (@sql, @bind);
482 for (ref $lhs ? @$lhs : $lhs) {
483 if (! ref $_) {
484 push @sql, $self->_quote($_);
485 }
486 elsif (ref $_ eq 'SCALAR') {
487 push @sql, $$_;
488 }
489 elsif (ref $_ eq 'REF' and ref $$_ eq 'ARRAY') {
490 my ($s, @b) = @$$_;
491 push @sql, $s;
492 push @bind, @b;
493 }
494 else {
495 $self->throw_exception("ARRAY of @{[ ref $_ ]}es unsupported for multicolumn IN lhs...");
496 }
497 }
498 $lhs = \[ join(', ', @sql), @bind];
499 }
500 elsif (ref $lhs eq 'SCALAR') {
501 $lhs = \[ $$lhs ];
502 }
503 elsif (ref $lhs eq 'REF' and ref $$lhs eq 'ARRAY' ) {
504 # noop
505 }
506 else {
507 $self->throw_exception( ref($lhs) . "es unsupported for multicolumn IN lhs...");
508 }
509
510 # is this proper...?
511 $rhs = \[ $self->_recurse_where($rhs) ];
512
513 for ($lhs, $rhs) {
514 $$_->[0] = "( $$_->[0] )"
1d1ccc94 515 unless $$_->[0] =~ /^ \s* \( .* \) \s* $/xs;
66137dff 516 }
517
518 \[ join( ' IN ', shift @$$lhs, shift @$$rhs ), @$$lhs, @$$rhs ];
519}
520
6f4ddea1 5211;
d5dedbd6 522
523=head1 AUTHORS
524
525See L<DBIx::Class/CONTRIBUTORS>.
526
527=head1 LICENSE
528
529You may distribute this code under the same terms as Perl itself.
530
531=cut