[GtkUI] Fix ETA being copied to neighboring empty cells

An optimization that avoided re-rendering treeview cells sometimes
went wrong, and rendered a value from the wrong row when moving
the mouse around the torrentview window.

Closes: https://dev.deluge-torrent.org/ticket/3500
Closes: https://github.com/deluge-torrent/deluge/pull/371
This commit is contained in:
Chase Sterling 2022-02-09 23:57:18 -05:00 committed by Calum Lind
parent b29829f571
commit aa74261d50
No known key found for this signature in database
GPG Key ID: 90597A687B836BA3
1 changed files with 0 additions and 5 deletions

View File

@ -39,7 +39,6 @@ ICON_STATE = {
# renderer. This is much cheaper than fetch the current value and test if # renderer. This is much cheaper than fetch the current value and test if
# it's equal. # it's equal.
func_last_value = { func_last_value = {
'cell_data_time': None,
'cell_data_ratio_seeds_peers': None, 'cell_data_ratio_seeds_peers': None,
'cell_data_ratio_ratio': None, 'cell_data_ratio_ratio': None,
'cell_data_ratio_avail': None, 'cell_data_ratio_avail': None,
@ -219,10 +218,6 @@ def cell_data_peer(column, cell, model, row, data):
def cell_data_time(column, cell, model, row, data): def cell_data_time(column, cell, model, row, data):
"""Display value as time, eg 1m10s""" """Display value as time, eg 1m10s"""
time = model.get_value(row, data) time = model.get_value(row, data)
if func_last_value['cell_data_time'] == time:
return
func_last_value['cell_data_time'] = time
if time <= 0: if time <= 0:
time_str = '' time_str = ''
else: else: