From ad26d809a4bdaa9fd552ead33d48d035b80407f3 Mon Sep 17 00:00:00 2001
From: Sander van Grieken <sander@outrightsolutions.nl>
Date: Mon, 6 Feb 2023 21:39:41 +0100
Subject: [PATCH] qml: update history in more cases when saving/removing tx

---
 electrum/gui/qml/qetxdetails.py | 2 ++
 electrum/gui/qml/qewallet.py    | 5 ++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/electrum/gui/qml/qetxdetails.py b/electrum/gui/qml/qetxdetails.py
index f3520a2f3..3a0d125c7 100644
--- a/electrum/gui/qml/qetxdetails.py
+++ b/electrum/gui/qml/qetxdetails.py
@@ -354,6 +354,7 @@ class QETxDetails(QObject, QtEventListener):
 
         self._wallet.wallet.adb.remove_transaction(txid)
         self._wallet.wallet.save_db()
+        self._wallet.historyModel.init_model(True)
 
     @pyqtSlot()
     def save(self):
@@ -367,6 +368,7 @@ class QETxDetails(QObject, QtEventListener):
                 return
             self._wallet.wallet.save_db()
             self.saveTxSuccess.emit()
+            self._wallet.historyModel.init_model(True)
         except AddTransactionException as e:
             self.saveTxError.emit('error', str(e))
         finally:
diff --git a/electrum/gui/qml/qewallet.py b/electrum/gui/qml/qewallet.py
index 563393d15..e6e3dcd59 100644
--- a/electrum/gui/qml/qewallet.py
+++ b/electrum/gui/qml/qewallet.py
@@ -513,10 +513,13 @@ class QEWallet(AuthMixin, QObject, QtEventListener):
 
         if not tx.is_complete():
             self._logger.debug('tx not complete')
-            return
+            broadcast = False
 
         if broadcast:
             self.broadcast(tx)
+        else:
+            # not broadcasted, so add to history now
+            self.historyModel.init_model(True)
 
     # this assumes a 2fa wallet, but there are no other tc_sign_wrapper hooks, so that's ok
     def on_sign_complete(self, broadcast, tx):