chore(PostgresStorageAdapter): nits (#3870)
refactoring method + removing a comment that's no longer needed.
This commit is contained in:
committed by
Florent Vilmart
parent
1f11ad5d74
commit
14e2d26d12
@@ -778,7 +778,7 @@ export class PostgresStorageAdapter {
|
|||||||
const qs = `INSERT INTO $1:name (${columnsPattern}) VALUES (${valuesPattern})`
|
const qs = `INSERT INTO $1:name (${columnsPattern}) VALUES (${valuesPattern})`
|
||||||
const values = [className, ...columnsArray, ...valuesArray]
|
const values = [className, ...columnsArray, ...valuesArray]
|
||||||
debug(qs, values);
|
debug(qs, values);
|
||||||
return this._client.any(qs, values)
|
return this._client.none(qs, values)
|
||||||
.then(() => ({ ops: [object] }))
|
.then(() => ({ ops: [object] }))
|
||||||
.catch(error => {
|
.catch(error => {
|
||||||
if (error.code === PostgresUniqueIndexViolationError) {
|
if (error.code === PostgresUniqueIndexViolationError) {
|
||||||
@@ -993,7 +993,7 @@ export class PostgresStorageAdapter {
|
|||||||
|
|
||||||
const qs = `UPDATE $1:name SET ${updatePatterns.join(',')} WHERE ${where.pattern} RETURNING *`;
|
const qs = `UPDATE $1:name SET ${updatePatterns.join(',')} WHERE ${where.pattern} RETURNING *`;
|
||||||
debug('update: ', qs, values);
|
debug('update: ', qs, values);
|
||||||
return this._client.any(qs, values); // TODO: This is unsafe, verification is needed, or a different query method;
|
return this._client.any(qs, values);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Hopefully, we can get rid of this. It's only used for config and hooks.
|
// Hopefully, we can get rid of this. It's only used for config and hooks.
|
||||||
|
|||||||
Reference in New Issue
Block a user