ujjwalguptaofficial / JsStore

Simplifying IndexedDB with SQL like syntax and promises
http://jsstore.net/
MIT License
858 stars 110 forks source link

between not inclusive #202

Closed Madman600 closed 3 years ago

Madman600 commented 3 years ago

The between operator is not inclusive as it should be.

ujjwalguptaofficial commented 3 years ago

@Madman600 could you add some description ? an example perhaps.

Madman600 commented 3 years ago

If I select using between 1 and 10 , only 2-9 will be returned. It should return 1-10.

ujjwalguptaofficial commented 3 years ago

This is with multiple column query in where right ? If there is no issue, could you paste your query please.

ujjwalguptaofficial commented 3 years ago

There was issue with filtering when where has multiple column. e.g -

select({
   from:'Products',
   where:{
      id: 5,
      price:{
        '>':10
      }
   }
})

This has been fixed in new version - 3.13.6

Madman600 commented 3 years ago

Between is still an issue it's only returning < high, not <= high.

ujjwalguptaofficial commented 3 years ago

@Madman600 not able to reproduce, if you can provide a query or a POC then i will be able to look.