more i18n work, add some report notices

functions.js

@@ -1343,7 +1343,9 @@ function infobox_submit_callback() {
 			}
 		} catch (e) { }
 
-//		notify_info(xmlhttp.responseText);
+		if (xmlhttp.responseText) {
+			notify_info(xmlhttp.responseText);
+		}
 
 	} 
 }
@@ -1364,7 +1366,7 @@ function infobox_callback() {
 	}
 }
 
-function qaddFilter() {
+function addFilter() {
 
 	if (!xmlhttp_ready(xmlhttp)) {
 		printLockingError();
@@ -1400,7 +1402,7 @@ function isValidURL(s) {
 	return s.match("http://") != null || s.match("https://") != null || s.match("feed://") != null;
 }
 
-function qafAdd() {
+function qaddFeed() {
 
 	if (!xmlhttp_ready(xmlhttp)) {
 		printLockingError();

modules/popup-dialog.php

@@ -36,6 +36,7 @@
 
 			print "</div>";
 
+			return;
 		}
 
 		if ($id == "quickAddFeed") {
@@ -52,7 +53,7 @@
 			print "<table width='100%'>
 			<tr><td>Feed URL:</td><td>
 				<input class=\"iedit\" onblur=\"javascript:enableHotkeys()\" 
-					onkeypress=\"return filterCR(event, qafAdd)\"
+					onkeypress=\"return filterCR(event, qaddFeed)\"
 					onkeyup=\"toggleSubmitNotEmpty(this, 'fadd_submit_btn')\"
 					onchange=\"toggleSubmitNotEmpty(this, 'fadd_submit_btn')\"
 					onfocus=\"javascript:disableHotkeys()\" name=\"feed_url\"></td></tr>";
@@ -69,10 +70,12 @@
 			print "<div align='right'>
 				<input class=\"button\"
 					id=\"fadd_submit_btn\" disabled=\"true\"
-					type=\"submit\" onclick=\"return qafAdd()\" value=\"".__('Subscribe')."\">
+					type=\"submit\" onclick=\"return qaddFeed()\" value=\"".__('Subscribe')."\">
 				<input class=\"button\"
 					type=\"submit\" onclick=\"return closeInfoBox()\" 
 					value=\"".__('Cancel')."\"></div>";
+
+			return;
 		}
 
 		if ($id == "search") {
@@ -153,6 +156,8 @@
 
 			print "</div>";
 
+			return;
+
 		}
 
 		if ($id == "quickAddLabel") {
@@ -200,6 +205,8 @@
 			print "<input class=\"button\"
 				type=\"submit\" onclick=\"return labelEditCancel()\" 
 				value=\"".__('Cancel')."\">";
+
+			return;
 		}
 
 		if ($id == "quickAddFilter") {
@@ -279,7 +286,7 @@
 
 			print "<input type=\"submit\" 
 				id=\"infobox_submit\"
-				class=\"button\" onclick=\"return qaddFilter()\" 
+				class=\"button\" onclick=\"return addFilter()\" 
 				disabled=\"true\" value=\"".__('Create')."\"> ";
 
 			print "<input class=\"button\"
@@ -290,6 +297,7 @@
 
 //			print "</td></tr></table>"; 
 
+			return;
 		}
 
 		if ($id == "feedUpdateErrors") {
@@ -379,8 +387,13 @@
 
 			print "</div>";
 
+			return;
 		}
 
-		print "</div>";
+		print "<div id='infoBoxTitle'>Internal Error</div>
+			<div id='infoBoxContents'>
+			<p>Unknown dialog <b>$id</b></p>
+			</div></div>";
+	
 	}
 ?>

modules/pref-feeds.php

@@ -751,8 +751,7 @@
 				onclick=\"javascript:updateFeedList()\" value=\"".__('Search')."\">
 			</div>";
 		
-		print "<div class=\"prefGenericAddBox\">
+		print "<input id=\"fadd_link\" 
-			<input id=\"fadd_link\" 
 				onkeyup=\"toggleSubmitNotEmpty(this, 'fadd_submit_btn')\"
 				onchange=\"toggleSubmitNotEmpty(this, 'fadd_submit_btn')\"
 				size=\"40\">
@@ -765,8 +764,6 @@
 				onclick=\"javascript:browseFeeds()\" value=\"".__('Top 25')."\">";
 		}
 
-		print "</div>";
-
 		$feeds_sort = db_escape_string($_GET["sort"]);
 
 		if (!$feeds_sort || $feeds_sort == "undefined") {

modules/pref-filters.php

@@ -150,6 +150,10 @@
 					action_param = '$action_param'
 					WHERE id = '$filter_id' AND owner_uid = " . $_SESSION["uid"]);
 
+			if (db_affected_rows($link, $result) != 0) {
+				print_notice(T_sprintf("Saved filter <b>%s</b>", htmlspecialchars($reg_exp)));
+			}
+
 		}
 
 		if ($subop == "remove") {
@@ -186,7 +190,11 @@
 					('$regexp', '$filter_type','".$_SESSION["uid"]."', 
 						$feed_id, '$action_id', '$action_param', $inverse)");
 
-			print_notice(T_sprintf("Created filter <b>%s</b>", htmlspecialchars($regexp)));
+			if (db_affected_rows($link, $result) != 0) {
+				print T_sprintf("Created filter <b>%s</b>", htmlspecialchars($regexp));
+			}
+
+			return;
 		}
 
 		if ($quiet) return;

modules/pref-labels.php

@@ -153,6 +153,11 @@
 				sql_exp = '$sql_exp', 
 				description = '$descr'
 				WHERE id = '$label_id'");
+
+			if (db_affected_rows($link, $result) != 0) {
+				print_notice(T_sprintf("Saved label <b>%s</b>", htmlspecialchars($descr)));
+			}
+
 		}
 
 		if ($subop == "remove") {
@@ -170,18 +175,21 @@
 
 		if ($subop == "add") {
 
-			if (!WEB_DEMO_MODE) {
+			// no escaping is done here on purpose
+			$sql_exp = trim($_GET["sql_exp"]);
+			$description = db_escape_string($_GET["description"]);
 
-				// no escaping is done here on purpose
+			if (!$sql_exp || !$description) return;
-				$sql_exp = trim($_GET["sql_exp"]);
-				$description = db_escape_string($_GET["description"]);
 
-				if (!$sql_exp || !$description) return;
+			$result = db_query($link,
+				"INSERT INTO ttrss_labels (sql_exp,description,owner_uid) 
+				VALUES ('$sql_exp', '$description', '".$_SESSION["uid"]."')");
 
-				$result = db_query($link,
+			if (db_affected_rows($link, $result) != 0) {
-					"INSERT INTO ttrss_labels (sql_exp,description,owner_uid) 
+				print T_sprintf("Created label <b>%s</b>", htmlspecialchars($description));
-						VALUES ('$sql_exp', '$description', '".$_SESSION["uid"]."')");
 			}
+
+			return;
 		}
 
 		$sort = db_escape_string($_GET["sort"]);