Page MenuHomePhabricator

[qt] send: Clear All also resets coin control options
ClosedPublic

Authored by fpelliccioni on Sep 13 2019, 22:28.

Details

Summary

This change makes it so that a custom change address and manual input selection are removed if the user clicks Clear All in the send screen.

Backport of Bitcoin Core PR12432
https://github.com/bitcoin/bitcoin/pull/12432

Test Plan
  1. Start BitcoinABC wallet: ./src/qt/bitcoin-qt -regtest
  2. Enable the check named Enable coin control features, at Settings -> Options -> Wallet.
  3. Go to Send dialog.
  4. Enable the check Custom change address and fill with one address at the right.
  5. Click on Clear all button.
  6. Verify that the checkbox and the Custom change address text are cleaned.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
feature-backport-9fb3898c3
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7454
Build 12951: Bitcoin ABC Buildbot (legacy)
Build 12950: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Sep 14 2019, 07:53

The test plan is not appropriate.

This revision now requires changes to proceed.Sep 14 2019, 07:53
fpelliccioni edited the test plan for this revision. (Show Details)

Test plan updated.

This revision is now accepted and ready to land.Sep 19 2019, 11:57