tqt history list: small clean-up - electrum - Electrum Bitcoin wallet
HTML git clone https://git.parazyd.org/electrum
DIR Log
DIR Files
DIR Refs
DIR Submodules
---
DIR commit 059beab700fcc4fe5cb979bae8c975ff6968554d
DIR parent ea235a1468bacb609cf91c2639e2493df0a0306d
HTML Author: SomberNight <somber.night@protonmail.com>
Date: Mon, 3 Dec 2018 19:12:36 +0100
qt history list: small clean-up
Diffstat:
M electrum/gui/qt/history_list.py | 22 ++++++----------------
M electrum/gui/qt/main_window.py | 1 +
M electrum/gui/qt/util.py | 11 ++++++-----
M electrum/util.py | 4 ++--
4 files changed, 15 insertions(+), 23 deletions(-)
---
DIR diff --git a/electrum/gui/qt/history_list.py b/electrum/gui/qt/history_list.py
t@@ -104,9 +104,6 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
self.end_timestamp = None
self.years = []
self.create_toolbar_buttons()
- self.wallet = None
-
- root = self.std_model.invisibleRootItem()
self.wallet = self.parent.wallet # type: Abstract_Wallet
fx = self.parent.fx
t@@ -144,7 +141,6 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
self.editable_columns -= {5}
col_count = self.std_model.columnCount()
diff = col_count-len(headers)
- grew = False
if col_count > len(headers):
if diff == 2:
self.std_model.removeColumns(6, diff)
t@@ -155,7 +151,6 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
while len(items) > col_count:
items.pop()
elif col_count < len(headers):
- grew = True
self.std_model.clear()
self.txid_to_items.clear()
self.transactions.clear()
t@@ -300,11 +295,9 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
tx_mined_status = TxMinedStatus(height, conf, timestamp, None)
status, status_str = self.wallet.get_tx_status(tx_hash, tx_mined_status)
has_invoice = self.wallet.invoices.paid.get(tx_hash)
- icon = self.icon_cache.get(":icons/" + TX_ICONS[status])
v_str = self.parent.format_amount(value, is_diff=True, whitespaces=True)
balance_str = self.parent.format_amount(balance, whitespaces=True)
entry = ['', status_str, label, v_str, balance_str]
- fiat_value = None
item = [QStandardItem(e) for e in entry]
item[3].setData(value, self.SORT_ROLE)
item[4].setData(balance, self.SORT_ROLE)
t@@ -316,11 +309,11 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
item[2].setForeground(self.red_brush)
item[3].setForeground(self.red_brush)
self.txid_to_items[tx_hash] = item
- self.update_item(tx_hash, self.parent.wallet.get_tx_height(tx_hash))
+ self.update_item(tx_hash, self.wallet.get_tx_height(tx_hash))
source_row_idx = self.std_model.rowCount()
self.std_model.insertRow(source_row_idx, item)
new_idx = self.std_model.index(source_row_idx, 0)
- history = self.parent.fx.show_history()
+ history = fx.show_history()
if history:
self.update_fiat(tx_hash, tx_item)
self.hide_row(self.proxy.mapFromSource(new_idx).row())
t@@ -344,7 +337,6 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
@profiler
def update(self):
- self.wallet = self.parent.wallet # type: Abstract_Wallet
fx = self.parent.fx
r = self.wallet.get_full_history(domain=self.get_domain(), from_timestamp=None, to_timestamp=None, fx=fx)
seen = set()
t@@ -374,7 +366,7 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
old = self.transactions[txid]
if old == row:
continue
- self.update_item(txid, self.parent.wallet.get_tx_height(txid))
+ self.update_item(txid, self.wallet.get_tx_height(txid))
if history:
self.update_fiat(txid, row)
balance_str = self.parent.format_amount(row['balance'].value, whitespaces=True)
t@@ -422,16 +414,16 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
key = item.data(self.TX_HASH_ROLE)
# fixme
if column == 2:
- self.parent.wallet.set_label(key, text)
+ self.wallet.set_label(key, text)
self.update_labels()
self.parent.update_completions()
elif column == 5:
tx_item = self.transactions[key]
- self.parent.wallet.set_fiat_value(key, self.parent.fx.ccy, text, self.parent.fx, tx_item['value'].value)
+ self.wallet.set_fiat_value(key, self.parent.fx.ccy, text, self.parent.fx, tx_item['value'].value)
value = tx_item['value'].value
if value is not None:
fee = tx_item['fee']
- fiat_fields = self.parent.wallet.get_tx_item_fiat(key, value, self.parent.fx, fee.value if fee else None)
+ fiat_fields = self.wallet.get_tx_item_fiat(key, value, self.parent.fx, fee.value if fee else None)
tx_item.update(fiat_fields)
self.update_fiat(key, tx_item)
else:
t@@ -463,8 +455,6 @@ class HistoryList(MyTreeView, AcceptFileDragDrop):
item.setText(label)
def update_item(self, tx_hash, tx_mined_status):
- if self.wallet is None:
- return
conf = tx_mined_status.conf
status, status_str = self.wallet.get_tx_status(tx_hash, tx_mined_status)
icon = self.icon_cache.get(":icons/" + TX_ICONS[status])
DIR diff --git a/electrum/gui/qt/main_window.py b/electrum/gui/qt/main_window.py
t@@ -117,6 +117,7 @@ class ElectrumWindow(QMainWindow, MessageBoxMixin, PrintError):
self.setup_exception_hook()
self.network = gui_object.daemon.network # type: Network
+ assert wallet, "no wallet"
self.wallet = wallet
self.fx = gui_object.daemon.fx # type: FxThread
self.invoices = wallet.invoices
DIR diff --git a/electrum/gui/qt/util.py b/electrum/gui/qt/util.py
t@@ -4,8 +4,7 @@ import sys
import platform
import queue
from functools import partial
-from typing import NamedTuple, Callable, Optional
-from abc import abstractmethod
+from typing import NamedTuple, Callable, Optional, TYPE_CHECKING
from PyQt5.QtGui import *
from PyQt5.QtCore import *
t@@ -15,6 +14,9 @@ from electrum.i18n import _, languages
from electrum.util import FileImportFailed, FileExportFailed
from electrum.paymentrequest import PR_UNPAID, PR_PAID, PR_EXPIRED
+if TYPE_CHECKING:
+ from .main_window import ElectrumWindow
+
if platform.system() == 'Windows':
MONOSPACE_FONT = 'Lucida Console'
t@@ -401,7 +403,7 @@ class ElectrumItemDelegate(QStyledItemDelegate):
class MyTreeView(QTreeView):
- def __init__(self, parent, create_menu, stretch_column=None, editable_columns=None):
+ def __init__(self, parent: 'ElectrumWindow', create_menu, stretch_column=None, editable_columns=None):
super().__init__(parent)
self.parent = parent
self.config = self.parent.config
t@@ -513,13 +515,12 @@ class MyTreeView(QTreeView):
if self.current_filter:
self.filter(self.current_filter)
- @abstractmethod
def should_hide(self, row):
"""
row_num is for self.model(). So if there is a proxy, it is the row number
in that!
"""
- pass
+ return False
def item_from_coordinate(self, row_num, column):
if isinstance(self.model(), QSortFilterProxyModel):
DIR diff --git a/electrum/util.py b/electrum/util.py
t@@ -860,8 +860,8 @@ def ignore_exceptions(func):
class TxMinedStatus(NamedTuple):
height: int
conf: int
- timestamp: int
- header_hash: str
+ timestamp: Optional[int]
+ header_hash: Optional[str]
class VerifiedTxInfo(NamedTuple):