remove nulls for raw and rely on defaults

This commit is contained in:
Martin McKeaveney 2025-01-16 14:29:49 +00:00
parent c077f7e84c
commit 3bd56bdca7
1 changed files with 3 additions and 6 deletions

View File

@ -1167,17 +1167,15 @@ class InternalBuilder {
let nulls: "first" | "last" | undefined = undefined
if (
this.client === SqlClient.POSTGRES ||
this.client === SqlClient.ORACLE ||
this.client === SqlClient.SQL_LITE
this.client === SqlClient.ORACLE
) {
nulls = value.direction === SortOrder.ASCENDING ? "first" : "last"
}
if (this.isAggregateField(key)) {
query = query.orderByRaw(`?? ?? nulls ??`, [
query = query.orderByRaw(`?? ??`, [
this.rawQuotedIdentifier(key),
this.knex.raw(direction),
this.knex.raw(nulls as string),
])
} else {
let composite = `${aliased}.${key}`
@ -1188,10 +1186,9 @@ class InternalBuilder {
this.knex.raw(nulls as string),
])
} else {
query = query.orderByRaw(`?? ?? nulls ??`, [
query = query.orderByRaw(`?? ??`, [
this.rawQuotedIdentifier(composite),
this.knex.raw(direction),
this.knex.raw(nulls as string),
])
}
}