Skip to content

Commit fd38e79

Browse files
authored
fix(oracle): For Raw queries avoid converting the input parameters passed (#16067)
1 parent eb71077 commit fd38e79

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

Diff for: src/dialects/oracle/query.js

+2
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,8 @@ export class OracleQuery extends AbstractQuery {
121121
bindDef.push(...Object.values(this.options.inbindAttributes));
122122
bindDef.push(...outParameters);
123123
this.bindParameters = parameters;
124+
} else if (this.isRawQuery()) {
125+
this.bindParameters = parameters;
124126
} else {
125127
Object.values(parameters).forEach(value => {
126128
bindParameters.push(value);

Diff for: test/integration/sequelize/query.test.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -713,8 +713,8 @@ describe(Support.getTestDialectTeaser('Sequelize'), () => {
713713
it('binds named parameters with the passed object using the same key twice', async function() {
714714
const typeCast = dialect === 'postgres' ? '::int' : '';
715715
let logSql;
716-
const result = await this.sequelize.query(`select $one${typeCast} as foo, $two${typeCast} as bar, $one${typeCast} as baz${Support.addDualInSelect()}`, { raw: true, bind: { one: 1, two: 2 }, logging(s) { logSql = s; } });
717-
const expected = dialect !== 'oracle' ? [{ foo: 1, bar: 2, baz: 1 }] : [{ FOO: 1, BAR: 2, BAZ: 1 }];
716+
const result = await this.sequelize.query(`select $one${typeCast} as foo, $one${typeCast} as bar, $two${typeCast} as baz${Support.addDualInSelect()}`, { raw: true, bind: { one: 1, two: 2 }, logging(s) { logSql = s; } });
717+
const expected = dialect !== 'oracle' ? [{ foo: 1, bar: 1, baz: 2 }] : [{ FOO: 1, BAR: 1, BAZ: 2 }];
718718
expect(result[0]).to.deep.equal(expected);
719719
if (dialect === 'postgres') {
720720
expect(logSql).to.include('$1');

0 commit comments

Comments
 (0)