diff options
author | Alexander Tseung <alextsg@users.noreply.github.com> | 2018-01-04 03:35:14 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-01-04 03:35:14 +0800 |
commit | dc9d8b24c6fccd584975c5b9f25c8090b7e8051d (patch) | |
tree | 732806322abf5602a17533caf1d6a875a97e10a6 /app/scripts/lib | |
parent | dfb22471087f040d8345a5a17321e1462842045c (diff) | |
parent | 099f078a3d73b2aed30dc5e1cd3a2facde58606a (diff) | |
download | tangerine-wallet-browser-dc9d8b24c6fccd584975c5b9f25c8090b7e8051d.tar.gz tangerine-wallet-browser-dc9d8b24c6fccd584975c5b9f25c8090b7e8051d.tar.zst tangerine-wallet-browser-dc9d8b24c6fccd584975c5b9f25c8090b7e8051d.zip |
Merge pull request #2845 from alextsg/NewUI-flat-4.0.5c
[NewUI] Merge master into NewUI-flat
Diffstat (limited to 'app/scripts/lib')
-rw-r--r-- | app/scripts/lib/pending-tx-tracker.js | 3 | ||||
-rw-r--r-- | app/scripts/lib/tx-gas-utils.js | 13 |
2 files changed, 12 insertions, 4 deletions
diff --git a/app/scripts/lib/pending-tx-tracker.js b/app/scripts/lib/pending-tx-tracker.js index 7956a3329..e8869e6b8 100644 --- a/app/scripts/lib/pending-tx-tracker.js +++ b/app/scripts/lib/pending-tx-tracker.js @@ -178,7 +178,8 @@ module.exports = class PendingTransactionTracker extends EventEmitter { } async _checkIfNonceIsTaken (txMeta) { - const completed = this.getCompletedTransactions() + const address = txMeta.txParams.from + const completed = this.getCompletedTransactions(address) const sameNonce = completed.filter((otherMeta) => { return otherMeta.txParams.nonce === txMeta.txParams.nonce }) diff --git a/app/scripts/lib/tx-gas-utils.js b/app/scripts/lib/tx-gas-utils.js index 56bee19f7..ccf8bb1b1 100644 --- a/app/scripts/lib/tx-gas-utils.js +++ b/app/scripts/lib/tx-gas-utils.js @@ -26,7 +26,7 @@ module.exports = class txProvideUtil { err.message.includes('Transaction execution error.') || err.message.includes('gas required exceeds allowance or always failing transaction') ) - if ( simulationFailed ) { + if (simulationFailed) { txMeta.simulationFails = true return txMeta } @@ -81,8 +81,15 @@ module.exports = class txProvideUtil { } async validateTxParams (txParams) { - if (('value' in txParams) && txParams.value.indexOf('-') === 0) { - throw new Error(`Invalid transaction value of ${txParams.value} not a positive number.`) + if ('value' in txParams) { + const value = txParams.value.toString() + if (value.includes('-')) { + throw new Error(`Invalid transaction value of ${txParams.value} not a positive number.`) + } + + if (value.includes('.')) { + throw new Error(`Invalid transaction value of ${txParams.value} number must be in wei`) + } } } } |