Allow where_in and where_not_in to have empty arrays about node-querybuilder HOT 6 CLOSED

This is officially fixed in the v2 branch. Beta should be out soon.

from node-querybuilder.

from node-querybuilder.

Would you rather it just silently ignore it?

from node-querybuilder.

Yes. Ignore will be great. No need to encode in SQL.

from node-querybuilder.

@Flamenco Thank you for being so active in reporting issues. A v2 Beta is officially out on NPM. If you wouldn't mind testing it, that would be very much appreciated.

You can install the beta like so:

npm i node-querybuilder@next

If you need to submit issues for it, please prefix the issue with v2Beta:

Thanks again!

from node-querybuilder.

@kylefarris I am not sure how you implemented this, but maybe instead of ignoring, replace where_in('foo', []) with false or (1=0) and where_not_in('foo', []) with true or (1=1). I am about to switch over to v2 and I have some complex queries that may have issues.

from node-querybuilder.