Page MenuHomePhabricator

[backport#16952] gui: make sure to update the UI when deleting a transaction
ClosedPublic

Authored by majcosta on Mon, Jul 27, 21:04.

Details

Summary

make sure to update the UI when deleting a transaction (Jonas Schnelli)

Pull request description:

`CWallet::ZapSelectTx` removes transactions from the internal model, but leaves the UI in the dark.
Adding a `NotifyTransactionChanged()` should avoid having invalid transactions in the GUI.

Fixes #16950

https://github.com/bitcoin/bitcoin/pull/16952/commits/addaf8af8268d918973883a304025d40af5a33c1


Backport of Core PR16952

Test Plan
ninja check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

majcosta created this revision.Mon, Jul 27, 21:04
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Jul 27, 21:04
majcosta requested review of this revision.Mon, Jul 27, 21:04
Fabien accepted this revision.Mon, Jul 27, 21:43
This revision is now accepted and ready to land.Mon, Jul 27, 21:43
teamcity edited the summary of this revision. (Show Details)Mon, Jul 27, 22:07

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.