Skip to content

wallet: a split tx is marked as a self tx - #2169

Open
profinch wants to merge 1 commit into
BeamMW:masterfrom
profinch:wallet-split-self-tx
Open

profinch wants to merge 1 commit into
BeamMW:masterfrom
profinch:wallet-split-self-tx

Conversation

@profinch

Copy link
Copy Markdown

A split has no receiver, so IsSelfTx was never set for it: its status read "sent" instead of "sent to own address", and clients showed the split amount as spent. SimpleTransaction::IsSelfTx() already treats a tx without a receiver as a self tx; its creator now records that.

Test: wallet_test.cpp checks m_selfTx on the split tx.

Patch authored by @Maxnflaxl, opened on their behalf (account suspended).

🤖 Generated with Claude Code

A split has no receiver, so IsSelfTx was never set for it: its status read
"sent" instead of "sent to own address", and clients showed the split
amount as spent. SimpleTransaction::IsSelfTx() already treats a tx without
a receiver as a self tx; its creator now records that.
@profinch
profinch force-pushed the wallet-split-self-tx branch from a1d2d3e to bbab51d Compare October 24, 2026 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant