prefs: code cleanup

prefs.js

@@ -416,10 +416,7 @@ function addUser() {
 
 function editUser(id) {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 		disableHotkeys();
 
@@ -430,89 +427,70 @@ function editUser(id) {
 
 		disableContainerChildren("userOpToolbar", false);
 
-	xmlhttp.open("GET", "backend.php?op=pref-users&subop=edit&id=" +
+		var query = "backend.php?op=pref-users&subop=edit&id=" +
-		param_escape(id), true);
+			param_escape(id);
-	xmlhttp.onreadystatechange=infobox_callback;
+
-	xmlhttp.send(null);
+		new Ajax.Request(query,	{
+			onComplete: function(transport) {
+					infobox_callback2(transport);
+				} });
+
+	} catch (e) {
+		exception_error("editUser", e);
+	}
 		
 }
 
 function editFilter(id) {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 		disableHotkeys();
 
 		notify_progress("Loading, please wait...");
 
-//	document.getElementById("create_filter_btn").disabled = true;
-
 		disableContainerChildren("filterOpToolbar", false);
 
 		selectTableRowsByIdPrefix('prefFilterList', 'FILRR-', 'FICHK-', false);
 		selectTableRowById('FILRR-'+id, 'FICHK-'+id, true);
 
-	xmlhttp.open("GET", "backend.php?op=pref-filters&subop=edit&id=" + param_escape(id), true);
+		var query = "backend.php?op=pref-filters&subop=edit&id=" + 
-	xmlhttp.onreadystatechange=infobox_callback;
+			param_escape(id);
-	xmlhttp.send(null);
+
+		new Ajax.Request(query,	{
+			onComplete: function(transport) {
+					infobox_callback2(transport);
+				} });
+	} catch (e) {
+		exception_error("editFilter", e);
+	}
 }
 
 function editFeed(feed) {
 
-//	notify("Editing feed...");
+	try {
-
-	if (!xmlhttp_ready(xmlhttp)) {
-		printLockingError();
-		return
-	}
 
 		disableHotkeys();
 	
 		notify_progress("Loading, please wait...");
 	
-/*	document.getElementById("subscribe_to_feed_btn").disabled = true;
-
-	try {
-		document.getElementById("top25_feeds_btn").disabled = true;
-	} catch (e) {
-		// this button is not always available, no-op if not found
-	} */
-
 		// clean selection from all rows & select row being edited
 		selectTableRowsByIdPrefix('prefFeedList', 'FEEDR-', 'FRCHK-', false);
 		selectTableRowById('FEEDR-'+feed, 'FRCHK-'+feed, true);
 	
 		disableContainerChildren("feedOpToolbar", false);
 	
-	xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=editfeed&id=" +
+		var query = "backend.php?op=pref-feeds&subop=editfeed&id=" +
-		param_escape(feed), true);
+			param_escape(feed);
-
-	xmlhttp.onreadystatechange=infobox_callback;
-	xmlhttp.send(null);
-
-}
 	
-function editFeedCat(cat) {
+		new Ajax.Request(query,	{
+			onComplete: function(transport) {
+					infobox_callback2(transport);
+				} });
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	} catch (e) {
-		printLockingError();
+		exception_error("editFeed", e);
-		return
 	}
-
-	disableHotkeys();
-
-	notify_progress("Loading, please wait...");
-
-	active_feed_cat = cat;
-
-	xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=editCats&action=edit&id=" +
-		param_escape(cat), true);
-	xmlhttp.onreadystatechange=infobox_callback;
-	xmlhttp.send(null);
-
 }
 
 function getSelectedLabels() {
@@ -774,29 +752,6 @@ function feedEditCancel() {
 	return false;
 }
 
-function feedCatEditCancel() {
-
-	if (!xmlhttp_ready(xmlhttp)) {
-		printLockingError();
-		return
-	}
-
-	active_feed_cat = false;
-
-	try {
-		document.getElementById("subscribe_to_feed_btn").disabled = false;
-		document.getElementById("top25_feeds_btn").disabled = false;
-	} catch (e) {
-		// this button is not always available, no-op if not found
-	}
-
-	xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=editCats", true);
-	xmlhttp.onreadystatechange=infobox_callback;
-	xmlhttp.send(null);
-
-	return false;
-}
-
 function feedEditSave() {
 
 	try {
@@ -1087,25 +1042,6 @@ function editSelectedFeeds() {
 
 }
 
-function editSelectedFeedCat() {
-	var rows = getSelectedFeedCats();
-
-	if (rows.length == 0) {
-		alert(__("No categories are selected."));
-		return;
-	}
-
-	if (rows.length > 1) {
-		alert(__("Please select only one category."));
-		return;
-	}
-
-	notify("");
-
-	editFeedCat(rows[0]);
-
-}
-
 function piggie(enable) {
 	if (enable) {
 		debug("I LOVEDED IT!");