crysalead-js / sql-dialect

SQL Builder
MIT License
6 stars 2 forks source link

[breaking] dialect: re-map (not) exists operators to subquery builder #12

Closed esatterwhite closed 4 years ago

esatterwhite commented 4 years ago

The list builder was generating incorrect sql for the exits and not exists operators. This adds a sub query build for these two cases to render the sql accurately.

These two operators now use the subquery builder rather than the list operator

Semver: major Ref: #11

jails commented 4 years ago

Fantastic :100: !! By the way why using the array syntax ':exists': [subquery] instead of simply ':exists': subquery ? not sure exists can handle multiple queries.

esatterwhite commented 4 years ago

Ya, thats fair. will remove

jails commented 4 years ago

Perfect ! Thank you :+1: