prefs: some async work (2)

prefs.js

@@ -55,35 +55,11 @@ function replace_pubkey_callback() {
 
 function feedlist_callback() {
 	if (xmlhttp.readyState == 4) {
-		try {	
+		return feedlist_callback2(xmlhttp);
-			var container = document.getElementById('prefContent');	
-			container.innerHTML=xmlhttp.responseText;
-			selectTab("feedConfig", true);
-
-			if (caller_subop) {
-				var tuple = caller_subop.split(":");
-				if (tuple[0] == 'editFeed') {
-					window.setTimeout('editFeed('+tuple[1]+')', 100);
-				}				
-
-				caller_subop = false;
-			}
-			if (typeof correctPNG != 'undefined') {
-				correctPNG();
-			}
-			notify("");
-			remove_splash();
-
-		} catch (e) {
-			exception_error("feedlist_callback", e);
-		}
 	}
 }
 
-/* stub for subscription dialog */
+function feedlist_callback2(transport) {
-
-function dlg_frefresh_callback(transport) {
-
 	try {	
 		var container = document.getElementById('prefContent');	
 		container.innerHTML=transport.responseText;
@@ -101,10 +77,17 @@ function dlg_frefresh_callback(transport) {
 			correctPNG();
 		}
 		notify("");
+		remove_splash();
+
 	} catch (e) {
-		exception_error("feedlist_callback", e);
+		exception_error("feedlist_callback2", e);
+	}
 }
 
+/* stub for subscription dialog */
+
+function dlg_frefresh_callback(transport) {
+	return feedlist_callback2(transport);
 }
 
 function filterlist_callback() {
@@ -119,14 +102,13 @@ function filterlist_callback() {
 	}
 }
 
-function labellist_callback() {
+function labellist_callback2(transport) {
 
 	try {
 
 		var container = document.getElementById('prefContent');
-		if (xmlhttp.readyState == 4) {
 			closeInfoBox();
-			container.innerHTML=xmlhttp.responseText;
+			container.innerHTML=transport.responseText;
 	
 			if (document.getElementById("prefLabelList")) {
 				var elems = document.getElementById("prefLabelList").getElementsByTagName("SPAN");
@@ -147,10 +129,9 @@ function labellist_callback() {
 			}
 			notify("");
 			remove_splash();
-		}
 
 	} catch (e) {
-		exception_error("labellist_callback", e);
+		exception_error("labellist_callback2", e);
 	}
 }
 
@@ -304,11 +285,6 @@ function addLabel() {
 
 	try {
 
-	if (!xmlhttp_ready(xmlhttp)) {
-		printLockingError();
-		return
-	}
-
 		var caption = prompt(__("Please enter label caption:"), "");
 	
 		if (caption == null) { 
@@ -323,11 +299,13 @@ function addLabel() {
 		// we can be called from some other tab
 		active_tab = "labelConfig";
 	
-	var query = "caption=" + param_escape(caption);
+		query = "backend.php?op=pref-labels&subop=add&caption=" + 
+			param_escape(caption);
 	
-	xmlhttp.open("GET", "backend.php?op=pref-labels&subop=add&" + query, true);			
+		new Ajax.Request(query,	{
-	xmlhttp.onreadystatechange=infobox_submit_callback;
+			onComplete: function(transport) {
-	xmlhttp.send(null);
+					infobox_submit_callback2(transport);
+				} });
 
 	} catch (e) {
 		exception_error("addLabel", e);
@@ -336,10 +314,7 @@ function addLabel() {
 
 function addFeed() {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 		var link = document.getElementById("fadd_link");
 	
@@ -350,15 +325,22 @@ function addFeed() {
 		} else {
 			notify_progress("Adding feed...");
 	
-		xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=add&from=tt-rss&feed_url=" +
+			var query = "backend.php?op=pref-feeds&subop=add&from=tt-rss&feed_url=" +
-			param_escape(link.value), true);
+				param_escape(link.value);
-		xmlhttp.onreadystatechange=feedlist_callback;
+	
-		xmlhttp.send(null);
+			new Ajax.Request(query,	{
+				onComplete: function(transport) {
+						feedlist_callback2(transport);
+					} });
 	
 			link.value = "";
 	
 		}
 
+	} catch (e) {
+		exception_error("addFeed", e);
+	}
+
 }
 
 function addFeedCat() {
@@ -515,11 +497,6 @@ function getSelectedFeedCats() {
 
 function removeSelectedLabels() {
 
-	if (!xmlhttp_ready(xmlhttp)) {
-		printLockingError();
-		return
-	}
-
 	var sel_rows = getSelectedLabels();
 
 	if (sel_rows.length > 0) {
@@ -529,10 +506,14 @@ function removeSelectedLabels() {
 		if (ok) {
 			notify_progress("Removing selected labels...");
 	
-			xmlhttp.open("GET", "backend.php?op=pref-labels&subop=remove&ids="+
+			var query = "backend.php?op=pref-labels&subop=remove&ids="+
-				param_escape(sel_rows.toString()), true);
+				param_escape(sel_rows.toString());
-			xmlhttp.onreadystatechange=labellist_callback;
+
-			xmlhttp.send(null);
+			new Ajax.Request(query,	{
+				onComplete: function(transport) {
+						labellist_callback2(transport);
+					} });
+
 		}
 	} else {
 		alert(__("No labels are selected."));
@@ -601,10 +582,7 @@ function removeSelectedFilters() {
 
 function removeSelectedFeeds() {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 		var sel_rows = getSelectedFeeds();
 	
@@ -616,16 +594,21 @@ function removeSelectedFeeds() {
 	
 				notify_progress("Unsubscribing from selected feeds...");
 		
-			xmlhttp.open("GET", "backend.php?op=pref-feeds&subop=remove&ids="+
+				var query = "backend.php?op=pref-feeds&subop=remove&ids="+
-				param_escape(sel_rows.toString()), true);
+					param_escape(sel_rows.toString());
-			xmlhttp.onreadystatechange=feedlist_callback;
+	
-			xmlhttp.send(null);
+				new Ajax.Request(query,	{
+					onComplete: function(transport) {
+							feedlist_callback2(transport);
+						} });
 			}
 	
 		} else {
-
 			alert(__("No feeds are selected."));
+		}
 
+	} catch (e) {
+		exception_error("removeSelectedFeeds", e);
 	}
 	
 	return false;
@@ -1064,20 +1047,24 @@ function updateFilterList(sort_key) {
 
 function updateLabelList(sort_key) {
 
-	if (!xmlhttp_ready(xmlhttp)) {
+	try {
-		printLockingError();
-		return
-	}
 
 		var label_search = document.getElementById("label_search");
 		var search = "";
 		if (label_search) { search = label_search.value; }
 	
-	xmlhttp.open("GET", "backend.php?op=pref-labels&sort=" + 
+		var query = "backend.php?op=pref-labels&sort=" + 
 			param_escape(sort_key) +
-		"&search=" + param_escape(search), true);
+			"&search=" + param_escape(search);
-	xmlhttp.onreadystatechange=labellist_callback;
+	
-	xmlhttp.send(null);
+		new Ajax.Request(query,	{
+			onComplete: function(transport) {
+				labellist_callback2(transport);
+			} });
+
+	} catch (e) {
+		exception_error("updateLabelList", e);
+	}
 }
 
 function updatePrefsList() {