From 43cb787b444e62bae0f60284ce01b7afce50ba54 Mon Sep 17 00:00:00 2001
From: Damien Churchill <damoxc@gmail.com>
Date: Mon, 3 Oct 2011 21:11:13 +0100
Subject: [PATCH] webui: fix closing the connection manager

Don't override the onHide method in the ConnectionManager, this
was stopping the window from being able to close.
---
 deluge/ui/web/js/deluge-all/ConnectionManager.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/deluge/ui/web/js/deluge-all/ConnectionManager.js b/deluge/ui/web/js/deluge-all/ConnectionManager.js
index 999041e91..4fda8e815 100644
--- a/deluge/ui/web/js/deluge-all/ConnectionManager.js
+++ b/deluge/ui/web/js/deluge-all/ConnectionManager.js
@@ -46,8 +46,8 @@ Ext.define('Deluge.ConnectionManager', {
 
     initComponent: function() {
         this.callParent(arguments);
-        this.on('hide',  this.onHide, this);
-        this.on('show', this.onShow, this);
+        this.afterMethod('onHide', this.afterHide, this);
+        this.afterMethod('onShow', this.afterShow, this);
 
         deluge.events.on('login', this.onLogin, this);
         deluge.events.on('logout', this.onLogout, this);
@@ -261,7 +261,7 @@ Ext.define('Deluge.ConnectionManager', {
     },
 
     // private
-    onHide: function() {
+    afterHide: function() {
         if (this.running) window.clearInterval(this.running);
     },
 
@@ -333,7 +333,7 @@ Ext.define('Deluge.ConnectionManager', {
 
     // FIXME: Find out why this is being fired twice
     // private
-    onShow: function() {
+    afterShow: function() {
         if (!this.addHostButton) {
             var buttons = this.grid.query('button');
             this.addHostButton = buttons[0];