Page MenuHomePhabricator

Show addresses for "SendToSelf" transactions
ClosedPublic

Authored by PiRK on Oct 26 2020, 10:53.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABC16f28961a94f: Show addresses for "SendToSelf" transactions
Summary

With this change, the transactions tab of bitcoin-qt now shows the address for "Payment to yourself" transactions.

This is a backport of Core PR15098

Test Plan

ninja && src/qt/bitcoin-qt
Create a receive address on the Receive tab and send some BCH to it on the Send tab.
Verify that the Transactions tab now show the involved addresses.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a reviewer: Restricted Owners Package.Oct 26 2020, 10:53
PiRK requested review of this revision.Oct 26 2020, 10:53
deadalnix requested changes to this revision.Oct 26 2020, 13:08
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/qt/transactionrecord.cpp
102 ↗(On Diff #25106)

Why not encode destination? This will break for user who chose to not use cashaddr.

This revision now requires changes to proceed.Oct 26 2020, 13:08
PiRK requested review of this revision.Oct 26 2020, 20:43
PiRK added inline comments.
src/qt/transactionrecord.cpp
102 ↗(On Diff #25106)

CashAddr is already used everywhere in bitcoin-qt.

$ grep -r EncodeDestination src/qt | wc -l
0
$ grep -r EncodeCashAddr src/qt | wc -l
14
This revision is now accepted and ready to land.Oct 26 2020, 22:59