Question/Hotfix - where query "OR" about nano-sql HOT 3 CLOSED

Hello Te,

Thanks for looking into this!

The old query code was unnecessarily complex, the change made .where conditions much faster to check. I guess I forgot to clear out all the old code before I pushed the update. 😺

The current code/version passes the test in your fork already. CodePen

Can you you share with me a condition the current code fails under that your hotfix takes care of?

Best,
Scott

Edit: Ah, my apologies. Looks like I had pulled an older version into the CodePen, the newest version does fail on the test you provided.

There's another condition the current version fails on as well as your forked version, looks like I need to revisit the .where code and get it working for this test case also. CodePen

from nano-sql.

Just pushed version 1.4.4, reworked the .where() code and added a bunch of tests, let me know if the update didn't resolve everything.

from nano-sql.

Marking this as resolved since there hasn't been a response in a few days. Feel free to reopen if the issue comes up again or isn't resolved.

from nano-sql.