edit filter dialog: code cleanup

1 files changed, 22 insertions, 33 deletions

diff --git a/prefs.js b/prefs.js
index bb8286d05..d82761609 100644
--- a/prefs.js
+++ b/prefs.js
@@ -265,6 +265,28 @@ function editFilter(id, event) {
 			id: "filterEditDlg",
 			title: __("Edit Filter"),
 			style: "width: 600px",
+			removeFilter: function() {
+				var title = this.attr('value').reg_exp;
+				var msg = __("Remove filter %s?").replace("%s", title);
+		
+				if (confirm(msg)) {
+					this.hide();
+
+					notify_progress("Removing filter...");
+		
+					var id = this.attr('value').id;
+
+					var query = "?op=pref-filters&subop=remove&ids="+
+						param_escape(id);
+
+					new Ajax.Request("backend.php",	{
+						parameters: query,
+						onComplete: function(transport) {
+							updateFilterList();
+						} });
+				}
+			},
+
 			execute: function() {
 				if (this.validate()) {
 
@@ -1437,39 +1459,6 @@ function rescore_all_feeds() {
 	}
 }
 
-function removeFilter(id, title) {
-
-	try {
-
-		var msg = __("Remove filter %s?").replace("%s", title);
-	
-		var ok = confirm(msg);
-	
-		if (ok) {
-
-			if (dijit.byId("filterEditDlg"))
-				dijit.byId("filterEditDlg").hide();
-
-			notify_progress("Removing filter...");
-		
-			var query = "?op=pref-filters&subop=remove&ids="+
-				param_escape(id);
-
-			new Ajax.Request("backend.php",	{
-				parameters: query,
-				onComplete: function(transport) {
-						filterlist_callback2(transport);
-			} });
-
-		}
-
-	} catch (e) {
-		exception_error("removeFilter", e);
-	}
-
-	return false;
-}
-
 function labelColorReset() {
 	try {
 		var labels = getSelectedLabels();