Add unconfirmedbalance field to getwalletinfo #5522

pull esotericnonsense wants to merge 1 commits into bitcoin:master from esotericnonsense:getwalletinfo changing 1 files +3 −1

esotericnonsense commented at 5:47 AM on December 21, 2014: contributor

The default behaviour of RPC 'getbalance' is to exclude unconfirmed wallet transactions in the balance calculation.

There exists an additional RPC command 'getunconfirmedbalance' which returns only the sum of incoming unconfirmed transactions.

This patch adds an 'unconfirmedbalance' field to getwalletinfo, reducing the need for additional RPC queries. The functionality is equivalent to 'getunconfirmedbalance'.

I have tested this patch on testnet and it works as expected.
Add unconfirmedbalance field to getwalletinfo 6247c42aa3
fanquake commented at 12:53 PM on December 27, 2014: member

@azeteki Do you want to close you pull now that gmaxwell has added your commit to #5540?
jonasschnelli commented at 9:19 AM on December 29, 2014: contributor

accidentally superseded by #5540.
jgarzik added the label RPC on Dec 31, 2014
sipa commented at 4:07 PM on January 4, 2015: member

Merged through #5540.
sipa closed this on Jan 4, 2015
esotericnonsense commented at 3:23 PM on January 29, 2015: contributor

Apologies for the delay in replying/failure to close!

I seem to remember chatting to gmaxwell about this on IRC so it may be that we just duplicated work. A ten second change anyway. :)
DrahtBot locked this on Sep 8, 2021

Contributors

Labels