Merge pull request #775 from morintd/fix/lt-lte-null

fix: 🐛 ensure we don't accept null from lt/lte filters
morintd · Jan 31, 2024 · d80de8e · d80de8e
2 parents ff329f1 + 34add0c
commit d80de8e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/lib/operations/find/match.ts b/src/lib/operations/find/match.ts
@@ -131,10 +131,10 @@ export const matchMultiple = (item: Item, where: FindWhereArgs, current: Delegat
           match = val >= formatValueWithMode(filter.gte, filter, info)!;
         }
         if ('lt' in filter && match) {
-          match = val < formatValueWithMode(filter.lt, filter, info)!;
+          match = val !== null && val < formatValueWithMode(filter.lt, filter, info)!;
         }
         if ('lte' in filter && match) {
-          match = val <= formatValueWithMode(filter.lte, filter, info)!;
+          match = val !== null && val <= formatValueWithMode(filter.lte, filter, info)!;
         }
         if ('in' in filter && match) {
           match = (filter.in as any[]).map((inEntry) => formatValueWithMode(inEntry, filter, info)).includes(val);