From bd98bf42c3d6b000007ed185972b3043b8af278a Mon Sep 17 00:00:00 2001 From: Vitaly Tomilov Date: Sat, 18 Jun 2016 08:04:33 +0100 Subject: [PATCH] Update PostgresStorageAdapter.js If it really needs to reject with `undefined`, then this is the right way. Usually one just returns the result, let the caller provide the `.catch` ;) This way real errors can be swallowed, not very good ;) --- src/Adapters/Storage/Postgres/PostgresStorageAdapter.js | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/Adapters/Storage/Postgres/PostgresStorageAdapter.js b/src/Adapters/Storage/Postgres/PostgresStorageAdapter.js index f449e20f68..26ce906d00 100644 --- a/src/Adapters/Storage/Postgres/PostgresStorageAdapter.js +++ b/src/Adapters/Storage/Postgres/PostgresStorageAdapter.js @@ -220,13 +220,9 @@ export class PostgresStorageAdapter { // this adapter doesn't know about the schema, return a promise that rejects with // undefined as the reason. getClass(className) { - return this._client.query('SELECT * FROM "_SCHEMA" WHERE "className"=$', { className }) - .then(result => { - if (result.length === 1) { - return result[0].schema; - } else { + return this._client.one('SELECT * FROM "_SCHEMA" WHERE "className"=$', { className }, res=>res.schema) + .catch(error => { throw undefined; - } }); }