Page MenuHomePhabricator

Merge #13498: [wallet] Fixups from account API deprecation

Authored by jasonbcox on Nov 23 2019, 17:28.



df10f07db1 [wallet] Don't use accounts when checking balance in sendmany (John Newbery)
e209184101 [wallet] deprecate sendfrom RPC method. (John Newbery)

Pull request description:

A couple of fixups from the accounts API deprecation PR (#12953):

- properly deprecate `sendfrom`
- don't use accounts when calculating balance in `sendmany` (unless the `-deprecatedrpc=accounts` flag is being used)

Tree-SHA512: 1befde055067438c4c3391bbff1aaed0e6249efd708c567db3f1faad40a0f28e64f95e5bad0679ae826d24a0239e4bc8a1c392dc93e2e7502343a7f6b1d1845c

Backport of Core PR13498

Depends on D4519

Test Plan
ninja check

Diff Detail

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

Event Timeline

jasonbcox created this revision.Nov 23 2019, 17:28
jasonbcox planned changes to this revision.Nov 23 2019, 17:28
jasonbcox requested review of this revision.Nov 23 2019, 17:42
jasonbcox added inline comments.
119 ↗(On Diff #14355)

The missing change from the PR is due to D1799

Fabien accepted this revision.Nov 24 2019, 09:51
This revision is now accepted and ready to land.Nov 24 2019, 09:51