search

ruckus / quickbooks-ruby

Quickbooks Online REST API V3 - Ruby
MIT License
374 stars 302 forks source link

ensure we `#to_s` any non-date, non-array query values #521

Closed jethrodaniel closed 4 years ago

jethrodaniel commented 4 years ago

Before this was applied, the following code would blow up with a NoMethodError.

util = Quickbooks::Util::QueryBuilder.new
clause = util.clause('Id', '=', 4)

fixes https://github.com/ruckus/quickbooks-ruby/issues/520

ruckus commented 4 years ago

Awesome, your PR is way better. Thanks!

jethrodaniel commented 4 years ago

Thanks @ruckus, never had a maintainer respond so quickly. 👍