Allow where_in and where_not_in to have empty arrays

[Flamenco]

I have to put many checks in my code for filters because where_in('foo',[]) will throw an error.
Please consider allowing these conditions.

[Flamenco]

[kylefarris]

Would you rather it just silently ignore it?

[Flamenco]

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

[kylefarris]

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

[kylefarris]

@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!

[Flamenco]

@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.