From 6a46e9ce0662413f6351756658b2bae8a895a33b Mon Sep 17 00:00:00 2001
From: Dan Finlay <dan@danfinlay.com>
Date: Thu, 23 Mar 2017 17:00:59 -0700
Subject: Make gas calculations on render more consistent

---
 ui/app/components/pending-tx.js | 26 ++++++++++++++++----------
 1 file changed, 16 insertions(+), 10 deletions(-)

(limited to 'ui/app')

diff --git a/ui/app/components/pending-tx.js b/ui/app/components/pending-tx.js
index 9a3ef272d..2f9e9885b 100644
--- a/ui/app/components/pending-tx.js
+++ b/ui/app/components/pending-tx.js
@@ -21,40 +21,46 @@ const MIN_GAS_PRICE_BN = new BN(20000000)
 module.exports = connect(mapStateToProps)(PendingTx)
 
 function mapStateToProps (state) {
-  return {
-
-  }
+  return {}
 }
 
 inherits(PendingTx, Component)
 function PendingTx () {
   Component.call(this)
-  this.state = { valid: true }
+  this.state = {
+    valid: true,
+    gas: null,
+    gasPrice: null,
+    txData: null,
+  }
 }
 
 PendingTx.prototype.render = function () {
   const props = this.props
-  const txData = props.txData
-
   const state = this.state
 
+  const txData = state.txData || props.txData
   const txParams = txData.txParams || {}
+
   const address = txParams.from || props.selectedAddress
   const identity = props.identities[address] || { address: address }
   const account = props.accounts[address]
   const balance = account ? account.balance : '0x0'
 
-
-  const gas = (state.gas === undefined) ? txParams.gas : state.gas
-  const gasPrice = (state.gasPrice === undefined) ? txData.gasPrice : state.gasPrice
+  const gas = state.gas || txParams.gas
+  const gasPrice = state.gasPrice || txData.gasPrice
   const gasBn = new BN(gas, 16)
   const gasPriceBn = new BN(gasPrice, 16)
 
   const txFeeBn = gasBn.mul(gasPriceBn)
-  const maxCost = state.maxCost || txData.maxCost || ''
+  const valueBn = new BN(ethUtil.stripHexPrefix(txParams.value), 16)
+  const maxCost = txFeeBn.add(valueBn)
+
   const dataLength = txParams.data ? (txParams.data.length - 2) / 2 : 0
   const imageify = props.imageifyIdenticons === undefined ? true : props.imageifyIdenticons
 
+  log.info(`rendering pending-tx form with gas limit ${gas.toString()}, gasPrice ${gasPrice.toString()}, `)
+
   this.inputs = []
 
   return (
-- 
cgit