gui: Fix multiple dialogs for same tx in TransactionView

Limit to one the transaction details dialogs that a user can open.
bitcoin-core · Apr 12, 2024 · dbbec32 · dbbec32
1 parent e319569
commit dbbec32
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 8 deletions.
diff --git a/src/qt/transactiondescdialog.cpp b/src/qt/transactiondescdialog.cpp
@@ -15,7 +15,8 @@ TransactionDescDialog::TransactionDescDialog(const QModelIndex &idx, QWidget *pa
     ui(new Ui::TransactionDescDialog)
 {
     ui->setupUi(this);
-    setWindowTitle(tr("Details for %1").arg(idx.data(TransactionTableModel::TxHashRole).toString()));
+    m_transaction_id = idx.data(TransactionTableModel::TxHashRole).toString();
+    setWindowTitle(tr("Details for %1").arg(m_transaction_id));
     QString desc = idx.data(TransactionTableModel::LongDescriptionRole).toString();
     ui->detailText->setHtml(desc);
 

diff --git a/src/qt/transactiondescdialog.h b/src/qt/transactiondescdialog.h
@@ -24,8 +24,12 @@ class TransactionDescDialog : public QDialog
     explicit TransactionDescDialog(const QModelIndex &idx, QWidget *parent = nullptr);
     ~TransactionDescDialog();
 
+    QString getTransactionId() {return m_transaction_id;};
+
 private:
     Ui::TransactionDescDialog *ui;
+    QString m_transaction_id;
+
 };
 
 #endif // BITCOIN_QT_TRANSACTIONDESCDIALOG_H
diff --git a/src/qt/transactionview.cpp b/src/qt/transactionview.cpp
@@ -528,13 +528,15 @@ void TransactionView::showDetails()
     QModelIndexList selection = transactionView->selectionModel()->selectedRows();
     if(!selection.isEmpty())
     {
-        TransactionDescDialog *dlg = new TransactionDescDialog(selection.at(0));
-        dlg->setAttribute(Qt::WA_DeleteOnClose);
-        m_opened_dialogs.append(dlg);
-        connect(dlg, &QObject::destroyed, [this, dlg] {
-            m_opened_dialogs.removeOne(dlg);
-        });
-        dlg->show();
+        if(!detailsAlreadyShown(selection.at(0))) {
+            TransactionDescDialog *dlg = new TransactionDescDialog(selection.at(0));
+            dlg->setAttribute(Qt::WA_DeleteOnClose);
+            m_opened_dialogs.append(dlg);
+            connect(dlg, &QObject::destroyed, [this, dlg] {
+                m_opened_dialogs.removeOne(dlg);
+            });
+            dlg->show();
+        }
     }
 }
 
@@ -663,3 +665,13 @@ void TransactionView::closeOpenedDialogs()
     }
     m_opened_dialogs.clear();
 }
+
+bool TransactionView::detailsAlreadyShown(const QModelIndex &idx)
+{
+    for (TransactionDescDialog* dlg : m_opened_dialogs) {
+        if (dlg->getTransactionId() == idx.data(TransactionTableModel::TxHashRole).toString()) {
+            return true;
+        }
+    }
+    return false;
+}
diff --git a/src/qt/transactionview.h b/src/qt/transactionview.h
@@ -108,6 +108,7 @@ private Q_SLOTS:
     void updateWatchOnlyColumn(bool fHaveWatchOnly);
     void abandonTx();
     void bumpFee(bool checked);
+    bool detailsAlreadyShown(const QModelIndex &idx);
 
 Q_SIGNALS:
     void doubleClicked(const QModelIndex&);