transactions - lint fixes

feature/default_network_editable
frankiebee 7 years ago
parent 2d7c3c2b00
commit 5494aa4f9c
  1. 10
      app/scripts/controllers/transactions/lib/util.js
  2. 2
      app/scripts/controllers/transactions/tx-gas-utils.js
  3. 12
      app/scripts/controllers/transactions/tx-state-manager.js

@ -7,7 +7,7 @@ module.exports = {
normalizeTxParams, normalizeTxParams,
validateTxParams, validateTxParams,
validateFrom, validateFrom,
validateRecipient validateRecipient,
} }
@ -48,19 +48,19 @@ function validateTxParams (txParams) {
} }
function validateFrom (txParams) { function validateFrom (txParams) {
if ( !(typeof txParams.from === 'string') ) throw new Error(`Invalid from address ${txParams.from} not a string`) if (!(typeof txParams.from === 'string')) throw new Error(`Invalid from address ${txParams.from} not a string`)
if (!isValidAddress(txParams.from)) throw new Error('Invalid from address') if (!isValidAddress(txParams.from)) throw new Error('Invalid from address')
} }
function validateRecipient (txParams) { function validateRecipient (txParams) {
if (txParams.to === '0x' || txParams.to === null ) { if (txParams.to === '0x' || txParams.to === null) {
if (txParams.data) { if (txParams.data) {
delete txParams.to delete txParams.to
} else { } else {
throw new Error('Invalid recipient address') throw new Error('Invalid recipient address')
} }
} else if ( txParams.to !== undefined && !isValidAddress(txParams.to) ) { } else if (txParams.to !== undefined && !isValidAddress(txParams.to)) {
throw new Error('Invalid recipient address') throw new Error('Invalid recipient address')
} }
return txParams return txParams
} }

@ -100,4 +100,4 @@ module.exports = class TxGasUtil {
// otherwise use blockGasLimit // otherwise use blockGasLimit
return bnToHex(upperGasLimitBn) return bnToHex(upperGasLimitBn)
} }
} }

@ -92,7 +92,7 @@ module.exports = class TransactionStateManager extends EventEmitter {
// or rejected tx's. // or rejected tx's.
// not tx's that are pending or unapproved // not tx's that are pending or unapproved
if (txCount > txHistoryLimit - 1) { if (txCount > txHistoryLimit - 1) {
let index = transactions.findIndex((metaTx) => { const index = transactions.findIndex((metaTx) => {
return this.getFinalStates().includes(metaTx.status) return this.getFinalStates().includes(metaTx.status)
}) })
if (index !== -1) { if (index !== -1) {
@ -145,7 +145,7 @@ module.exports = class TransactionStateManager extends EventEmitter {
} }
// validates txParams members by type // validates txParams members by type
validateTxParams(txParams) { validateTxParams (txParams) {
Object.keys(txParams).forEach((key) => { Object.keys(txParams).forEach((key) => {
const value = txParams[key] const value = txParams[key]
// validate types // validate types
@ -263,10 +263,10 @@ module.exports = class TransactionStateManager extends EventEmitter {
// returns an array of states that can be considered final // returns an array of states that can be considered final
getFinalStates () { getFinalStates () {
return [ return [
'rejected', // the user has responded no! 'rejected', // the user has responded no!
'confirmed', // the tx has been included in a block. 'confirmed', // the tx has been included in a block.
'failed', // the tx failed for some reason, included on tx data. 'failed', // the tx failed for some reason, included on tx data.
'dropped', // the tx nonce was already used 'dropped', // the tx nonce was already used
] ]
} }

Loading…
Cancel
Save