anders/Wallet
1
0
Fork 0
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #1298 from isocolsky/fix/update_balance

Browse source 
Fixed reentrant event updating balance multiple times
This commit is contained in:
Matias Alejo Garcia 2014-09-05 15:07:25 -03:00
commit 1aa46b6ad0
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
js/models/core/Wallet.js
View file

@ -1558,6 +1558,8 @@ Wallet.prototype.removeTxWithSpentInputs = function(cb) {
if (inputs.length === 0) if (inputs.length === 0)
return; return;
var proposalsChanged = false;
this.blockchain.getUnspent(this.getAddressesStr(), function(err, unspentList) { this.blockchain.getUnspent(this.getAddressesStr(), function(err, unspentList) {
if (err) return cb(err); if (err) return cb(err);
@ -1569,12 +1571,15 @@ Wallet.prototype.removeTxWithSpentInputs = function(cb) {
inputs.forEach(function (input) { inputs.forEach(function (input) {
if (!input.unspent) { if (!input.unspent) {
proposalsChanged = true;
self.txProposals.deleteOne(input.ntxid); self.txProposals.deleteOne(input.ntxid);
} }
}); });
self.emit('txProposalsUpdated'); if (proposalsChanged) {
self.store(); self.emit('txProposalsUpdated');
self.store();
}
cb(null); cb(null);
}); });