[Qt] Fix Transaction details shows wrong To: #4238

pull cozz wants to merge 1 commits into bitcoin:master from cozz:cozz11 changing 3 files +16 −22

cozz commented at 12:06 AM on May 28, 2014: contributor

Fixes #1725.

When we decompose a transaction into a TransactionRecord we already loop the outputs and assign rec->address. In TransactionDesc we can simply show the value of rec->address. What we currently do is loop the outputs again and always show the first of our outputs which results in the described bug.
[Qt] Fix Transaction details shows wrong To: b90711cabf
BitcoinPullTester commented at 12:45 AM on May 28, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/b90711cabffa8bf83ddee98cfc179c0aef5732ef for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj commented at 7:44 AM on May 28, 2014: member

Untested ACK
laanwj merged this on Jun 2, 2014
laanwj closed this on Jun 2, 2014
laanwj referenced this in commit be7d0adcdb on Jun 2, 2014
DrahtBot locked this on Sep 8, 2021

Contributors