From b4d607bfefc8cfeacb4cfd0028667bf0b6e9f3b8 Mon Sep 17 00:00:00 2001 From: Sagiv Ofek Date: Sun, 24 Aug 2014 01:38:31 -0400 Subject: [PATCH] updated the code to 1-liner statements using typeof instead of 2 if checks. --- lib/TransactionBuilder.js | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/lib/TransactionBuilder.js b/lib/TransactionBuilder.js index ce33313..024ad29 100644 --- a/lib/TransactionBuilder.js +++ b/lib/TransactionBuilder.js @@ -109,10 +109,8 @@ function TransactionBuilder(opts) { this.lockTime = opts.lockTime || 0; this.spendUnconfirmed = opts.spendUnconfirmed || false; - if (opts.fee || opts.feeSat) { - this.givenFeeSat = opts.fee ? opts.fee * util.COIN : opts.feeSat; - } - if (opts.fee == 0 || opts.feeSat == 0) this.givenFeeSat = 0; + this.givenFeeSat = typeof(opts.fee) !== 'undefined' ? opts.fee * util.COIN : opts.feeSat; + this.remainderOut = opts.remainderOut; this.signhash = opts.signhash || Transaction.SIGHASH_ALL; @@ -359,8 +357,7 @@ TransactionBuilder.prototype._setFeeAndRemainder = function(txobj) { /* based on https://en.bitcoin.it/wiki/Transaction_fees */ maxSizeK = parseInt(size / 1000) + 1; - var feeSat = this.givenFeeSat || this.givenFeeSat == 0 ? - this.givenFeeSat : maxSizeK * FEE_PER_1000B_SAT; + var feeSat = typeof(this.givenFeeSat) !== 'undefined' ? this.givenFeeSat : maxSizeK * FEE_PER_1000B_SAT; var neededAmountSat = this.valueOutSat.add(feeSat);