MirrorSave
/
budibase
mirror of https://github.com/Budibase/budibase.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Update date filtering to only use a between statement when 2 dates are provided

This commit is contained in:
Andrew Kingston 2022-03-10 10:18:03 +00:00
parent 448f43fb0d
commit 7a1b362515
1 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
packages/server/src/integrations/base/sql.ts
View File

@ -27,11 +27,8 @@ function parse(input: any) {
if (typeof input !== "string") { if (typeof input !== "string") {
return input return input
} }
if (input === MAX_ISO_DATE) { if (input === MAX_ISO_DATE || input === MIN_ISO_DATE) {
return new Date(8640000000000000) return null
}
if (input === MIN_ISO_DATE) {
return new Date(-8640000000000000)
} }
if (isIsoDateString(input)) { if (isIsoDateString(input)) {
return new Date(input) return new Date(input)
@ -130,11 +127,19 @@ class InternalBuilder {
} }
if (filters.range) { if (filters.range) {
iterate(filters.range, (key, value) => { iterate(filters.range, (key, value) => {
if (!value.high || !value.low) { if (value.low && value.high) {
return // Use a between operator if we have 2 valid range values
}
const fnc = allOr ? "orWhereBetween" : "whereBetween" const fnc = allOr ? "orWhereBetween" : "whereBetween"
query = query[fnc](key, [value.low, value.high]) query = query[fnc](key, [value.low, value.high])
} else if (value.low) {
// Use just a single greater than operator if we only have a low
const fnc = allOr ? "orWhere" : "where"
query = query[fnc](key, ">", value.low)
} else if (value.high) {
// Use just a single less than operator if we only have a high
const fnc = allOr ? "orWhere" : "where"
query = query[fnc](key, "<", value.high)
}
}) })
} }
if (filters.equal) { if (filters.equal) {