lamassu/lamassu-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: remove duplicated error handling

Browse source
This commit is contained in:
José Oliveira 2021-06-11 12:00:16 +01:00 • committed by Josh Harvey
parent 8567b7887c
commit 700244e45b
1 changed files with 4 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

7
lib/plugins.js
View file

@ -521,9 +521,11 @@ function plugins (settings, deviceId) {
execute(settings, tradeEntry, newEntry.id) execute(settings, tradeEntry, newEntry.id)
.catch(err => { .catch(err => {
updateTradeEntry(tradeEntry, newEntry, err) updateTradeEntry(tradeEntry, newEntry, err)
.catch(console.log) .then(() => {
console.log(err)
throw err throw err
}) })
})
) )
} }
@ -531,9 +533,6 @@ function plugins (settings, deviceId) {
const data = mergeTradeEntryAndError(tradeEntry, err) const data = mergeTradeEntryAndError(tradeEntry, err)
const sql = pgp.helpers.update(data, ['error'], 'trades') + ` WHERE id = ${newEntry.id}` const sql = pgp.helpers.update(data, ['error'], 'trades') + ` WHERE id = ${newEntry.id}`
return db.none(sql) return db.none(sql)
.then(() => {
throw err
})
} }
function recordTradeAndTx (tradeId, { cashInTxs, cashOutTxs }, dbTx) { function recordTradeAndTx (tradeId, { cashInTxs, cashOutTxs }, dbTx) {