batch feed editor: UI cleanup

1 files changed, 14 insertions, 12 deletions

diff --git a/prefs.js b/prefs.js
index 2f8ca4c3b..4f315297c 100644
--- a/prefs.js
+++ b/prefs.js
@@ -2101,7 +2101,14 @@ function feedActionChange() {
 function feedActionGo(op) {	
 	try {
 		if (op == "facEdit") {
-			editSelectedFeed();
+
+			var rows = getSelectedFeeds();
+
+			if (rows.length > 1) {
+				editSelectedFeeds();
+			} else {
+				editSelectedFeed();
+			}
 		}
 
 		if (op == "facClear") {
@@ -2124,10 +2131,6 @@ function feedActionGo(op) {
 			removeSelectedFeeds();
 		}
 
-		if (op == "facBatchEdit") {
-			editSelectedFeeds();
-		}
-
 	} catch (e) {
 		exception_error("feedActionGo", e);
 
@@ -2250,8 +2253,7 @@ function feedsEditSave() {
 			return
 		}
 
-//		var ok = confirm(__("Save changes to selected feeds?"));
-		var ok = true;
+		var ok = confirm(__("Save changes to selected feeds?"));
 
 		if (ok) {
 
@@ -2261,27 +2263,27 @@ function feedsEditSave() {
 
 			/* Form.serialize ignores unchecked checkboxes */
 
-			if (!query.match("hidden=") && 
+			if (!query.match("&hidden=") && 
 					f.hidden.disabled == false) {
 				query = query + "&hidden=false";
 			}
 
-			if (!query.match("rtl_content=") && 
+			if (!query.match("&rtl_content=") && 
 					f.rtl_content.disabled == false) {
 				query = query + "&rtl_content=false";
 			}
 
-			if (!query.match("private=") && 
+			if (!query.match("&private=") && 
 					f.private.disabled == false) {
 				query = query + "&private=false";
 			}
 
-			if (!query.match("cache_images=") && 
+			if (!query.match("&cache_images=") && 
 					f.cache_images.disabled == false) {
 				query = query + "&cache_images=false";
 			}
 
-			if (!query.match("include_in_digest=") && 
+			if (!query.match("&include_in_digest=") && 
 					f.include_in_digest.disabled == false) {
 				query = query + "&include_in_digest=false";
 			}