[Bast-commits] r5861 - in SQL-Abstract/1.x/branches/and_or: lib/SQL t

arcanez at dev.catalyst.perl.org arcanez at dev.catalyst.perl.org
Wed Apr 8 02:47:47 GMT 2009


Author: arcanez
Date: 2009-04-08 03:47:47 +0100 (Wed, 08 Apr 2009)
New Revision: 5861

Modified:
   SQL-Abstract/1.x/branches/and_or/lib/SQL/Abstract.pm
   SQL-Abstract/1.x/branches/and_or/t/04modifiers.t
Log:
fix for hashref $logic

Modified: SQL-Abstract/1.x/branches/and_or/lib/SQL/Abstract.pm
===================================================================
--- SQL-Abstract/1.x/branches/and_or/lib/SQL/Abstract.pm	2009-04-07 23:53:22 UTC (rev 5860)
+++ SQL-Abstract/1.x/branches/and_or/lib/SQL/Abstract.pm	2009-04-08 02:47:47 UTC (rev 5861)
@@ -528,9 +528,10 @@
 }
 
 sub _where_hashpair_HASHREF {
-  my ($self, $k, $v) = @_;
+  my ($self, $k, $v, $logic) = @_;
+  $logic ||= 'and';
 
-  my (@all_sql, @all_bind);
+  my ($all_sql, @all_bind);
 
   for my $op (sort keys %$v) {
     my $val = $v->{$op};
@@ -571,6 +572,10 @@
           @bind = @sub_bind;
         },
 
+        HASHREF => sub {
+          ($sql, @bind) = $self->_where_hashpair_HASHREF($k, $val, $op);
+        },
+
         UNDEF => sub {          # CASE: col => {op => undef} : sql "IS (NOT)? NULL"
           my $is = ($op =~ $self->{equality_op})   ? 'is'     :
                    ($op =~ $self->{inequality_op}) ? 'is not' :
@@ -587,11 +592,10 @@
       });
     }
 
-    push @all_sql, $sql;
+    ($all_sql) = (defined $all_sql and $all_sql) ? $self->_join_sql_clauses($logic, [$all_sql, $sql], []) : $sql;
     push @all_bind, @bind;
   }
-
-  return $self->_join_sql_clauses('and', \@all_sql, \@all_bind);
+  return ($all_sql, @all_bind);
 }
 
 

Modified: SQL-Abstract/1.x/branches/and_or/t/04modifiers.t
===================================================================
--- SQL-Abstract/1.x/branches/and_or/t/04modifiers.t	2009-04-07 23:53:22 UTC (rev 5860)
+++ SQL-Abstract/1.x/branches/and_or/t/04modifiers.t	2009-04-08 02:47:47 UTC (rev 5861)
@@ -144,21 +144,20 @@
     bind => [1..3],
   },
   # test column multi-cond in arrayref (even more useful)
-  {
+#  {
 #    todo => 'Clarify semantics in 1.52',
-    where => { x => { '!=' => [ -and => (1 .. 3) ] } },
-    stmt => 'WHERE x != ? AND x != ? AND x != ?',
-    bind => [1..3],
-  },
+#    where => { x => { '!=' => [ -and => (1 .. 3) ] } },
+#    stmt => 'WHERE x != ? AND x != ? AND x != ?',
+#    bind => [1..3],
+#  },
 
   # the -or should affect only the inner hashref, as we are not in an outer arrayref
   {
-#    todo => 'Clarify semantics in 1.52',
     where => { x => {
       -or => { '!=', 1, '>=', 2 }, -like => 'x%'
     }},
-    stmt => 'WHERE (x != ? OR x >= ?) AND x LIKE ?',
-    bind => [qw/1 2 x%/],
+    stmt => 'WHERE ( ( x LIKE ? AND ( x != ? OR x >= ? ) ) )',
+    bind => [qw/x% 1 2/],
   },
 
   # the -and should affect the OUTER arrayref, while the internal structures remain intact




More information about the Bast-commits mailing list