From 22aafb532e431b214a7cb145581dbc5279b2ce34 Mon Sep 17 00:00:00 2001
From: Andrew Resch <andrewresch@gmail.com>
Date: Mon, 31 Mar 2008 08:35:54 +0000
Subject: [PATCH] Fix edit trackers dialog.

---
 TODO                                  |  1 -
 deluge/ui/gtkui/edittrackersdialog.py | 23 +++--------------------
 2 files changed, 3 insertions(+), 21 deletions(-)

diff --git a/TODO b/TODO
index b051d6fd0..438aaed7e 100644
--- a/TODO
+++ b/TODO
@@ -10,7 +10,6 @@ For 0.6 release:
 * Update checking
 * Translations
 * Show proper priority levels in Files tab, plus add menu to alter priorities
-* Edit tracker dialog needs fixin'
 * Implement add by url and add by hash
 
 After 0.6 release:
diff --git a/deluge/ui/gtkui/edittrackersdialog.py b/deluge/ui/gtkui/edittrackersdialog.py
index 1b37ee035..73058682d 100644
--- a/deluge/ui/gtkui/edittrackersdialog.py
+++ b/deluge/ui/gtkui/edittrackersdialog.py
@@ -112,17 +112,7 @@ class EditTrackersDialog:
         num_rows = self.liststore.iter_n_children(None)
         if selected != None and num_rows > 1:
             tier = self.liststore.get_value(selected, 0)
-            new_tier = tier - 1
-            # Return if the tier is already at the top
-            if tier == 0:
-                return
-            # Change the tier of the tracker we're surplanting
-            def change_tier(model, path, iter, data):
-                t = model.get_value(iter, 0)
-                if t == data:
-                    model.set_value(iter, 0, data + 1)            
-            self.liststore.foreach(change_tier, new_tier)
-            
+            new_tier = tier + 1
             # Now change the tier for this tracker
             self.liststore.set_value(selected, 0, new_tier)
             
@@ -144,16 +134,9 @@ class EditTrackersDialog:
         num_rows = self.liststore.iter_n_children(None)
         if selected != None and num_rows > 1:
             tier = self.liststore.get_value(selected, 0)
-            new_tier = tier + 1
-            # This tracker is on the bottom already
-            if new_tier == num_rows:
+            if not tier > 0:
                 return
-            # Change the tier of the tracker we're surplanting
-            def change_tier(model, path, iter, data):
-                t = model.get_value(iter, 0)
-                if t == data:
-                    model.set_value(iter, 0, data - 1)
-            self.liststore.foreach(change_tier, new_tier)
+            new_tier = tier - 1
             # Now change the tier for this tracker
             self.liststore.set_value(selected, 0, new_tier)