Remove "conflicted" as transaction category in listtransactions #3720

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2014_02_conflicted_non_category changing 1 files +2 −8

laanwj commented at 3:34 PM on February 21, 2014: member

We were losing information about sent/received by overriding the category in case of a conflicted transaction.

Hence, remove the "conflicted" category.

Conflicted status of a transaction can still be determined by looking for confirmations<0.

Remove "conflicted" as transaction category.

We were losing information about sent/received by overriding the
category in case of a conflicted transaction.

Hence, remove the "conflicted" category.

Conflicted status of a transaction can still be determined by looking
for confirmations<0.

b96f6a77f8

gavinandresen commented at 3:41 PM on February 21, 2014: contributor

ACK
luke-jr commented at 3:41 PM on February 21, 2014: member

ACK
BitcoinPullTester commented at 3:51 PM on February 21, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/b96f6a77f8a33fe39f480d4b1c617b7ba57d61b8 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 referenced this in commit 39fae6cfdd on Feb 22, 2014
laanwj merged this on Feb 22, 2014
laanwj closed this on Feb 22, 2014
laanwj deleted the branch on Apr 9, 2014
DrahtBot locked this on Sep 8, 2021

Contributors