fix(knex): Only apply default order for MSSQL (#3145)

feathersjs · Apr 5, 2023 · 28c2627 · 28c2627
1 parent 3eb7428
commit 28c2627
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/knex/src/adapter.ts b/packages/knex/src/adapter.ts
@@ -181,7 +181,7 @@ export class KnexAdapter<
     }
 
     // provide default sorting if its not set
-    if (!filters.$sort) {
+    if (!filters.$sort && builder.client.driverName === 'mssql') {
       builder.orderBy(`${name}.${id}`, 'asc')
     }
 
@@ -238,7 +238,7 @@ export class KnexAdapter<
       return Promise.all(data.map((current) => this._create(current, params)))
     }
 
-    const client = this.db(params).client.config.client
+    const { client } = this.db(params).client.config
     const returning = RETURNING_CLIENTS.includes(client as string) ? [this.id] : []
     const rows: any = await this.db(params).insert(data, returning).catch(errorHandler)
     const id = data[this.id] || rows[0][this.id] || rows[0]