# GLOBALS
#======================================================================
-our $VERSION = '1.51';
+our $VERSION = '1.53';
# This would confuse some packagers
#$VERSION = eval $VERSION; # numify for warning-free dev releases
my ($self, $where) = @_;
my (@sql_clauses, @all_bind);
- # LDNOTE : don't really know why we need to sort keys
for my $k (sort keys %$where) {
my $v = $where->{$k};
sub _where_op_in_hash {
my ($self, $op_str, $v) = @_;
- $op_str =~ /^ (AND|OR|NEST) ( \_? \d* ) $/xi
+ $op_str =~ /^ (AND|OR|PAREN|NEST) ( \_? \d* ) $/xi
or puke "unknown operator: -$op_str";
my $op = uc($1); # uppercase, remove trailing digits
if ($2) {
- belch 'Use of [and|or|nest]_N modifiers is deprecated and will be removed in SQLA v2.0. '
+ belch 'Use of op_N modifiers is deprecated and will be removed in SQLA v2.0. '
. "You probably wanted ...-and => [ $op_str => COND1, $op_str => COND2 ... ]";
}
+ if ($op eq 'NEST') {
+ belch 'The -nest modifier is deprecated in favor of -paren and will be removed in SQLA v2.0';
+ $op = 'PAREN';
+ }
+
$self->_debug("OP(-$op) within hashref, recursing...");
$self->_SWITCH_refkind($v, {
ARRAYREF => sub {
- return $self->_where_ARRAYREF($v, $op eq 'NEST' ? '' : $op);
+ return $self->_where_ARRAYREF($v, $op eq 'PAREN' ? '' : $op);
},
HASHREF => sub {
if ($op eq 'OR') {
- return $self->_where_ARRAYREF([%$v], 'OR');
+ return $self->_where_ARRAYREF([ map { $_ => $v->{$_} } (sort keys %$v) ], 'OR');
}
- else { # NEST | AND
+ else { # PAREN | AND
return $self->_where_HASHREF($v);
}
},
SCALARREF => sub { # literal SQL
- $op eq 'NEST'
- or puke "-$op => \\\$scalar not supported, use -nest => ...";
+ $op eq 'PAREN'
+ or puke "-$op => \\\$scalar not supported, use -paren => ...";
return ($$v);
},
ARRAYREFREF => sub { # literal SQL
- $op eq 'NEST'
- or puke "-$op => \\[..] not supported, use -nest => ...";
+ $op eq 'PAREN'
+ or puke "-$op => \\[..] not supported, use -paren => ...";
return @{${$v}};
},
SCALAR => sub { # permissively interpreted as SQL
- $op eq 'NEST'
- or puke "-$op => 'scalar' not supported, use -nest => \\'scalar'";
- belch "literal SQL should be -nest => \\'scalar' "
- . "instead of -nest => 'scalar' ";
+ $op eq 'PAREN'
+ or puke "-$op => 'scalar' not supported, use -paren => \\'scalar'";
+ belch "literal SQL should be -paren => \\'scalar' "
+ . "instead of -paren => 'scalar' ";
return ($v);
},
if(@$vals) {
$self->_debug("ARRAY($vals) means multiple elements: [ @$vals ]");
+ # see if the first element is an -and/-or op
+ my $logic;
+ if ($vals->[0] =~ /^ - ( AND|OR ) $/ix) {
+ $logic = uc $1;
+ shift @$vals;
+ }
+
+ # distribute $op over each remaining member of @$vals, append logic if exists
+ return $self->_recurse_where([map { {$k => {$op, $_}} } @$vals], $logic);
+
# LDNOTE : had planned to change the distribution logic when
# $op =~ $self->{inequality_op}, because of Morgan laws :
# with {field => {'!=' => [22, 33]}}, it would be ridiculous to generate
# WHERE field != 22 OR field != 33 : the user probably means
# WHERE field != 22 AND field != 33.
- # To do this, replace the line below by :
+ # To do this, replace the above to roughly :
# my $logic = ($op =~ $self->{inequality_op}) ? 'AND' : 'OR';
# return $self->_recurse_where([map { {$k => {$op, $_}} } @$vals], $logic);
- # distribute $op over each member of @$vals
- return $self->_recurse_where([map { {$k => {$op, $_}} } @$vals]);
}
else {
# try to DWIM on equality operators
@bind = ('nwiger', 'pending', 'dispatched', 'robot', 'unassigned');
-There is also a special C<-nest>
+There is also a special C<-paren>
operator which adds an additional set of parens, to create a subquery.
For example, to get something like this:
my %where = (
user => 'nwiger',
- -nest => [ workhrs => {'>', 20}, geo => 'ASIA' ],
+ -paren => [ workhrs => {'>', 20}, geo => 'ASIA' ],
);
my @where = (
-and => [
user => 'nwiger',
- -nest => [
+ -paren => [
-and => [workhrs => {'>', 20}, geo => 'ASIA' ],
-and => [workhrs => {'<', 50}, geo => 'EURO' ]
],
= $sql->select("t1", "*", {c1 => 1, c2 => \"> t0.c0"});
my %where = (
foo => 1234,
- -nest => \["EXISTS ($sub_stmt)" => @sub_bind],
+ -paren => \["EXISTS ($sub_stmt)" => @sub_bind],
);
which yields
NATIVE clauses, etc. Here is an example of a fulltext query in MySQL :
my %where = (
- -nest => \["MATCH (col1, col2) AGAINST (?)" => qw/apples/]
+ -paren => \["MATCH (col1, col2) AGAINST (?)" => qw/apples/]
);
Finally, here is an example where a subquery is used
$sub_stmt =~ s/^ where //i; # don't want "WHERE" in the subclause
my %where = (
lname => {like => '%son%'},
- -nest => \["NOT ($sub_stmt)" => @sub_bind],
+ -paren => \["NOT ($sub_stmt)" => @sub_bind],
);
This yields
Mike Fragassi (enhancements to "BETWEEN" and "LIKE")
Dan Kubb (support for "quote_char" and "name_sep")
Guillermo Roditi (patch to cleanup "IN" and "BETWEEN", fix and tests for _order_by)
- Laurent Dami (internal refactoring, multiple -nest, extensible list of special operators, literal SQL)
+ Laurent Dami (internal refactoring, multiple -paren, extensible list of special operators, literal SQL)
Norbert Buchmuller (support for literal SQL in hashpair, misc. fixes & tests)
+ Peter Rabbitson (rewrite of SQLA::Test, misc. fixes & tests)
Thanks!
C<SQL::Abstract>, and as such list members there are very familiar with
how to create queries.
+=head1 LICENSE
+
This module is free software; you may copy this under the terms of
the GNU General Public License, or the Artistic License, copies of
which should have accompanied your Perl kit.