From 183064f85715cd21931f55607822881de1ea0a46 Mon Sep 17 00:00:00 2001
From: Damien Churchill <damoxc@gmail.com>
Date: Tue, 30 Mar 2010 14:31:36 +0100
Subject: [PATCH] store the other menu item when we encounter it looping over
 for later

---
 deluge/ui/web/js/deluge-all/Menus.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/deluge/ui/web/js/deluge-all/Menus.js b/deluge/ui/web/js/deluge-all/Menus.js
index 082204ec0..6951633e7 100644
--- a/deluge/ui/web/js/deluge-all/Menus.js
+++ b/deluge/ui/web/js/deluge-all/Menus.js
@@ -251,6 +251,7 @@ Deluge.StatusbarMenu = Ext.extend(Ext.menu.Menu, {
 		// set the new value
 		value = (value == 0) ? -1 : value;
 
+		var other = null;
 		// uncheck all items
 		this.items.each(function(item) {
 			if (item.setChecked) {
@@ -263,14 +264,15 @@ Deluge.StatusbarMenu = Ext.extend(Ext.menu.Menu, {
 				}
 				item.resumeEvents();
 			}
+
+			if (item.value == 'other') other = item;
 		});
 
 		if (beenSet) return;
 
-		var item = this.items.get('other');
-		item.suspendEvents();
-		item.setChecked(true);
-		item.resumeEvents();
+		other.suspendEvents();
+		other.setChecked(true);
+		other.resumeEvents();
 	}
 });