Ticket #1210: 0001-Correctly-store-and-restore-the-sort_column_id.patch

File 0001-Correctly-store-and-restore-the-sort_column_id.patch, 1.3 KB (added by squisher, 14 years ago)

Patch to correctly check the return values of get_sort_column_id(). Not tested, sorry.

  • deluge/ui/gtkui/files_tab.py 

    From 6aa925a8edc5b3e9038ec5a74086c94ac87781ce Mon Sep 17 00:00:00 2001
From: David Mohr <david@mcbf.net>
Date: Wed, 31 Mar 2010 22:45:57 -0600
Subject: [PATCH] Correctly store and restore the sort_column_id

---
 deluge/ui/gtkui/files_tab.py |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/deluge/ui/gtkui/files_tab.py b/deluge/ui/gtkui/files_tab.py
index f81820b..bc1bdcd 100644
    a b def save_state(self): 
    230230        # Setup state dict 
    231231        state = { 
    232232            "columns": {}, 
    233             "sort_id": column_id, 
    234             "sort_order": int(sort_order) if sort_order else None 
     233            "sort_id": int(column_id) if column_id >= 0 else None, 
     234            "sort_order": int(sort_order) if sort_order >= 0 else None 
    235235        } 
    236236 
    237237        for index, column in enumerate(self.listview.get_columns()): 
    … … def load_state(self): 
    268268        if state == None: 
    269269            return 
    270270 
    271         if state["sort_id"] and state["sort_order"] is not None: 
     271        if state["sort_id"] is not None and state["sort_order"] is not None: 
    272272            self.treestore.set_sort_column_id(state["sort_id"], state["sort_order"]) 
    273273 
    274274        for (index, column) in enumerate(self.listview.get_columns()):