consolidate counter parsing code

functions.js

@@ -344,66 +344,74 @@ if (!xmlhttp_rpc && typeof XMLHttpRequest!='undefined') {
 	xmlhttp_rpc = new XMLHttpRequest();
 }
 
-function all_counters_callback() {
-	if (xmlhttp_rpc.readyState == 4) {
-		try {
-			if (!xmlhttp_rpc.responseXML || !xmlhttp_rpc.responseXML.firstChild) {
-				notify("[all_counters_callback] backend did not return valid XML");
-				return;
+function parse_counters(reply, f_document) {
+	try {
+		for (var l = 0; l < reply.childNodes.length; l++) {
+			var id = reply.childNodes[l].getAttribute("id");
+			var t = reply.childNodes[l].getAttribute("type");
+			var ctr = reply.childNodes[l].getAttribute("counter");
+	
+			if (id == "global-unread") {
+				parent.global_unread = ctr;
+				parent.updateTitle();
+				continue;
 			}
 	
-			var reply = xmlhttp_rpc.responseXML.firstChild;
-	
-			var f_document = parent.frames["feeds-frame"].document;
-	
-			for (var l = 0; l < reply.childNodes.length; l++) {
-				var id = reply.childNodes[l].getAttribute("id");
-				var t = reply.childNodes[l].getAttribute("type");
-				var ctr = reply.childNodes[l].getAttribute("counter");
-
-				if (id == "global-unread") {
-					parent.global_unread = ctr;
-					parent.updateTitle();
-					continue;
-				}
-
-				if (t == "category") {
-					var catctr = f_document.getElementById("FCATCTR-" + id);
-					if (catctr) {
-						catctr.innerHTML = "(" + ctr + " unread)";
-					}
-					continue;
+			if (t == "category") {
+				var catctr = f_document.getElementById("FCATCTR-" + id);
+				if (catctr) {
+					catctr.innerHTML = "(" + ctr + " unread)";
 				}
+				continue;
+			}
 		
-				var feedctr = f_document.getElementById("FEEDCTR-" + id);
-				var feedu = f_document.getElementById("FEEDU-" + id);
-				var feedr = f_document.getElementById("FEEDR-" + id);
-	
-				if (feedctr && feedu && feedr) {
+			var feedctr = f_document.getElementById("FEEDCTR-" + id);
+			var feedu = f_document.getElementById("FEEDU-" + id);
+			var feedr = f_document.getElementById("FEEDR-" + id);
 		
-					feedu.innerHTML = ctr;
+			if (feedctr && feedu && feedr) {
 		
-					if (ctr > 0) {					
-						feedctr.className = "odd";
-						if (!feedr.className.match("Unread")) {
-							var is_selected = feedr.className.match("Selected");
+				feedu.innerHTML = ctr;
 	
-							feedr.className = feedr.className.replace("Selected", "");
-							feedr.className = feedr.className.replace("Unread", "");
+				if (ctr > 0) {					
+					feedctr.className = "odd";
+					if (!feedr.className.match("Unread")) {
+						var is_selected = feedr.className.match("Selected");
 		
-							feedr.className = feedr.className + "Unread";
+						feedr.className = feedr.className.replace("Selected", "");
+						feedr.className = feedr.className.replace("Unread", "");
 		
-							if (is_selected) {
-								feedr.className = feedr.className + "Selected";
-							}
+						feedr.className = feedr.className + "Unread";
 		
+						if (is_selected) {
+							feedr.className = feedr.className + "Selected";
 						}
-					} else {
-						feedctr.className = "invisible";
-						feedr.className = feedr.className.replace("Unread", "");
+		
 					}
+				} else {
+					feedctr.className = "invisible";
+					feedr.className = feedr.className.replace("Unread", "");
 				}			
 			}
+		}
+	} catch (e) {
+		exception_error(e);
+	}
+}
+
+function all_counters_callback() {
+	if (xmlhttp_rpc.readyState == 4) {
+		try {
+			if (!xmlhttp_rpc.responseXML || !xmlhttp_rpc.responseXML.firstChild) {
+				notify("[all_counters_callback] backend did not return valid XML");
+				return;
+			}
+	
+			var reply = xmlhttp_rpc.responseXML.firstChild;
+			var f_document = parent.frames["feeds-frame"].document;
+
+			parse_counters(reply, f_document);
+	
 		} catch (e) {
 			exception_error("all_counters_callback", e);
 		}

tt-rss.js

@@ -93,34 +93,7 @@ function refetch_callback() {
 	
 			var f_document = window.frames["feeds-frame"].document;
 
-			for (var l = 0; l < reply.childNodes.length; l++) {
-				var id = reply.childNodes[l].getAttribute("id");
-				var ctr = reply.childNodes[l].getAttribute("counter");
-	
-				var feedctr = f_document.getElementById("FEEDCTR-" + id);
-				var feedu = f_document.getElementById("FEEDU-" + id);
-				var feedr = f_document.getElementById("FEEDR-" + id);
-	
-				if (id == "global-unread") {
-					global_unread = ctr;
-					continue;
-				}
-	
-				if (feedctr && feedu && feedr) {
-	
-					feedu.innerHTML = ctr;
-		
-					if (ctr > 0) {
-						feedctr.className = "odd";
-						if (!feedr.className.match("Unread")) {
-							feedr.className = feedr.className + "Unread";
-						}
-					} else {
-						feedctr.className = "invisible";
-						feedr.className = feedr.className.replace("Unread", "");
-					}
-				}
-			}  
+			parse_counters(reply, f_document);
 	
 			updateTitle("");
 			notify("All feeds updated.");