Upgraded to SQL::Abstract 1.19 syntax, imported tests from S::A
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / SQL / Abstract.pm
index 8a45a2a..42657ac 100644 (file)
@@ -6,6 +6,7 @@ sub _debug { }
 
 sub _cond_resolve {
   my ($self, $cond, $attrs, $join) = @_;
+  $cond = $self->_anoncopy($cond);   # prevent destroying original
   my $ref   = ref $cond || '';
   $join   ||= $attrs->{logic} || ($ref eq 'ARRAY' ? 'OR' : 'AND');
   my $cmp   = uc($attrs->{cmp}) || '=';
@@ -15,17 +16,19 @@ sub _cond_resolve {
 
   # If an arrayref, then we join each element
   if ($ref eq 'ARRAY') {
+    use Data::Dumper;
+    #$self->_debug( Dumper($cond) );
     # need to use while() so can shift() for arrays
+    my $subjoin;
     while (my $el = shift @$cond) {
-      my $subjoin = 'OR';
-
+      
       # skip empty elements, otherwise get invalid trailing AND stuff
       if (my $ref2 = ref $el) {
         if ($ref2 eq 'ARRAY') {
           next unless @$el;
         } elsif ($ref2 eq 'HASH') {
           next unless %$el;
-          $subjoin = 'AND';
+          $subjoin ||= 'AND';
         } elsif ($ref2 eq 'SCALAR') {
           # literal SQL
           push @sqlf, $$el;
@@ -34,10 +37,11 @@ sub _cond_resolve {
         $self->_debug("$ref2(*top) means join with $subjoin");
       } else {
         # top-level arrayref with scalars, recurse in pairs
-        $self->_debug("NOREF(*top) means join with $subjoin");
+        $self->_debug("NOREF(*top) means join with $subjoin") if $subjoin;
         $el = {$el => shift(@$cond)};
       }
-      push @sqlf, scalar $self->_cond_resolve($el, $attrs, $subjoin);
+      my @ret = $self->_cond_resolve($el, $attrs, $subjoin);
+      push @sqlf, shift @ret;
     }
   }
   elsif ($ref eq 'HASH') {
@@ -45,19 +49,33 @@ sub _cond_resolve {
     # since it needs to point a column => value. So this be the end.
     for my $k (sort keys %$cond) {
       my $v = $cond->{$k};
-      if (! defined($v)) {
+      if ($k =~ /^-(.*)/) {
+        # special nesting, like -and, -or, -nest, so shift over
+        my $subjoin = $self->_modlogic($attrs, uc($1));
+        $self->_debug("OP(-$1) means special logic ($subjoin), recursing...");
+        my @ret = $self->_cond_resolve($v, $attrs, $subjoin);
+        push @sqlf, shift @ret;
+      } elsif (! defined($v)) {
         # undef = null
         $self->_debug("UNDEF($k) means IS NULL");
         push @sqlf, $k . ' IS NULL'
       } elsif (ref $v eq 'ARRAY') {
         # multiple elements: multiple options
-        $self->_debug("ARRAY($k) means multiple elements: [ @$v ]");
+        # warnings... $self->_debug("ARRAY($k) means multiple elements: [ @$v ]");
+        
+        # special nesting, like -and, -or, -nest, so shift over
+        my $subjoin = 'OR';
+        if ($v->[0] =~ /^-(.*)/) {
+          $subjoin = $self->_modlogic($attrs, uc($1));    # override subjoin
+          $self->_debug("OP(-$1) means special logic ($subjoin), shifting...");
+          shift @$v;
+        }
 
         # map into an array of hashrefs and recurse
-        my @w = ();
-        push @w, { $k => $_ } for @$v;
-        push @sqlf, scalar $self->_cond_resolve(\@w, $attrs, 'OR');
-
+        my @ret = $self->_cond_resolve([map { {$k => $_} } @$v], $attrs, $subjoin);
+        
+        # push results into our structure
+        push @sqlf, shift @ret;        
       } elsif (ref $v eq 'HASH') {
         # modified operator { '!=', 'completed' }
         for my $f (sort keys %$v) {
@@ -65,38 +83,44 @@ sub _cond_resolve {
           $self->_debug("HASH($k) means modified operator: { $f }");
 
           # check for the operator being "IN" or "BETWEEN" or whatever
-          if ($f =~ /^([\s\w]+)$/i && ref $x eq 'ARRAY') {
-            my $u = uc($1);
-            if ($u =~ /BETWEEN/) {
-              # SQL sucks
-              $self->throw( "BETWEEN must have exactly two arguments" ) unless @$x == 2;
-              push @sqlf, join ' ',
-                            $self->_cond_key($attrs => $k), $u,
-                            $self->_cond_value($attrs => $k => $x->[0]),
-                            'AND',
-                            $self->_cond_value($attrs => $k => $x->[1]);
+          if (ref $x eq 'ARRAY') {
+            if ($f =~ /^-?\s*(not[\s_]+)?(in|between)\s*$/i) {
+              my $mod = $1 ? $1 . $2 : $2;  # avoid uninitialized value warnings
+              my $u = $self->_modlogic($attrs, uc($mod));
+              $self->_debug("HASH($f => $x) uses special operator: [ $u ]");
+              if ($u =~ /BETWEEN/) {
+                # SQL sucks
+                $self->throw( "BETWEEN must have exactly two arguments" ) unless @$x == 2;
+                push @sqlf, join ' ',
+                              $self->_cond_key($attrs => $k), $u,
+                              $self->_cond_value($attrs => $k => $x->[0]),
+                              'AND',
+                              $self->_cond_value($attrs => $k => $x->[1]);
+              } else {
+                push @sqlf, join ' ', $self->_cond_key($attrs, $k), $u, '(',
+                        join(', ',
+                          map { $self->_cond_value($attrs, $k, $_) } @$x),
+                      ')';
+              }
             } else {
-              push @sqlf, join ' ', $self->_cond_key($attrs, $k), $u, '(',
-                      join(', ',
-                        map { $self->_cond_value($attrs, $k, $_) } @$x),
-                    ')';
-            }
-          } elsif (ref $x eq 'ARRAY') {
-            # multiple elements: multiple options
-            $self->_debug("ARRAY($x) means multiple elements: [ @$x ]");
-
-            # map into an array of hashrefs and recurse
-            my @w = ();
-            push @w, { $k => { $f => $_ } } for @$x;
-            push @sqlf, scalar $self->_cond_resolve(\@w, $attrs, 'OR');
+              # multiple elements: multiple options
+              $self->_debug("ARRAY($x) means multiple elements: [ @$x ]");
+  
+              # map into an array of hashrefs and recurse
+              my @ret = $self->_cond_resolve([map { {$k => {$f, $_}} } @$x], $attrs);
 
+              # push results into our structure
+              push @sqlf, shift @ret;              
+            }
           } elsif (! defined($x)) {
             # undef = NOT null
             my $not = ($f eq '!=' || $f eq 'not like') ? ' NOT' : '';
             push @sqlf, $self->_cond_key($attrs => $k) . " IS${not} NULL";
           } else {
             # regular ol' value
-            push @sqlf, join ' ', $self->_cond_key($attrs => $k), $f,
+            $f =~ s/^-//;   # strip leading -like =>
+            $f =~ s/_/ /;   # _ => " "
+            push @sqlf, join ' ', $self->_cond_key($attrs => $k), uc($f),
                           $self->_cond_value($attrs => $k => $x);
           }
         }
@@ -138,6 +162,22 @@ sub _cond_value {
   push(@{$attrs->{bind}}, $value);
   return '?';
 }
+
+# Anon copies of arrays/hashes
+sub _anoncopy {
+  my ($self, $orig) = @_;
+  return (ref $orig eq 'HASH' ) ? { %$orig }
+     : (ref $orig eq 'ARRAY') ? [ @$orig ]
+     : $orig;     # rest passthru ok
+}
+
+sub _modlogic {
+  my ($self, $attrs, $sym) = @_;
+  $sym ||= $attrs->{logic};
+  $sym =~ tr/_/ /;
+  $sym = $attrs->{logic} if $sym eq 'nest';
+  return uc($sym);  # override join
+}
   
 1;