Args parsing is wrong with integer "queryable" and KV args

camsaul / toucan2

Successor library to Toucan with a modern and more-extensible API, more consistent behavior, and support for different backends including non-JDBC databases and non-HoneySQL queries. Currently in active beta.

Eclipse Public License 1.0

81 stars 11 forks source link

Args parsing is wrong with integer "queryable" and KV args #160

Closed camsaul closed 12 months ago

camsaul commented 12 months ago

(t2/debug
  (t2/select-one :metadata/segment 1 :table.db_id 2))

;; =>
;; Parsed args: {:kv-args {1 :table.db_id}}
;;
;; Resolved query: 2

This should have been

Parsed args: {:kv-args {:table.db_id 2}}

Resolved query: 1

camsaul commented 12 months ago

I guess this is actually intentional