lift undef checking out of _order_by_chunks
Matt S Trout [Mon, 17 Sep 2018 03:32:02 +0000 (03:32 +0000)]
lib/SQL/Abstract.pm

index fc50257..ebec784 100644 (file)
@@ -1062,6 +1062,8 @@ sub _open_outer_paren {
 sub _order_by {
   my ($self, $arg) = @_;
 
+  return '' unless defined($arg);
+
   my @chunks = $self->_order_by_chunks($arg);
 
   my @sql;
@@ -1094,10 +1096,6 @@ sub _order_by_chunks {
   if (!ref($arg)) {
     return +{ -ident => $arg };
   }
-  if (!defined($arg)) {
-    # Seriously?
-    return
-  }
   if (ref($arg) eq 'HASH') {
     my ($key, $val, @rest) = %$arg;