prefs: some async work (3)

prefs.js

@@ -60,7 +60,9 @@ function feedlist_callback() {
 }
 
 function feedlist_callback2(transport) {
+
 	try {	
+
 		var container = document.getElementById('prefContent');	
 		container.innerHTML=transport.responseText;
 		selectTab("feedConfig", true);
@@ -90,15 +92,20 @@ function dlg_frefresh_callback(transport) {
 	return feedlist_callback2(transport);
 }
 
-function filterlist_callback() {
+function filterlist_callback2(transport) {
 	var container = document.getElementById('prefContent');
-	if (xmlhttp.readyState == 4) {
+	container.innerHTML=transport.responseText;
-		container.innerHTML=xmlhttp.responseText;
 	if (typeof correctPNG != 'undefined') {
 		correctPNG();
 	}
 	notify("");
 	remove_splash();
+}
+
+
+function filterlist_callback() {
+	if (xmlhttp.readyState == 4) {
+		filterlist_callback2(xmlhttp);
 	}
 }
 
@@ -174,10 +181,14 @@ function gethelp_callback() {
 
 function notify_callback() {
 	if (xmlhttp.readyState == 4) {
-		notify_info(xmlhttp.responseText);
+		notify_callback2(xmlhttp);
 	} 
 }
 
+function notify_callback2(transport) {
+	notify_info(transport.responseText);	 
+}
+
 function prefs_reset_callback() {
 	if (xmlhttp.readyState == 4) {
 		notify_info(xmlhttp.responseText);
@@ -237,10 +248,7 @@ function infobox_feed_cat_callback2(transport) {
 
 function updateFeedList(sort_key) {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 	var feed_search = document.getElementById("feed_search");
 	var search = "";
@@ -253,13 +261,18 @@ function updateFeedList(sort_key) {
 		slat_checked = slat.checked;
 	}
 
-	xmlhttp.open("GET", "backend.php?op=pref-feeds" +
+	var query = "backend.php?op=pref-feeds" +
 		"&sort=" + param_escape(sort_key) + 
 		"&slat=" + param_escape(slat_checked) +
-		"&search=" + param_escape(search), true);
+		"&search=" + param_escape(search);
-	xmlhttp.onreadystatechange=feedlist_callback;
-	xmlhttp.send(null);
 
+	new Ajax.Request(query, {
+		onComplete: function(transport) { 
+			feedlist_callback2(transport); 
+		} });
+	} catch (e) {
+		exception_error("updateFeedList", e);
+	}
 }
 
 function updateUsersList(sort_key) {
@@ -616,11 +629,6 @@ function removeSelectedFeeds() {
 
 function clearSelectedFeeds() {
 
-	if (!xmlhttp_ready(xmlhttp)) {
-		printLockingError();
-		return
-	}
-
 	var sel_rows = getSelectedFeeds();
 
 	if (sel_rows.length > 1) {
@@ -742,21 +750,17 @@ function feedEditSave() {
 
 	try {
 	
-		if (!xmlhttp_ready(xmlhttp)) {
-			printLockingError();
-			return
-		}
-
 		// FIXME: add parameter validation
 
 		var query = Form.serialize("edit_feed_form");
 
 		notify_progress("Saving feed...");
 
-		xmlhttp.open("POST", "backend.php", true);
+		new Ajax.Request("backend.php", {
-		xmlhttp.onreadystatechange=feedlist_callback;
+			parameters: query,
-		xmlhttp.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
+			onComplete: function(transport) { 
-		xmlhttp.send(query);
+				feedlist_callback2(transport); 
+			} });
 
 		closeInfoBox();
 
@@ -1296,10 +1300,13 @@ function categorizeSelectedFeeds() {
 
 		notify_progress("Changing category of selected feeds...");
 
-		xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=categorize&ids="+
+		var query = "backend.php?op=pref-feeds&subop=categorize&ids="+
-			param_escape(sel_rows.toString()) + "&cat_id=" + param_escape(cat_id), true);
+			param_escape(sel_rows.toString()) + "&cat_id=" + param_escape(cat_id);
-		xmlhttp.onreadystatechange=feedlist_callback;
+
-		xmlhttp.send(null);
+		new Ajax.Request(query, {
+			onComplete: function(transport) { 
+				feedlist_callback2(transport); 
+			} });
 
 	} else {
 
@@ -1340,10 +1347,15 @@ function feedBrowserSubscribe() {
 
 		if (selected.length > 0) {
 			closeInfoBox();
-			xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=massSubscribe&ids="+
+
-				param_escape(selected.toString()), true);
+			var query = "backend.php?op=pref-feeds&subop=massSubscribe&ids="+
-			xmlhttp.onreadystatechange=feedlist_callback;
+				param_escape(selected.toString());
-			xmlhttp.send(null);
+
+			new Ajax.Request(query, {
+				onComplete: function(transport) { 
+					feedlist_callback2(transport); 
+				} });
+
 		} else {
 			alert(__("No feeds are selected."));
 		}
@@ -1866,11 +1878,6 @@ function clearFeedArticles(feed_id) {
 
 function rescoreSelectedFeeds() {
 
-	if (!xmlhttp_ready(xmlhttp)) {
-		printLockingError();
-		return
-	}
-
 	var sel_rows = getSelectedFeeds();
 
 	if (sel_rows.length > 0) {
@@ -1881,10 +1888,14 @@ function rescoreSelectedFeeds() {
 		if (ok) {
 			notify_progress("Rescoring selected feeds...", true);
 	
-			xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=rescore&quiet=1&ids="+
+			var query = "backend.php?op=pref-feeds&subop=rescore&quiet=1&ids="+
-				param_escape(sel_rows.toString()), true);
+				param_escape(sel_rows.toString());
-			xmlhttp.onreadystatechange=notify_callback;
+
-			xmlhttp.send(null);
+			new Ajax.Request(query,	{
+				onComplete: function(transport) {
+						notify_callback2(transport);
+			} });
+
 		}
 	} else {
 		alert(__("No feeds are selected."));
@@ -1899,18 +1910,18 @@ function rescore_all_feeds() {
 	if (ok) {
 		notify_progress("Rescoring feeds...", true);
 
-			xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=rescoreAll&quiet=1", true);
+		var query = "backend.php?op=pref-feeds&subop=rescoreAll&quiet=1";
-			xmlhttp.onreadystatechange=notify_callback;
+
-			xmlhttp.send(null);
+		new Ajax.Request(query,	{
+			onComplete: function(transport) {
+					notify_callback2(transport);
+		} });
 	}
 }
 
 function removeFilter(id, title) {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 		var msg = __("Remove filter %s?").replace("%s", title);
 	
@@ -1921,10 +1932,18 @@ function removeFilter(id, title) {
 	
 			notify_progress("Removing filter...");
 		
-		xmlhttp.open("GET", "backend.php?op=pref-filters&subop=remove&ids="+
+			var query = "backend.php?op=pref-filters&subop=remove&ids="+
-			param_escape(id), true);
+				param_escape(id);
-		xmlhttp.onreadystatechange=filterlist_callback;
+
-		xmlhttp.send(null);
+			new Ajax.Request(query,	{
+				onComplete: function(transport) {
+						filterlist_callback2(transport);
+			} });
+
+		}
+
+	} catch (e) {
+		exception_error("removeFilter", e);
 	}
 
 	return false;
@@ -2005,10 +2024,12 @@ function feedsEditSave() {
 	
 			notify_progress("Saving feeds...");
 	
-			xmlhttp.open("POST", "backend.php", true);
+			new Ajax.Request("backend.php", {
-			xmlhttp.onreadystatechange=feedlist_callback;
+				parameters: query,
-			xmlhttp.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
+				onComplete: function(transport) { 
-			xmlhttp.send(query);
+					feedlist_callback2(transport); 
+				} });
+
 		}
 
 		return false;