v30.3-roots.1 · Commit 26: [PATCH 26/68] feat(qt): enhance privacy-aware coin control

src/qt/sendcoinsdialog.cpp

Before: src/qt/sendcoinsdialog.cpp · After: src/qt/sendcoinsdialog.cpp

File metadata

index 8fea0af8e27d7852f6e5df6d02532a928bf9339f..e8308101af7d49e1a4a75c516376af080260545c 100644

Download original patch · Release notes

Diff rows 1–93 of 93. Line numbers refer to the original before/after files.

+ Added− RemovedUnmarked lines are context.

Scroll the code area horizontally to read long lines. The two line-number columns refer to before and after.

src/qt/sendcoinsdialog.cpp CHANGED Viewed

@@ -164,6 +164,8 @@ void SendCoinsDialog::setModel(WalletModel *_model)
         }
         connect(_model, &WalletModel::balanceChanged, this, &SendCoinsDialog::setBalance);
+        connect(_model, &WalletModel::balanceChanged, this, &SendCoinsDialog::walletChanged);
+        connect(_model, &WalletModel::unload, this, &SendCoinsDialog::reject);
         connect(_model->getOptionsModel(), &OptionsModel::displayUnitChanged, this, &SendCoinsDialog::refreshBalance);
         refreshBalance();
@@ -370,6 +372,8 @@ bool SendCoinsDialog::PrepareSendText(QString& question_string, QString& informa
     }
+    appendCoinControlSummary(question_string);
     // Show the signaling in the prepared transaction, rather than the
     // requested value, so the confirmation states the effective result even
     // when no fee is required.
@@ -484,6 +488,45 @@ bool SendCoinsDialog::signWithExternalSigner(PartiallySignedTransaction& psbtx,
     return true;
 }
+void SendCoinsDialog::appendCoinControlSummary(QString& text) const
+{
+    if (!m_current_transaction || !m_current_transaction->getWtx()) return;
+    const CTransaction& tx{*m_current_transaction->getWtx()};
+    std::vector<COutPoint> outpoints;
+    outpoints.reserve(tx.vin.size());
+    for (const CTxIn& input : tx.vin) outpoints.push_back(input.prevout);
+    CAmount input_amount{0};
+    for (const auto& coin : model->wallet().getCoins(outpoints)) input_amount += coin.txout.nValue;
+    const CAmount fee{m_current_transaction->getTransactionFee()};
+    const CAmount change{input_amount - m_current_transaction->getTotalTransactionAmount() - fee};
+    const BitcoinUnit unit{model->getOptionsModel()->getDisplayUnit()};
+    text.append("<hr /><b>" + tr("Effective coin control") + "</b><br />");
+    text.append(tr("Inputs: %1 (%2)")
+                    .arg(tx.vin.size())
+                    .arg(BitcoinUnits::formatHtmlWithUnit(unit, input_amount)) + "<br />");
+    text.append(tr("Estimated virtual size: %1 vB").arg(m_current_transaction->getTransactionSize()) + "<br />");
+    const unsigned int vsize{m_current_transaction->getTransactionSize()};
+    text.append(tr("Fee rate: %1 sat/vB")
+                    .arg(QString::number(vsize > 0 ? static_cast<double>(fee) / vsize : 0, 'f', 1)) + "<br />");
+    text.append(tr("Change: %1").arg(BitcoinUnits::formatHtmlWithUnit(unit, std::max<CAmount>(change, 0))) + "<br />");
+}
+void SendCoinsDialog::invalidateSendPreview()
+{
+    if (m_sending) return;
+    m_current_transaction.reset();
+    m_coin_control->UnSelectAll();
+    coinControlUpdateLabels();
+}
+void SendCoinsDialog::walletChanged([[maybe_unused]] const interfaces::WalletBalances& balances)
+{
+    invalidateSendPreview();
+}
 void SendCoinsDialog::sendButtonClicked([[maybe_unused]] bool checked)
 {
     if(!model || !model->getOptionsModel())
@@ -506,6 +549,11 @@ void SendCoinsDialog::sendButtonClicked([[maybe_unused]] bool checked)
         fNewRecipientAllowed = true;
         return;
     }
+    if (!m_current_transaction) {
+        Q_EMIT message(tr("Send Coins"), tr("Wallet state changed. Review the current inputs and create a new preview."), CClientUIInterface::MSG_WARNING);
+        fNewRecipientAllowed = true;
+        return;
+    }
     bool send_failure = false;
     if (retval == QMessageBox::Save) {
@@ -552,9 +600,18 @@ void SendCoinsDialog::sendButtonClicked([[maybe_unused]] bool checked)
         // Broadcast the transaction, unless an external signer was used and it
         // failed, or more signatures are needed.
         if (broadcast) {
-            // now send the prepared transaction
-            model->sendCoins(*m_current_transaction);
-            Q_EMIT coinsSent(m_current_transaction->getWtx()->GetHash());
+            // Keep the prepared choice valid through the synchronous commit.
+            // Wallet balance notifications from the successful commit must not
+            // invalidate its own transaction before it is announced.
+            m_sending = true;
+            try {
+                model->sendCoins(*m_current_transaction);
+                Q_EMIT coinsSent(m_current_transaction->getWtx()->GetHash());
+            } catch (const std::runtime_error& e) {
+                Q_EMIT message(tr("Send Coins"), QString::fromStdString(e.what()), CClientUIInterface::MSG_ERROR);
+                send_failure = true;
+            }
+            m_sending = false;
         }
     }
     if (!send_failure) {