coldbox-modules / qb

Fluent Query Builder for CFML
https://qb.ortusbooks.com/
MIT License
40 stars 35 forks source link

Fix serializeBindings() for array bindings #281

Closed michaelborn closed 3 months ago

michaelborn commented 3 months ago

Resolves query logging on queries that use array bindings, such as the SqlServerGrammar's getAllTableNames() method:

https://github.com/coldbox-modules/qb/blob/main/models/Grammars/SqlServerGrammar.cfc#L544

See this stack trace.

https://pastebin.com/jgWFmdgj

elpete commented 3 months ago

Thanks for the PR! I implemented it in a different way in this commit. It should be out soon as 9.10.3.