Fix backend behavior for nested query drag-to-filter

src/metabase/query_processor/interface.clj

@@ -231,9 +231,14 @@
                                     field  :- (s/cond-pre DateTimeField
                                                           FieldPlaceholder)])
 
+;; e.g. an absolute point in time (literal)
+(s/defrecord DateTimeValue [value :- Timestamp
+                            field :- DateTimeField])
+
 (def OrderableValue
   "Schema for an instance of `Value` whose `:value` property is itself orderable (a datetime or number, i.e. a `OrderableValueLiteral`)."
   (s/named (s/cond-pre
+            DateTimeValue
             RelativeDateTimeValue
             (s/constrained Value (fn [{value :value}]
                                    (nil? (s/check OrderableValueLiteral value)))))
@@ -244,10 +249,6 @@
   (s/named (s/constrained Value (comp string? :value))
            "Value that is a string (Value whose :value is a string)"))
 
-;; e.g. an absolute point in time (literal)
-(s/defrecord DateTimeValue [value :- Timestamp
-                            field :- DateTimeField])
-
 (defprotocol ^:private IDateTimeValue
   (unit [this]
     "Get the `unit` associated with a `DateTimeValue` or `RelativeDateTimeValue`.")

test/metabase/query_processor_test/nested_queries_test.clj

@@ -416,3 +416,18 @@
           :breakout    [[:datetime-field [:field-literal (identifier :checkins :date) :type/Date] :month]])
         qp/process-query
         :status)))
+
+;; make sure timeseries queries generated by "drag-to-filter" work correctly
+(expect
+  :completed
+  (tt/with-temp Card [card (mbql-card-def
+                             :source-table (data/id :checkins))]
+    (-> (query-with-source-card card
+          :aggregation [[:count]]
+          :breakout    [[:datetime-field [:field-literal "DATE" :type/Date] :week]]
+          :filter      [:between
+                        [:datetime-field [:field-literal "DATE" :type/Date] :month]
+                        "2014-02-01T00:00:00-08:00"
+                        "2014-05-01T00:00:00-07:00"])
+        qp/process-query
+        :status)))