From a92b23b97d7b787783189f5837a7cae27ccc9fc1 Mon Sep 17 00:00:00 2001 From: Vitaly Tomilov Date: Sat, 9 Jul 2016 06:51:22 +0100 Subject: [PATCH] Update PostgresStorageAdapter.js (#2093) 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 3c0373af..e305fffe 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; - } }); }