CodeMirror/budibase
1
0
Fork 0
mirror of synced 2024-09-20 19:33:10 +12:00
Code Issues Projects Releases Wiki Activity

Oracle fix PR comments.

Browse source
This commit is contained in:
Sam Rose 2024-08-05 16:15:15 +01:00
parent 6c79f3410c
commit 7f49f87f27
No known key found for this signature in database
3 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/backend-core/src/sql/sql.ts
View file

@ -682,6 +682,8 @@ class InternalBuilder {
const direction = const direction =
value.direction === SortOrder.ASCENDING ? "asc" : "desc" value.direction === SortOrder.ASCENDING ? "asc" : "desc"
// TODO: figure out a way to remove this conditional, not relying on
// the defaults of each datastore.
let nulls: "first" | "last" | undefined = undefined let nulls: "first" | "last" | undefined = undefined
if ( if (
this.client === SqlClient.POSTGRES || this.client === SqlClient.POSTGRES ||

3
packages/server/src/api/routes/tests/search.spec.ts
View file

@ -1596,9 +1596,6 @@ describe.each([
// Our bigints are int64s in most datasources. // Our bigints are int64s in most datasources.
let BIG = "9223372036854775807" let BIG = "9223372036854775807"
if (name === DatabaseName.ORACLE) {
// BIG = "9223372036854775808"
}
beforeAll(async () => { beforeAll(async () => {
table = await createTable({ table = await createTable({

7
packages/server/src/integrations/oracle.ts
View file

@ -500,6 +500,13 @@ class OracleIntegration extends Sql implements DatasourcePlus {
password: this.config.password, password: this.config.password,
connectString, connectString,
} }
// We set the timezone of the connection to match the timezone of the
// Budibase server, this is because several column types (e.g. time-only
// timestamps) do not store timezone information, so to avoid storing one
// time and getting a different one back we need to make sure the timezone
// of the server matches the timezone of the database. There's an assumption
// here that the server is running in the same timezone as the database.
const tz = Intl.DateTimeFormat().resolvedOptions().timeZone const tz = Intl.DateTimeFormat().resolvedOptions().timeZone
const connection = await oracledb.getConnection(attributes) const connection = await oracledb.getConnection(attributes)
await connection.execute(`ALTER SESSION SET TIME_ZONE = '${tz}'`) await connection.execute(`ALTER SESSION SET TIME_ZONE = '${tz}'`)