diff options
| author | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-08-21 08:59:35 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-08-21 09:44:26 +0200 |
| commit | 8aa9badf5ea3ea98980f50d924e88e46c4d5ee38 (patch) | |
| tree | b96ab86c1c66db8b4b49d92a510e1f353b040f2f /src/qt/bitcoin.cpp | |
| parent | 4732fa133abdf374790825abbf1bbf56819cc806 (diff) | |
| parent | faaac5caaab4d5131040292f4ef2404074ad268b (diff) | |
Merge #13968: [wallet] couple of walletcreatefundedpsbt fixes
faaac5caaab4d5131040292f4ef2404074ad268b RPCTypeCheck bip32derivs arg in walletcreatefunded (Gregory Sanders)
1f0c4282e961baea85d5f74d7493bd7459784391 QA: add basic walletcreatefunded optional arg test (Gregory Sanders)
1f18d7b591ffcc8bb9422a9b728bd9a0d8da6a2a walletcreatefundedpsbt: remove duplicate replaceable arg (Gregory Sanders)
2252ec50085c151e7998ca9a30cda6a33ee862b6 Allow ConstructTransaction to not throw error with 0-input txn (Gregory Sanders)
Pull request description:
1) Previously an empty input argument transaction that is marked for replaceability fails to pass the `SignalsOptInRBF` check right before funding it. Explicitly check for that condition before throwing an error.
2) The rpc call had two separate `replaceable` arguments, each of which being used in mutually exclusive places. I preserved the `options` version to retain compatability with `fundtransaction`.
Tree-SHA512: 26eb0c9e2d38ea51d11f741d61100223253271a084adadeb7e78c6d4e9004636f089e4273c5bf64a41bd7e9ff795317acf30531cb36aeb0d8db9304b3c8270c3
Diffstat (limited to 'src/qt/bitcoin.cpp')
0 files changed, 0 insertions, 0 deletions